Bug 137602 - Add test to check stretchy value is case sensitive
Summary: Add test to check stretchy value is case sensitive
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: MathML (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-10 02:33 PDT by Alejandro G. Castro
Modified: 2014-10-14 03:58 PDT (History)
5 users (show)

See Also:


Attachments
patch (1.63 KB, patch)
2014-10-10 02:37 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alejandro G. Castro 2014-10-10 02:33:53 PDT
It is not usual for other specs so it is good we have a test checking the sensitiveness of the attribute. This is a followup of the bug #136883.
Comment 1 Alejandro G. Castro 2014-10-10 02:37:24 PDT
Created attachment 239611 [details]
patch
Comment 2 Darin Adler 2014-10-10 09:46:56 PDT
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.
Comment 3 Alejandro G. Castro 2014-10-14 03:11:10 PDT
(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.
Comment 4 Alejandro G. Castro 2014-10-14 03:58:26 PDT
Landed http://trac.webkit.org/changeset/174680

Thanks!