Chromium Code Reviews Chromium Code Reviews
Issue 714083003:
Revert "Move some scroll invalidations to the paint invalidation phase" (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| OLD | NEW |
|---|---|
| (Empty) | |
| 1 <!DOCTYPE html> | |
| 2 <style> | |
| 3 #searchbar { | |
| 4 bottom: 0; /* Useful for seeing the issue visually. */ | |
| 5 position: fixed; | |
| 6 } | |
| 7 #recentlink { | |
| 8 background: purple; | |
| 9 position: absolute; | |
| 10 width: 150px; | |
| 11 height: 150px; | |
| 12 } | |
| 13 header { | |
| 14 padding: 10px; | |
| 15 position: relative; | |
| 16 z-index: 50; | |
| 17 -webkit-backface-visibility: hidden; | |
| 18 } | |
| 19 </style> | |
| 20 To manually test, just scroll down, there should be no purple trace. | |
| 21 <header> | |
| 22 <div id="searchbar"> | |
| 23 <div id="recentlink"></div> | |
| 24 </div> | |
| 25 </header> | |
| 26 | |
| 27 <!-- Used to have some overflowing content to scroll --> | |
| 28 <div style="height: 5000px"></div> | |
| 29 | |
| 30 <script src="resources/text-based-repaint.js" type="text/javascript"></script> | |
| 31 <script> | |
| 32 window.scrollTo(0, 1000); | |
| 33 | |
| 34 window.testIsAsync = true; | |
| 35 if (window.testRunner) | |
| 36 testRunner.waitUntilDone(); | |
| 37 | |
| 38 function repaintTest() | |
| 39 { | |
| 40 window.scrollBy(0, 400); | |
| 41 finishRepaintTest(); | |
| 42 } | |
| 43 function showBug() | |
| 44 { | |
| 45 window.requestAnimationFrame(function() { | |
| 46 window.requestAnimationFrame(function() { | |
| 47 runRepaintTest(); | |
| 48 }); | |
| 49 }); | |
| 50 } | |
| 51 showBug(); | |
| 52 //window.addEventListener("load", showBug, false); | |
|
dsinclair
2014/11/11 19:51:23
remove?
Julien - ping for review
2014/11/12 18:46:01
Indeed, cleaned up the test (showBug() wasn't usef
| |
| 53 | |
| 54 </script> | |
| 55 | |
| OLD | NEW |
