[Webkit-unassigned] [Bug 137602] Add test to check stretchy value is case sensitive

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Oct 10 09:47:01 PDT 2014


https://bugs.webkit.org/show_bug.cgi?id=137602


Darin Adler <darin at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #239611|review?                     |review+
               Flag|                            |




--- Comment #2 from Darin Adler <darin at apple.com>  2014-10-10 09:46:56 PST ---
(From update of attachment 239611)
Two thoughts:

1) We should also check that "false" is case sensitive. I presume there is some case where the default for stretchy is true, and "false" changes it back to false, but "False" does not.

2) Tests that explain what the results should look like are generally better than tests that don't. Even though this is a reference test it would be nice if it was self-explanatory in the browser. In many cases I put expected results in the original as well as the expected file so a human being can compare things visually, and then put the expected result twice in the expected.html file. For example, fast/text/international/old-turkic-direction.html takes that approach.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list