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

Side by Side Diff: third_party/WebKit/LayoutTests/fast/scroll-behavior/visual-viewport-scroll-no-onscroll-event.html

Issue 1413493005: Update layout tests to work when smooth scrolling is enabled by default (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: worked on nit Created 5 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
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 #target { 4 #target {
5 width: 2000px; 5 width: 2000px;
6 height: 1500px; 6 height: 1500px;
7 } 7 }
8 </style> 8 </style>
9 <script> 9 <script>
10 var numScrollsReceived = 0; 10 var numScrollsReceived = 0;
11 var numRAFCalls = 0; 11 var numRAFCalls = 0;
12 if (window.testRunner && window.internals) { 12 if (window.testRunner && window.internals) {
13 window.jsTestIsAsync = true; 13 window.jsTestIsAsync = true;
14 testRunner.dumpAsText(); 14 testRunner.dumpAsText();
15 testRunner.waitUntilDone(); 15 testRunner.waitUntilDone();
16 setPrintTestResultsLazily(); 16 setPrintTestResultsLazily();
17 } 17 }
18 18
19 description("This test ensures that scrolling the visual viewport doesn't\ 19 description("This test ensures that scrolling the visual viewport doesn't\
20 trigger an onscroll event."); 20 trigger an onscroll event.");
21 21
22 function runTest() { 22 function runTest() {
23 if (!window.testRunner || !window.internals) { 23 if (!window.testRunner || !window.internals) {
24 testFailed("This test requires test runner and internals") 24 testFailed("This test requires test runner and internals")
25 finishJSTest(); 25 finishJSTest();
26 return; 26 return;
27 } 27 }
28 28
29 internals.settings.setScrollAnimatorEnabled(false);
30
29 window.internals.setPageScaleFactor(2.0); 31 window.internals.setPageScaleFactor(2.0);
30 var target = document.getElementById('target'); 32 var target = document.getElementById('target');
31 33
32 document.onscroll = function() { 34 document.onscroll = function() {
33 ++numScrollsReceived; 35 ++numScrollsReceived;
34 debug('OnScroll called for scroll #' + numScrollsReceived); 36 debug('OnScroll called for scroll #' + numScrollsReceived);
35 } 37 }
36 38
37 var failureSentinel = function() { 39 var failureSentinel = function() {
38 if (numRAFCalls == 0) { 40 if (numRAFCalls == 0) {
(...skipping 15 matching lines...) Expand all
54 ++numRAFCalls; 56 ++numRAFCalls;
55 window.requestAnimationFrame(failureSentinel); 57 window.requestAnimationFrame(failureSentinel);
56 } 58 }
57 59
58 window.requestAnimationFrame(failureSentinel); 60 window.requestAnimationFrame(failureSentinel);
59 } 61 }
60 62
61 onload = runTest; 63 onload = runTest;
62 </script> 64 </script>
63 <div id="target"></div> 65 <div id="target"></div>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698