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

Side by Side Diff: third_party/WebKit/LayoutTests/fast/events/pointerevents/mouse-pointer-event-properties.html

Issue 2478313003: Fixed composed & detail fields in PointerEvents. (Closed)
Patch Set: Created 4 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 <script src="../resources/input-modifiers.js"></script> 3 <script src="../resources/input-modifiers.js"></script>
4 <style> 4 <style>
5 div.box { 5 div.box {
6 margin: 10px; 6 margin: 10px;
7 padding: 50px; 7 padding: 50px;
8 float: left; 8 float: left;
9 } 9 }
10 </style> 10 </style>
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
105 if (!checkKeyModifiers) { 105 if (!checkKeyModifiers) {
106 106
107 if (lastPointerEvents[0].type=="pointerenter" || lastPointerEvents[0]. type=="pointerleave") { 107 if (lastPointerEvents[0].type=="pointerenter" || lastPointerEvents[0]. type=="pointerleave") {
108 shouldBeFalse("lastPointerEvents[0].bubbles"); 108 shouldBeFalse("lastPointerEvents[0].bubbles");
109 shouldBeFalse("lastPointerEvents[0].cancelable"); 109 shouldBeFalse("lastPointerEvents[0].cancelable");
110 } else { 110 } else {
111 shouldBeTrue("lastPointerEvents[0].bubbles"); 111 shouldBeTrue("lastPointerEvents[0].bubbles");
112 shouldBeTrue("lastPointerEvents[0].cancelable"); 112 shouldBeTrue("lastPointerEvents[0].cancelable");
113 } 113 }
114 114
115 shouldBeTrue("lastPointerEvents[0].composed");
116 shouldBeEqualToNumber("lastPointerEvents[0].detail", 0);
117
115 shouldBeEqualToNumber("lastPointerEvents[0].pointerId", 118 shouldBeEqualToNumber("lastPointerEvents[0].pointerId",
116 (pointerType == "mouse")? 1 : penId); 119 (pointerType == "mouse")? 1 : penId);
117 shouldBeEqualToString("lastPointerEvents[0].pointerType", pointerType) ; 120 shouldBeEqualToString("lastPointerEvents[0].pointerType", pointerType) ;
118 shouldBeTrue("lastPointerEvents[0].isPrimary"); 121 shouldBeTrue("lastPointerEvents[0].isPrimary");
119 122
120 numericAttributes.forEach(function(attribute) { 123 numericAttributes.forEach(function(attribute) {
121 var expectedValue = getExpectedNumericAttributeValueForPE(event, att ribute); 124 var expectedValue = getExpectedNumericAttributeValueForPE(event, att ribute);
122 shouldBeEqualToNumber("lastPointerEvents[0]." + attribute, expectedV alue); 125 shouldBeEqualToNumber("lastPointerEvents[0]." + attribute, expectedV alue);
123 }); 126 });
124 127
(...skipping 24 matching lines...) Expand all
149 152
150 debug("--- move mouse into target ---"); 153 debug("--- move mouse into target ---");
151 eventSender.mouseMoveTo(x + 5, y + 5); 154 eventSender.mouseMoveTo(x + 5, y + 5);
152 debug(""); 155 debug("");
153 156
154 debug("--- move within target ---"); 157 debug("--- move within target ---");
155 eventSender.mouseMoveTo(x + 7, y + 15); 158 eventSender.mouseMoveTo(x + 7, y + 15);
156 eventSender.mouseMoveTo(x + 5, y + 5); 159 eventSender.mouseMoveTo(x + 5, y + 5);
157 debug(""); 160 debug("");
158 161
159 debug("--- click each button ---"); 162 debug("--- click left/right button ---");
160 for (var button = 0; button <=2; button++) { 163 // TODO(mustaq): Skip middle button because it triggers autoscroll. Consider
161 eventSender.mouseDown(button); 164 // bringing it back if we choose to prevent autoscrolling by canceling middle
162 eventSender.mouseUp(button); 165 // mousedown. crbug.com/644488.
163 // TODO(crbug.com/548226): Investigate missing events in win_chromium_rel_ng on 3rd down/up. 166 eventSender.mouseDown(0);
164 } 167 eventSender.mouseUp(0);
168 eventSender.mouseDown(2);
169 eventSender.mouseUp(2);
165 debug(""); 170 debug("");
166 171
167 debug("--- click with each modifier ---"); 172 debug("--- click with each modifier ---");
168 checkKeyModifiers = true; 173 checkKeyModifiers = true;
169 forEachModifier(function(attr, modifierName, eventSenderName) { 174 forEachModifier(function(attr, modifierName, eventSenderName) {
170 eventSender.mouseDown(0, [eventSenderName]); 175 eventSender.mouseDown(0, [eventSenderName]);
171 eventSender.mouseUp(0, [eventSenderName]); 176 eventSender.mouseUp(0, [eventSenderName]);
172 }); 177 });
173 checkKeyModifiers = false; 178 checkKeyModifiers = false;
174 debug(""); 179 debug("");
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
215 runPenTests(rect.left, rect.top, 3, true); 220 runPenTests(rect.left, rect.top, 3, true);
216 } 221 }
217 222
218 init(); 223 init();
219 if (window.eventSender) 224 if (window.eventSender)
220 runAllTests(); 225 runAllTests();
221 else 226 else
222 debug("This test requires eventSender"); 227 debug("This test requires eventSender");
223 228
224 </script> 229 </script>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698