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

Side by Side Diff: third_party/WebKit/LayoutTests/fast/css/class-name-no-matching-selector.html

Issue 1896793002: Rename (updateLayout/updateStyle).*.() to updateStyleAndLayout.*.() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 months 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
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 2
3 <style> 3 <style>
4 .exists1, .exists2 { } 4 .exists1, .exists2 { }
5 </style> 5 </style>
6 6
7 <script src="../../resources/js-test.js"></script> 7 <script src="../../resources/js-test.js"></script>
8 8
9 <div id="target"></div> 9 <div id="target"></div>
10 10
11 <script> 11 <script>
12 description("Test that adding and removing class names with no matching selector does not cause a style recalc"); 12 description("Test that adding and removing class names with no matching selector does not cause a style recalc");
13 13
14 var target = document.getElementById('target'); 14 var target = document.getElementById('target');
15 15
16 function testClass(oldClasses, newClasses, needsRecalc) 16 function testClass(oldClasses, newClasses, needsRecalc)
17 { 17 {
18 target.className = oldClasses; 18 target.className = oldClasses;
19 document.body.offsetTop; 19 document.body.offsetTop;
20 target.className = newClasses; 20 target.className = newClasses;
21 shouldBe("internals.updateStyleAndReturnAffectedElementCount()", needsRecalc ? '1' : '0'); 21 shouldBe("internals.updateStyleAndLayoutAndReturnAffectedElementCount()", ne edsRecalc ? '1' : '0');
22 } 22 }
23 23
24 testClass('', 'exists1', true); 24 testClass('', 'exists1', true);
25 testClass('exists1', '', true); 25 testClass('exists1', '', true);
26 testClass('', 'not-exists', false); 26 testClass('', 'not-exists', false);
27 testClass('not-exists', '', false); 27 testClass('not-exists', '', false);
28 testClass('exists1', 'exists2', true); 28 testClass('exists1', 'exists2', true);
29 testClass('exists1 exists2', 'exists2 exists1', false); 29 testClass('exists1 exists2', 'exists2 exists1', false);
30 testClass('exists1 exists1', 'exists1 exists1', false); 30 testClass('exists1 exists1', 'exists1 exists1', false);
31 testClass('exists1 exists1', 'exists1 exists1 doesnt-exist', false); 31 testClass('exists1 exists1', 'exists1 exists1 doesnt-exist', false);
32 testClass('exists1 exists1 doesnt-exist', 'exists1 exists1', false); 32 testClass('exists1 exists1 doesnt-exist', 'exists1 exists1', false);
33 testClass('exists1 exists1 doesnt-exist', 'exists1 doesnt-exist exists1', false) ; 33 testClass('exists1 exists1 doesnt-exist', 'exists1 doesnt-exist exists1', false) ;
34 testClass('exists1 doesnt-exist exists2 doesnt-exist exists1', 'doesnt-exist doe snt-exist2 exists2 exists1 doesnt-exist exists1', false); 34 testClass('exists1 doesnt-exist exists2 doesnt-exist exists1', 'doesnt-exist doe snt-exist2 exists2 exists1 doesnt-exist exists1', false);
35 </script> 35 </script>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698