Summary: | Add test to check stretchy value is case sensitive | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Alejandro G. Castro <alex> | ||||
Component: | MathML | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, darin, dbarton, fred.wang, mrobinson | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Alejandro G. Castro
2014-10-10 02:33:53 PDT
Created attachment 239611 [details]
patch
Comment on attachment 239611 [details]
patch
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.
(In reply to comment #2) > (From update of attachment 239611 [details]) > 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. Thanks for the proposals, I'll add both of them before landing. Landed http://trac.webkit.org/changeset/174680 Thanks! |