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

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

Issue 1416283006: Make window.scroll properties relative to the layout viewport by default. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: worked on review comments 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
(Empty)
1 <!DOCTYPE html>
2 <script src="../../resources/js-test.js"></script>
3 <style>
4 #target {
5 width: 2000px;
6 height: 1500px;
7 }
8 </style>
9 <script>
10 var numScrollsReceived = 0;
11 var numRAFCalls = 0;
12 if (window.testRunner && window.internals) {
13 window.jsTestIsAsync = true;
14 testRunner.dumpAsText();
15 testRunner.waitUntilDone();
16 setPrintTestResultsLazily();
17 }
18
19 description("This test ensures that scrolling the visual viewport doesn't\
20 trigger an onscroll event.");
21
22 function runTest() {
23 if (!window.testRunner || !window.internals) {
24 testFailed("This test requires test runner and internals")
25 finishJSTest();
26 return;
27 }
28
29 window.internals.setPageScaleFactor(2.0);
30 var target = document.getElementById('target');
31
32 document.onscroll = function() {
33 ++numScrollsReceived;
34 debug('OnScroll called for scroll #' + numScrollsReceived);
35 }
36
37 var failureSentinel = function() {
38 if (numRAFCalls == 0) {
39 // The first scroll should trigger onscroll because it scrolls
40 // the layout viewport.
41 eventSender.mouseMoveTo(target.offsetLeft + 5, target.offsetTop + 5);
42 eventSender.continuousMouseScrollBy(0, -400);
43 } else if (numRAFCalls == 1) {
44 shouldBe('numScrollsReceived', '1');
45 // The second scroll should not trigger onscroll because it only
46 // scrolls the visual viewport.
47 eventSender.mouseMoveTo(target.offsetLeft + 5, target.offsetTop + 5);
48 eventSender.continuousMouseScrollBy(0, 40);
49 } else {
50 shouldBe('numScrollsReceived', '1');
51 finishJSTest();
52 return;
53 }
54 ++numRAFCalls;
55 window.requestAnimationFrame(failureSentinel);
56 }
57
58 window.requestAnimationFrame(failureSentinel);
59 }
60
61 onload = runTest;
62 </script>
63 <div id="target"></div>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698