Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(618)

Side by Side Diff: LayoutTests/fast/css/invalidation/fieldset-disabled.html

Issue 703473003: Fix bug where form/fieldset :valid/:invalid won't be recalculated upon control's willValidate change (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Update a css layout test now that number of SubtreeStyleChange went down Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | LayoutTests/fast/css/invalidation/fieldset-disabled-expected.txt » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <script src="../../../resources/js-test.js"></script> 2 <script src="../../../resources/js-test.js"></script>
3 <style> 3 <style>
4 fieldset:disabled { background-color: green } 4 fieldset:disabled { background-color: green }
5 fieldset + div { color: pink } 5 fieldset + div { color: pink }
6 </style> 6 </style>
7 <fieldset id="fieldset"> 7 <fieldset id="fieldset">
8 <legend></legend> 8 <legend></legend>
9 <label for="field"></label> 9 <label for="field"></label>
10 <input type="text" name="field" id="field"> 10 <input type="text" name="field" id="field">
(...skipping 17 matching lines...) Expand all
28 var green = "rgb(0, 128, 0)"; 28 var green = "rgb(0, 128, 0)";
29 29
30 shouldBe("getComputedStyle(fieldset, '').backgroundColor", "transparent"); 30 shouldBe("getComputedStyle(fieldset, '').backgroundColor", "transparent");
31 31
32 fieldset.offsetTop; // Force recalc. 32 fieldset.offsetTop; // Force recalc.
33 fieldset.disabled = true; 33 fieldset.disabled = true;
34 34
35 if (window.internals) { 35 if (window.internals) {
36 // There are still instances of SubtreeStyleChange left when updating 36 // There are still instances of SubtreeStyleChange left when updating
37 // disabled state. This count should become lower. 37 // disabled state. This count should become lower.
38 shouldBe("internals.updateStyleAndReturnAffectedElementCount()", "7"); 38 shouldBe("internals.updateStyleAndReturnAffectedElementCount()", "2");
39 } 39 }
40 40
41 shouldBe("getComputedStyle(fieldset, '').backgroundColor", "green"); 41 shouldBe("getComputedStyle(fieldset, '').backgroundColor", "green");
42 </script> 42 </script>
OLDNEW
« no previous file with comments | « no previous file | LayoutTests/fast/css/invalidation/fieldset-disabled-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698