Chromium Code Reviews Chromium Code Reviews
Issue 235933002:
Defer updateCompositingRequirementsState until the compositing update (Closed)
Base URL: svn://svn.chromium.org/blink/trunk| OLD | NEW |
|---|---|
| 1 <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" | 1 <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" |
| 2 "http://www.w3.org/TR/html4/loose.dtd"> | 2 "http://www.w3.org/TR/html4/loose.dtd"> |
| 3 | 3 |
| 4 <html lang="en"> | 4 <html lang="en"> |
| 5 <head> | 5 <head> |
| 6 <meta http-equiv="Content-Type" content="text/html; charset=utf-8"/> | 6 <meta http-equiv="Content-Type" content="text/html; charset=utf-8"/> |
| 7 <title>Overflow, positioning and compositing</title> | 7 <title>Overflow, positioning and compositing</title> |
| 8 <style type="text/css" media="screen"> | 8 <style type="text/css" media="screen"> |
| 9 .container { | 9 .container { |
| 10 display: inline-block; | 10 display: inline-block; |
| (...skipping 17 matching lines...) Expand all Loading... | |
| 28 testRunner.waitUntilDone(); | 28 testRunner.waitUntilDone(); |
| 29 | 29 |
| 30 function doTest() | 30 function doTest() |
| 31 { | 31 { |
| 32 var scrollables = document.querySelectorAll('.container'); | 32 var scrollables = document.querySelectorAll('.container'); |
| 33 for (var i = 0; i < scrollables.length; ++i) | 33 for (var i = 0; i < scrollables.length; ++i) |
| 34 { | 34 { |
| 35 scrollables[i].scrollTop = 50; | 35 scrollables[i].scrollTop = 50; |
| 36 } | 36 } |
| 37 | 37 |
| 38 if (window.testRunner) | 38 requestAnimationFrame(function() { |
| 39 testRunner.notifyDone(); | 39 setTimeout(function() { |
|
Ian Vollick
2014/04/11 21:09:37
why the timeout? could you use internals.forceComp
| |
| 40 if (window.testRunner) | |
| 41 testRunner.notifyDone(); | |
| 42 }); | |
| 43 }); | |
| 40 } | 44 } |
| 41 | 45 |
| 42 window.addEventListener('load', doTest, false); | 46 window.addEventListener('load', doTest, false); |
| 43 </script> | 47 </script> |
| 44 </head> | 48 </head> |
| 45 | 49 |
| 46 <body> | 50 <body> |
| 47 | 51 |
| 48 <p>All of the boxes should look the same.</p> | 52 <p>All of the boxes should look the same.</p> |
| 49 <div class="container"> | 53 <div class="container"> |
| (...skipping 95 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 145 </div> | 149 </div> |
| 146 </div> | 150 </div> |
| 147 | 151 |
| 148 | 152 |
| 149 | 153 |
| 150 | 154 |
| 151 | 155 |
| 152 </body> | 156 </body> |
| 153 </html> | 157 </html> |
| 154 | 158 |
| OLD | NEW |
