Convert *-appearance-pseudo-classes.html to dumpAsText tests
Created attachment 176649 [details] Patch
Comment on attachment 176649 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=176649&action=review > LayoutTests/fast/forms/datetimelocal/datetimelocal-pseudo-classes.html:33 > +shouldBeFalse('matchedNodesContainId(container, ":out-of-range", "outofrange")'); You might want to also check: shouldBeTrue('matchedNodesContainId(container, ":in-range", "outofrange")'); You might want to also check: document.getElementById('inrange').value = '1500-01-01T00:00'; shouldBeFalse('matchedNodesContainId(container, ":in-range", "inrange")'); shouldBeTrue('matchedNodesContainId(container, ":out-of-range", "inrange")'); > LayoutTests/fast/forms/month/month-pseudo-classes.html:33 > +shouldBeFalse('matchedNodesContainId(container, ":out-of-range", "outofrange")'); Ditto. > LayoutTests/fast/forms/time/time-pseudo-classes.html:33 > +shouldBeFalse('matchedNodesContainId(container, ":out-of-range", "outofrange")'); Ditto. > LayoutTests/fast/forms/week/week-pseudo-classes.html:33 > +shouldBeFalse('matchedNodesContainId(container, ":out-of-range", "outofrange")'); Ditto.
Created attachment 176663 [details] Patch 2 Follow reviewer's comments
Comment on attachment 176649 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=176649&action=review >> LayoutTests/fast/forms/datetimelocal/datetimelocal-pseudo-classes.html:33 >> +shouldBeFalse('matchedNodesContainId(container, ":out-of-range", "outofrange")'); > > You might want to also check: > > shouldBeTrue('matchedNodesContainId(container, ":in-range", "outofrange")'); > > You might want to also check: > > document.getElementById('inrange').value = '1500-01-01T00:00'; > shouldBeFalse('matchedNodesContainId(container, ":in-range", "inrange")'); > shouldBeTrue('matchedNodesContainId(container, ":out-of-range", "inrange")'); Done.
Comment on attachment 176663 [details] Patch 2 Looks good.
Comment on attachment 176663 [details] Patch 2 Thanks!
Comment on attachment 176663 [details] Patch 2 Clearing flags on attachment: 176663 Committed r136110: <http://trac.webkit.org/changeset/136110>
All reviewed patches have been landed. Closing bug.