[iOS] fast/events/ios/rotation/orientationchange-event-listener-on.body.html is flaky
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 11 Jul 2019 23:47:06 +0000 (23:47 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 11 Jul 2019 23:47:06 +0000 (23:47 +0000)
https://bugs.webkit.org/show_bug.cgi?id=199732
<rdar://problem/52693087>

Reviewed by Alex Christensen.

Stop logging in each event listener to not be reliant on the order in which listener are called.
Instead, simply make sure all event listeners that should be called were actually called.

* fast/events/ios/rotation/orientationchange-event-listener-on.body-expected.txt:
* fast/events/ios/rotation/orientationchange-event-listener-on.body.html:

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

LayoutTests/ChangeLog
LayoutTests/fast/events/ios/rotation/orientationchange-event-listener-on.body-expected.txt
LayoutTests/fast/events/ios/rotation/orientationchange-event-listener-on.body.html

index 92e5f18..441c0dc 100644 (file)
@@ -1,3 +1,17 @@
+2019-07-11  Chris Dumez  <cdumez@apple.com>
+
+        [iOS] fast/events/ios/rotation/orientationchange-event-listener-on.body.html is flaky
+        https://bugs.webkit.org/show_bug.cgi?id=199732
+        <rdar://problem/52693087>
+
+        Reviewed by Alex Christensen.
+
+        Stop logging in each event listener to not be reliant on the order in which listener are called.
+        Instead, simply make sure all event listeners that should be called were actually called.
+
+        * fast/events/ios/rotation/orientationchange-event-listener-on.body-expected.txt:
+        * fast/events/ios/rotation/orientationchange-event-listener-on.body.html:
+
 2019-07-11  Ryosuke Niwa  <rniwa@webkit.org>
 
         Make the test added in r247327 more reliable.
index 85c5edb..97f2e06 100644 (file)
@@ -4,11 +4,8 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
 
 
 Before rotation
-PASS In body's onorientationchange event handler
-PASS In window's orientationchange event listener
-PASS In body's onresize event handler
-PASS In window's resize event listener
 After rotation
+PASS eventListenerCalls is 4
 PASS successfullyParsed is true
 
 TEST COMPLETE
index f5b2d66..1c50c75 100644 (file)
 
             testRunner.runUIScript(getRotationUIScript(), function(result) {
                 debug('After rotation');
-
+                shouldBe("eventListenerCalls", "4");
                 finishJSTest();
             });
         }
+
+        eventListenerCalls = 0;
         window.addEventListener('resize', function() {
-            testPassed("In window's resize event listener");
+            ++eventListenerCalls;
         }, false);
 
         window.addEventListener('orientationchange', function() {
-            testPassed("In window's orientationchange event listener");
+            ++eventListenerCalls;
         }, false);
 
         document.body.addEventListener('resize', function() {
             testFailed("In body's resize event listener");
+            ++eventListenerCalls;
         }, false);
 
         document.body.addEventListener('orientationchange', function() {
             testFailed("In body's orientationchange event listener");
+            ++eventListenerCalls;
         }, false);
 
         function bodyOrientationChange()
         {
-            testPassed("In body's onorientationchange event handler");
+            ++eventListenerCalls;
         }
 
         function bodyResize()
         {
-            testPassed("In body's onresize event handler");
+            ++eventListenerCalls;
         }
 
         window.addEventListener('load', doTest, false);