REGRESSION(r214426): [GTK] Test media/video-click-dblckick-standalone.html started...
authorcarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 10 Apr 2017 11:44:21 +0000 (11:44 +0000)
committercarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 10 Apr 2017 11:44:21 +0000 (11:44 +0000)
https://bugs.webkit.org/show_bug.cgi?id=170667

Reviewed by Žan Doberšek.

Tools:

In r214666 we disabled modern media controls runtime feature for the GTK+ port, but layout tests still override
that value.

* WebKitTestRunner/gtk/TestControllerGtk.cpp:
(WTR::TestController::updatePlatformSpecificTestOptionsForTest): Disable modern media controls.

LayoutTests:

Unskip then test that is passing now.

* platform/gtk/TestExpectations:

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

LayoutTests/ChangeLog
LayoutTests/platform/gtk/TestExpectations
Tools/ChangeLog
Tools/WebKitTestRunner/gtk/TestControllerGtk.cpp

index 46cd721..9f60b8a 100644 (file)
@@ -1,5 +1,16 @@
 2017-04-10  Carlos Garcia Campos  <cgarcia@igalia.com>
 
+        REGRESSION(r214426): [GTK] Test media/video-click-dblckick-standalone.html started to fail in the bots after r214426
+        https://bugs.webkit.org/show_bug.cgi?id=170667
+
+        Reviewed by Žan Doberšek.
+
+        Unskip then test that is passing now.
+
+        * platform/gtk/TestExpectations:
+
+2017-04-10  Carlos Garcia Campos  <cgarcia@igalia.com>
+
         Unreviewed GTK+ gardening. Mark several tests as failing/timing out.
 
         * platform/gtk/TestExpectations:
index fb06003..3fdb4dc 100644 (file)
@@ -3495,8 +3495,6 @@ webkit.org/b/170053 media/video-load-require-user-gesture.html [ Failure ]
 webkit.org/b/170335 canvas/philip/tests/2d.pattern.animated.gif.html [ Failure ]
 webkit.org/b/170335 svg/animations/animated-svg-image-outside-viewport-paused.html [ Failure ]
 
-webkit.org/b/170667 media/video-click-dblckick-standalone.html [ Failure ]
-
 #////////////////////////////////////////////////////////////////////////////////////////
 # End of non-crashing, non-flaky tests failing
 #////////////////////////////////////////////////////////////////////////////////////////
index 53621f8..96ff368 100644 (file)
@@ -1,5 +1,18 @@
 2017-04-10  Carlos Garcia Campos  <cgarcia@igalia.com>
 
+        REGRESSION(r214426): [GTK] Test media/video-click-dblckick-standalone.html started to fail in the bots after r214426
+        https://bugs.webkit.org/show_bug.cgi?id=170667
+
+        Reviewed by Žan Doberšek.
+
+        In r214666 we disabled modern media controls runtime feature for the GTK+ port, but layout tests still override
+        that value.
+
+        * WebKitTestRunner/gtk/TestControllerGtk.cpp:
+        (WTR::TestController::updatePlatformSpecificTestOptionsForTest): Disable modern media controls.
+
+2017-04-10  Carlos Garcia Campos  <cgarcia@igalia.com>
+
         WTR: Avoid conversion from platform image to WKImage and then to platform image again when dumping pixel results
         https://bugs.webkit.org/show_bug.cgi?id=170653
 
index 60bc99b..d784972 100644 (file)
@@ -155,8 +155,9 @@ void TestController::platformResetPreferencesToConsistentValues()
     m_mainWebView->dismissAllPopupMenus();
 }
 
-void TestController::updatePlatformSpecificTestOptionsForTest(TestOptions&, const std::string&) const
+void TestController::updatePlatformSpecificTestOptionsForTest(TestOptions& options, const std::string&) const
 {
+    options.enableModernMediaControls = false;
 }
 
 } // namespace WTR