Flaky Test: plugins/mouse-events.html
authorap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Sep 2013 20:42:44 +0000 (20:42 +0000)
committerap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Sep 2013 20:42:44 +0000 (20:42 +0000)
https://bugs.webkit.org/show_bug.cgi?id=116665

Reviewed by Anders Carlsson.

* plugins/mouse-events-expected.txt:
* plugins/mouse-events.html:
This test used async calls for events, and then usually logged responses while waiting
for a plg.eventLoggingEnabled setter synchronously. Depending on sync/async event
order is not reliable, especially with old WebKit1 plug-in IPC code.

* platform/mac-wk2/plugins/mouse-events-expected.txt: For some unclear reason,
focus works differently in WebKit2.

* platform/gtk/plugins/mouse-events-expected.txt: Removed.
* platform/qt/plugins/mouse-events-expected.txt: Removed.
* platform/win/plugins/mouse-events-expected.txt: Removed.
These results looked similar to new cross-platform ones, hopefully custom results
won't be needed any more.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@155646 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/platform/gtk/plugins/mouse-events-expected.txt [deleted file]
LayoutTests/platform/mac-wk2/plugins/mouse-events-expected.txt
LayoutTests/platform/qt/plugins/mouse-events-expected.txt [deleted file]
LayoutTests/platform/win/plugins/mouse-events-expected.txt [deleted file]
LayoutTests/plugins/mouse-events-expected.txt
LayoutTests/plugins/mouse-events.html

index ed3a7f1..8f5505a 100644 (file)
@@ -1,3 +1,25 @@
+2013-09-12  Alexey Proskuryakov  <ap@apple.com>
+
+        Flaky Test: plugins/mouse-events.html
+        https://bugs.webkit.org/show_bug.cgi?id=116665
+
+        Reviewed by Anders Carlsson.
+
+        * plugins/mouse-events-expected.txt:
+        * plugins/mouse-events.html:
+        This test used async calls for events, and then usually logged responses while waiting
+        for a plg.eventLoggingEnabled setter synchronously. Depending on sync/async event
+        order is not reliable, especially with old WebKit1 plug-in IPC code.
+
+        * platform/mac-wk2/plugins/mouse-events-expected.txt: For some unclear reason,
+        focus works differently in WebKit2.
+
+        * platform/gtk/plugins/mouse-events-expected.txt: Removed.
+        * platform/qt/plugins/mouse-events-expected.txt: Removed.
+        * platform/win/plugins/mouse-events-expected.txt: Removed.
+        These results looked similar to new cross-platform ones, hopefully custom results
+        won't be needed any more.
+
 2013-09-12  Gurpreet Kaur  <k.gurpreet@samsung.com>
 
         CSS Unit vw in border-width maps to 0px.
diff --git a/LayoutTests/platform/gtk/plugins/mouse-events-expected.txt b/LayoutTests/platform/gtk/plugins/mouse-events-expected.txt
deleted file mode 100644 (file)
index e717574..0000000
+++ /dev/null
@@ -1,7 +0,0 @@
-CONSOLE MESSAGE: PLUGIN: getFocusEvent
-CONSOLE MESSAGE: PLUGIN: mouseDown at (12, 12)
-CONSOLE MESSAGE: PLUGIN: mouseUp at (12, 12)
-CONSOLE MESSAGE: PLUGIN: mouseDown at (22, 22)
-CONSOLE MESSAGE: PLUGIN: mouseUp at (32, 22)
-
-Test for bug 11517: Flash clicks/interactivity not working properly.
index 59c5aca..e3c4b75 100644 (file)
@@ -1,6 +1,12 @@
-CONSOLE MESSAGE: PLUGIN: mouseDown at (12, 12)
-CONSOLE MESSAGE: PLUGIN: mouseUp at (12, 12)
-CONSOLE MESSAGE: PLUGIN: mouseDown at (22, 22)
-CONSOLE MESSAGE: PLUGIN: mouseUp at (32, 22)
 
 Test for bug 11517: Flash clicks/interactivity not working properly.
+
+PLUGIN: mouseDown at (12, 12)
+PLUGIN: mouseUp at (12, 12)
+PLUGIN: mouseDown at (22, 22)
+PLUGIN: mouseUp at (32, 22)
+PLUGIN: getFocusEvent
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
diff --git a/LayoutTests/platform/qt/plugins/mouse-events-expected.txt b/LayoutTests/platform/qt/plugins/mouse-events-expected.txt
deleted file mode 100644 (file)
index e717574..0000000
+++ /dev/null
@@ -1,7 +0,0 @@
-CONSOLE MESSAGE: PLUGIN: getFocusEvent
-CONSOLE MESSAGE: PLUGIN: mouseDown at (12, 12)
-CONSOLE MESSAGE: PLUGIN: mouseUp at (12, 12)
-CONSOLE MESSAGE: PLUGIN: mouseDown at (22, 22)
-CONSOLE MESSAGE: PLUGIN: mouseUp at (32, 22)
-
-Test for bug 11517: Flash clicks/interactivity not working properly.
diff --git a/LayoutTests/platform/win/plugins/mouse-events-expected.txt b/LayoutTests/platform/win/plugins/mouse-events-expected.txt
deleted file mode 100644 (file)
index 4c89588..0000000
+++ /dev/null
@@ -1,7 +0,0 @@
-CONSOLE MESSAGE: PLUGIN: getFocusEvent
-CONSOLE MESSAGE: PLUGIN: mouseDown at (20, 20)
-CONSOLE MESSAGE: PLUGIN: mouseUp at (20, 20)
-CONSOLE MESSAGE: PLUGIN: mouseDown at (30, 30)
-CONSOLE MESSAGE: PLUGIN: mouseUp at (40, 30)
-
-Test for bug 11517: Flash clicks/interactivity not working properly.
index 09dd8ad..876817e 100644 (file)
@@ -1,7 +1,12 @@
-CONSOLE MESSAGE: line 27: PLUGIN: getFocusEvent
-CONSOLE MESSAGE: line 27: PLUGIN: mouseDown at (12, 12)
-CONSOLE MESSAGE: line 27: PLUGIN: mouseUp at (12, 12)
-CONSOLE MESSAGE: line 27: PLUGIN: mouseDown at (22, 22)
-CONSOLE MESSAGE: line 27: PLUGIN: mouseUp at (32, 22)
 
 Test for bug 11517: Flash clicks/interactivity not working properly.
+
+PLUGIN: getFocusEvent
+PLUGIN: mouseDown at (12, 12)
+PLUGIN: mouseUp at (12, 12)
+PLUGIN: mouseDown at (22, 22)
+PLUGIN: mouseUp at (32, 22)
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
index 60053fa..53e99f2 100644 (file)
@@ -1,18 +1,41 @@
 <html>
+<head>
+<script src="../resources/js-test-pre.js"></script>
+</script>
+</head>
 <body>
 <embed name="plg" type="application/x-webkit-test-netscape" windowedPlugin="false" width=100 height=100></embed>
 <p>Test for <a href="https://bugs.webkit.org/show_bug.cgi?id=11517">bug 11517</a>:
 Flash clicks/interactivity not working properly.</p>
+<div id=console></div>
 <script>
+    window.jsTestIsAsync = true;
 
     plg.windowedPlugin = false;
     plg.eventLoggingEnabled = true;
 
+    // Our test plug-in logs events through window.console. Override it with a custom
+    // implementation that counts the number of messages - due to the nature of IPC, it's
+    // difficult to predict when events get handled.
+    const expectedMessageCount = 5;
+    var messageCount = 0;
+    window.console = {
+        log: function(message) {
+            if (messageCount == expectedMessageCount) {
+                alert("Message unexpectedly leaked from preceding test, plugins/mouse-events.html: " + message);
+                return;
+            }
+
+            debug(message);
+
+            if (++messageCount == expectedMessageCount)
+                finishJSTest();
+        }
+    };
+
     if (!window.testRunner) {
-        document.write("This test does not work in manual mode.");
+        document.write("To test manually, please install WebKit test plug-in, and verify that mouse events get logged.");
     } else {
-        testRunner.dumpAsText();
-        
         eventSender.mouseMoveTo(0,0);
         eventSender.mouseMoveTo(20,20);
         eventSender.mouseDown();
@@ -23,9 +46,7 @@ Flash clicks/interactivity not working properly.</p>
         eventSender.mouseUp();
         eventSender.mouseMoveTo(0,0);
     }
-
-    plg.eventLoggingEnabled = false; // stop logging so our output doesn't bleed into the next test
-
 </script>
+<script src="../resources/js-test-post.js"></script>
 </body>
 </html>