LayoutTests/fast/events/touch/gesture/touch-gesture-noscroll-body.html - Issue 26472003: Fix improper scrollTop/Left usage in LayoutTests/

Side by Side Diff: LayoutTests/fast/events/touch/gesture/touch-gesture-noscroll-body.html

Issue 26472003: Fix improper scrollTop/Left usage in LayoutTests/ (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@issue_304768

Patch Set: Fix improper scrollTop/Left usage in LayoutTests/ Created 7 years, 2 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

« LayoutTests/fast/dom/resources/check-scroll-position-onload.html ('K') | « LayoutTests/fast/events/mouse-cursor-multiframecur.html ('k') | LayoutTests/fast/events/touch/gesture/touch-gesture-noscroll-body-expected.txt » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 <!DOCTYPE html>	1 <!DOCTYPE html>

2 <html>	2 <html>

3 <head>	3 <head>

4 <link rel="stylesheet" href="../../../js/resources/js-test-style.css">	4 <link rel="stylesheet" href="../../../js/resources/js-test-style.css">

5 <script src="../../../js/resources/js-test-pre.js"></script>	5 <script src="../../../js/resources/js-test-pre.js"></script>

6 <script src="resources/gesture-helpers.js"></script>	6 <script src="resources/gesture-helpers.js"></script>

7 <style type="text/css">	7 <style type="text/css">

8 #touchtarget {	8 #touchtarget {

9 width: 100px;	9 width: 100px;

10 height: 100px;	10 height: 100px;

(...skipping 46 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
57 <script type="text/javascript">	57 <script type="text/javascript">

58	58

59 var expectedGesturesTotal = 2;	59 var expectedGesturesTotal = 2;

60 var gesturesOccurred = 0;	60 var gesturesOccurred = 0;

61 var scrollAmountX = ['0', '0'];	61 var scrollAmountX = ['0', '0'];

62 var scrollAmountY = ['0', '0'];	62 var scrollAmountY = ['0', '0'];

63 var wheelEventsOccurred = 0;	63 var wheelEventsOccurred = 0;

64 var expectedWheelEventsOccurred = ['0', '0'];	64 var expectedWheelEventsOccurred = ['0', '0'];

65 var scrollEventsOccurred = 0;	65 var scrollEventsOccurred = 0;

66 var expectedScrollEventsOccurred = '0';	66 var expectedScrollEventsOccurred = '0';

67 var scrolledElement = 'document.body'	67 var scrolledElement = 'document.documentElement'
	Dan Beam 2013/10/08 19:25:30 nit: ; at end, also where is this used? nit: ; at end, also where is this used? tonikitoo_ 2013/10/08 19:52:32 will add. Show quoted text On 2013/10/08 19:25:30, Dan Beam wrote: > nit: ; at end, also where is this used? will add.
68	68

69 // Always construct a page larger than the vertical height of the window.	69 // Always construct a page larger than the vertical height of the window.

70 function buildPage()	70 function buildPage()

71 {	71 {

72 // build vertical table	72 // build vertical table

73 var table = document.getElementById('vertical_table_to_fill');	73 var table = document.getElementById('vertical_table_to_fill');

74 var targetHeight = document.body.offsetHeight;	74 var targetHeight = document.body.offsetHeight;

75 var cellPairHeight = table.offsetHeight;	75 var cellPairHeight = table.offsetHeight;

76 var numberOfReps = targetHeight / cellPairHeight * 2;	76 var numberOfReps = targetHeight / cellPairHeight * 2;

77 var i;	77 var i;

(...skipping 79 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
157 successfullyParsed = true;	157 successfullyParsed = true;

158 isSuccessfullyParsed();	158 isSuccessfullyParsed();

159 if (window.testRunner)	159 if (window.testRunner)

160 testRunner.notifyDone();	160 testRunner.notifyDone();

161 }	161 }

162 }	162 }

163 }	163 }

164 </script>	164 </script>

165 </body>	165 </body>

166 </html>	166 </html>

OLD	NEW