Fix mouse-event-buttons-attribute flakiness

third_party/WebKit/LayoutTests/fast/events/mouse-event-buttons-attribute.html

Index: third_party/WebKit/LayoutTests/fast/events/mouse-event-buttons-attribute.html
diff --git a/third_party/WebKit/LayoutTests/fast/events/mouse-event-buttons-attribute.html b/third_party/WebKit/LayoutTests/fast/events/mouse-event-buttons-attribute.html
index cc8ad65bc1bfbe51e752dad9836bc6862603a9ca..2c125e9fa2242882482fb78013b2216de91de58b 100644
--- a/third_party/WebKit/LayoutTests/fast/events/mouse-event-buttons-attribute.html
+++ b/third_party/WebKit/LayoutTests/fast/events/mouse-event-buttons-attribute.html
@@ -14,9 +14,9 @@
   top: 100px;
 }
 </style>
-
 <div id="target"></div>
 <img id="drag" src="resources/greenbox30.png" draggable>
+
 <script src="../../resources/js-test.js"></script>
 <script>
 
@@ -24,8 +24,6 @@ const L = 'leftButton';
 const R = 'rightButton';
 const M = 'middleButton';
 
-var targetDiv = document.getElementById('target');
-var dragImg = document.getElementById('drag');
 var seqNo = 0;
 var testSet = [
   { initialButtons: [],        action: doubleClickAction },
@@ -125,6 +123,8 @@ function printTestInfo(testItem)
 }
 
 function init() {
+  var targetDiv = document.getElementById('target');
+  var dragImg = document.getElementById('drag');
   var eventList = ['dblclick', 'click', 'mousedown', 'mouseup', 'mousemove', 'mouseenter', 'mouseleave', 'mouseover', 'mouseout', 'mousewheel', 'dragstart', 'dragend', 'dragenter', 'dragleave', 'dragover', 'drag', 'contextmenu'];
   eventList.forEach(function(eventName) {
     targetDiv.addEventListener(eventName, function(event) {
@@ -136,28 +136,33 @@ function init() {
   });
 }
 
-function runTests() {
-  for (var i = 0; i < testSet.length; i++) {
-    printTestInfo(testSet[i]);
+window.onload=function() {
+  init();
+  if(window.eventSender) {
+    for(var i=0; i<100; i++) {
+	    eventSender.mouseMoveTo(500+i, 500+i);

[mustaq, 2016/03/10 14:51:13]

Why do you need this loop?

Nit: there's a tab char at the beginning of line.

[Navid Zolghadr, 2016/03/10 15:55:26]

Oops. This whole for loop was just a test thing that slipped into the commit.
I'll remove it.

+    }
 
-    // Reset the state for the new test
-    seqNo = 0;
-    eventSender.setMouseButtonState(-1, testSet[i].initialButtons);
-    internals.settings.setShowContextMenuOnMouseUp(testSet[i].showContextMenuOnMouseUp);
+    for (var i = 0; i < testSet.length; i++) {
+      printTestInfo(testSet[i]);
 
-    testSet[i].action();
+      // Reset the state for the new test
+      seqNo = 0;
+      eventSender.setMouseButtonState(-1, testSet[i].initialButtons);
+      internals.settings.setShowContextMenuOnMouseUp(testSet[i].showContextMenuOnMouseUp);
 
-    // Reset the mouse position
-    eventSender.mouseMoveTo(-1, -1);
+      testSet[i].action();
 
-    debug('');
+      // Reset the mouse position
+      eventSender.mouseMoveTo(-1, -1);
+
+      debug('');
+    }
   }
 }
 
-init();
-if (window.eventSender)
-  runTests();
-else
+if (!window.eventSender) {
   debug("This test requires eventSender");
+}
 
 </script>