Status: Untriaged
Owner: [email protected]
CC: [email protected]
Labels: Type-Bug Pri-2 OS-All Area-Misc

New issue 5737 by [email protected]: RenderButton merge broke handful of  
layout tests
http://code.google.com/p/chromium/issues/detail?id=5737

Specifically:

LINUX WIN : LayoutTests/fast/box-sizing/percentage-height.html = FAIL
LINUX WIN : LayoutTests/fast/forms/001.html = FAIL
LINUX WIN : LayoutTests/fast/forms/button-default-title.html = FAIL
LINUX WIN : LayoutTests/fast/forms/control-clip.html = FAIL
LINUX WIN : LayoutTests/fast/forms/input-first-letter.html = FAIL
LINUX WIN : LayoutTests/fast/forms/select-baseline.html = FAIL

RenderButton::setupInnerStyle had:

if (true || ...)

the removal of true from the if block seems to have triggered the failures.

I suspect we should just rebaseline the tests, but I need to verify that
with Ojan, who originally had a TODO around removing this code.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Chromium-bugs" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/chromium-bugs?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to