[GTK] run-gtk-tests should use the driver environment for checking the accessibility bus
[WebKit-https.git] / Tools / ChangeLog
index f8874ad..f86470e 100644 (file)
@@ -1,3 +1,337 @@
+2016-08-26  Carlos Alberto Lopez Perez  <clopez@igalia.com>
+
+        [GTK] run-gtk-tests should use the driver environment for checking the accessibility bus
+        https://bugs.webkit.org/show_bug.cgi?id=161149
+
+        Reviewed by Carlos Garcia Campos.
+
+        Add a new function for waiting for the accesibility bus. It sets
+        the test environment variables before starting the GLib mainloop.
+        And restores the previous environment after it has finished.
+        It also adds a timeout (5 seconds) to abort in case it has not
+        been able to detect the accesibility bus before the timeout expires.
+
+        * Scripts/run-gtk-tests:
+        (TestRunner):
+        (TestRunner._wait_for_accessibility_bus):
+        (TestRunner._wait_for_accessibility_bus.timeout_accessibility_bus):
+        (TestRunner._start_accessibility_daemons):
+        (TestRunner._get_tests_from_google_test_suite): Run this command also with the driver test environment.
+
+2016-08-26  Per Arne Vollan  <pvollan@apple.com>
+
+        [Win] Warning fix.
+        https://bugs.webkit.org/show_bug.cgi?id=161236
+
+        Reviewed by Brent Fulgham.
+
+        The & operator in the Test struct has no body.
+
+        * TestWebKitAPI/Tests/WTF/Vector.cpp:
+        (TestWebKitAPI::TEST):
+
+2016-08-26  Carlos Alberto Lopez Perez  <clopez@igalia.com>
+
+        [GTK] run-gtk-tests should use webkitpy.port.gtk and webkitpy.port.xvfbdriver
+        https://bugs.webkit.org/show_bug.cgi?id=160736
+
+        Reviewed by Carlos Garcia Campos.
+
+        This plugs run-gtk-tests into the webkitpy testing infrastructure.
+        As a benefit of this, run-gtk-tests now can be executed over any of
+        the existing supported gtk test display servers (xvfb, weston-over-xvfb,
+        or native xorg). A new command line switch --wayland is added for
+        run-gtk-tests.
+
+        * Scripts/run-gtk-tests:
+        (TestRunner.__init__): Remove the --display command line switch, its not longer useful.
+        Add a --wayland command line switch.
+        (TestRunner._start_accessibility_daemons):
+        (TestRunner):
+        (TestRunner.__init__):
+        (TestRunner._run_xvfb): Deleted.
+        (TestRunner._create_driver):
+        (TestRunner._setup_testing_environment): Delete the environment variable definitions that
+        are now done by driver._setup_environ_for_test().
+        (TestRunner._tear_down_testing_environment):
+        * Scripts/webkitpy/port/driver.py:
+        (Driver.__init__): Define self._server_name in the constructor of the base class.
+        It is used by several functions, so it makes sense to store once the value here.
+        (Driver._setup_environ_for_test): Factorize setting the environment for a given test.
+        (Driver._start): Get the environment from self._setup_environ_for_test()
+        * Scripts/webkitpy/port/driver_unittest.py:
+        (DriverTest):
+        (DriverTest.test_setup_environ_for_test): Add unittest for driver._setup_environ_for_test()
+        * Scripts/webkitpy/port/gtk.py: Move the inspection of the env var USE_NATIVE_XDISPLAY to the constructor.
+        (GtkPort.__init__):
+        (GtkPort._driver_class):
+        * Scripts/webkitpy/port/westondriver.py: Define and use self._setup_environ_for_test()
+        (WestonDriver._setup_environ_for_test):
+        (WestonDriver._start):
+        * Scripts/webkitpy/port/xorgdriver.py: Define and use self._setup_environ_for_test()
+        (XorgDriver._setup_environ_for_test):
+        (XorgDriver._start):
+        * Scripts/webkitpy/port/xvfbdriver.py: Define and use self._setup_environ_for_test()
+        (XvfbDriver._setup_environ_for_test):
+        (XvfbDriver._start):
+
+2016-08-26  Csaba Osztrogon√°c  <ossy@webkit.org>
+
+        [EFL][GTK] REGRESSION(204877): Fix the clean build
+        https://bugs.webkit.org/show_bug.cgi?id=161240
+
+        Reviewed by Carlos Garcia Campos.
+
+        * WebKitTestRunner/PlatformEfl.cmake:
+        * WebKitTestRunner/PlatformGTK.cmake:
+
+2016-08-26  Aakash Jain  <aakash_jain@apple.com>
+
+        EWS should indicate which bot processed the patch
+        https://bugs.webkit.org/show_bug.cgi?id=161222
+
+        Reviewed by Daniel Bates.
+
+        * QueueStatusServer/handlers/statusbubble.py:
+        (StatusBubble._build_bubble): Display the bot id in status bubble. Also removed "Started processing" message since
+        after bug fix of 161223, bots send the "Started processing patch" message and it would be displayed in Recent messages.
+        Also added "Started processing patch" message to progress_statuses.
+        I might consider displaying bot id in other statuses as well later on.
+
+2016-08-26  Gyuyoung Kim  <gyuyoung.kim@webkit.org>
+
+        [EFL] Change homepage url to www.ewebkit.org on MiniBrowser
+        https://bugs.webkit.org/show_bug.cgi?id=161184
+
+        Reviewed by Carlos Garcia Campos.
+
+        * MiniBrowser/efl/main.c: Set www.ewebkit.org.
+
+2016-08-25  Aakash Jain  <aakash_jain@apple.com>
+
+        ews queues should update status server when they start processing patch
+        https://bugs.webkit.org/show_bug.cgi?id=161223
+
+        Reviewed by Daniel Bates.
+
+        * Scripts/webkitpy/tool/commands/queues.py:
+        (AbstractReviewQueue.process_work_item): Update status server that the bot started processing the patch.
+        * Scripts/webkitpy/tool/commands/earlywarningsystem_unittest.py:
+        (EarlyWarningSystemTest._default_expected_logs): Updated the unit test accordingly.
+        * Scripts/webkitpy/tool/commands/queues_unittest.py:
+        (StyleQueueTest.test_style_queue_with_style_exception): Same.
+        (test_style_queue_with_watch_list_exception): Same.
+        (test_non_valid_patch): Same.
+
+2016-08-25  Alex Christensen  <achristensen@webkit.org>
+
+        CMake build fix.
+
+        * DumpRenderTree/PlatformMac.cmake:
+
+2016-08-25  Alex Christensen  <achristensen@webkit.org>
+
+        Fix URLParserTest with GuardMalloc
+        https://bugs.webkit.org/show_bug.cgi?id=161214
+        rdar://problem/28014390
+
+        Reviewed by Tim Horton.
+
+        The CString returned by String::utf8 was going out of scope before we were doing a string comparison on its data.
+        This just does the comparison of the CStrings instead to keep them in scope during the comparison.
+
+        * TestWebKitAPI/Tests/WebCore/URLParser.cpp:
+        (TestWebKitAPI::eq):
+        (TestWebKitAPI::checkURL):
+        (TestWebKitAPI::checkURLDifferences):
+        (TestWebKitAPI::s): Deleted.
+
+2016-08-25  Wenson Hsieh  <wenson_hsieh@apple.com>
+
+        Dragging against the end of the inline media scrubber causes the media scrubber to hide
+        https://bugs.webkit.org/show_bug.cgi?id=161207
+
+        Reviewed by Eric Carlson.
+
+        Tweaks an existing WebKit API test covering this behavior change. After some discussion, rather than hide media
+        controls in this case, we should actually continue showing them. This is because seeking due to user gestures
+        similar to "scrubbing" are indistinguishable from gestures that immediately seek to the end.
+
+        * TestWebKitAPI/Tests/WebKit2Cocoa/VideoControlsManager.mm:
+        (TestWebKitAPI::TEST):
+
+2016-08-25  Daniel Bates  <dabates@apple.com>
+
+        Watch more things.
+
+        * Scripts/webkitpy/common/config/watchlist:
+
+2016-08-25  Daniel Bates  <dabates@apple.com>
+
+        Fix test failure on GTK Linux bots after <http://trac.webkit.org/changeset/204695>
+        (https://bugs.webkit.org/show_bug.cgi?id=148437)
+
+        Remove use of module File::Slurp as it is not part of the standard Perl 5 distribution
+        and is not installed on the GTK Linux bots.
+
+        * Scripts/webkitperl/prepare-ChangeLog_unittest/generateFunctionLists.pl:
+        (writeFileWithContent): Added.
+
+2016-08-25  Per Arne Vollan  <pvollan@apple.com>
+
+        [Win] run-api-tests is failing.
+        https://bugs.webkit.org/show_bug.cgi?id=161140
+
+        Reviewed by Brent Fulgham.
+
+        Since the move constructor is called 3 more times on Windows, we will have 3 more destructor calls.
+
+        * TestWebKitAPI/Tests/WTF/CrossThreadTask.cpp:
+        (TestWebKitAPI::TEST):
+
+2016-08-24  Per Arne Vollan  <pvollan@apple.com>
+
+        [Win] run-api-tests is failing.
+        https://bugs.webkit.org/show_bug.cgi?id=161140
+
+        Reviewed by Brent Fulgham.
+
+        The number of times the move constructor is called is different on Windows in this test. This seems
+        to be caused by differences in MSVC's implementation of lambdas or std functions like std::make_tuple.
+
+        * TestWebKitAPI/Tests/WTF/CrossThreadTask.cpp:
+        (TestWebKitAPI::TEST):
+
+2016-08-24  Simon Fraser  <simon.fraser@apple.com>
+
+        Fix iOS DRT build.
+
+        * DumpRenderTree/ios/UIScriptControllerIOS.mm:
+        (WTR::UIScriptController::zoomToScale):
+
+2016-08-24  Simon Fraser  <simon.fraser@apple.com>
+
+        [iOS DRT] Hook up uiController.zoomToScale() for DRT
+        https://bugs.webkit.org/show_bug.cgi?id=161173
+
+        Reviewed by Tim Horton.
+        
+        Make a UIWebScrollView subclass, DumpRenderTreeWebScrollView, which has an internal delegate to get
+        UIScrollView delegate callbacks. Store the zoomToScaleCompletionHandler on it.
+        
+        UIScriptController::zoomToScale() has to bounce the zoom to the main thread (no poking UIKit from
+        the WebThread!), but queue up the callback first.
+
+        * DumpRenderTree/ios/DumpRenderTreeBrowserView.h:
+        * DumpRenderTree/ios/DumpRenderTreeBrowserView.mm:
+        (-[DumpRenderTreeWebScrollView initWithFrame:]):
+        (-[DumpRenderTreeWebScrollView dealloc]):
+        (-[DumpRenderTreeWebScrollView zoomToScale:animated:completionHandler:]):
+        (-[DumpRenderTreeWebScrollView completedZoomToScale]):
+        (-[DumpRenderTreeWebScrollViewDelegate initWithScrollView:]):
+        (-[DumpRenderTreeWebScrollViewDelegate viewForZoomingInScrollView:]): All UIScrollView delegates need to implement this.
+        (-[DumpRenderTreeWebScrollViewDelegate scrollViewDidEndZooming:withView:atScale:]):
+        * DumpRenderTree/ios/UIScriptControllerIOS.mm:
+        (WTR::UIScriptController::zoomToScale):
+        * DumpRenderTree/mac/DumpRenderTree.mm:
+        (createWebViewAndOffscreenWindow):
+        * TestRunnerShared/UIScriptContext/UIScriptController.h:
+        (WTR::UIScriptController::context):
+
+2016-08-24  Ryan Haddad  <ryanhaddad@apple.com>
+
+        Attempt to fix Windows build after r204918.
+
+        Unreviewed build fix.
+
+        * DumpRenderTree/win/ResourceLoadDelegate.cpp:
+        (ResourceLoadDelegate::willSendRequest):
+
+2016-08-24  Simon Fraser  <simon.fraser@apple.com>
+
+        [iOS DRT] Get basic viewport tests working in DRT
+        https://bugs.webkit.org/show_bug.cgi?id=161160
+
+        Reviewed by Tim Horton.
+        
+        Hook up enough of UIScriptController in iOS DRT to be able to read viewport
+        scale, and get the contentVisibleRect.
+        
+        Requires exposing a bit of UIWebBrowserView SPI from UIKit, which in turn
+        requires typing the gWebBrowserView as a DumpRenderTreeBrowserView * everywhere.
+        
+        One test has different results between DRT and WTR, but DRT behavior seems to differ
+        from UIWebView here, so just give it a WK1-specific result.
+
+        * DumpRenderTree/ios/DumpRenderTreeBrowserView.h:
+        * DumpRenderTree/ios/DumpRenderTreeBrowserView.mm:
+        (-[DumpRenderTreeBrowserView documentVisibleRect]):
+        * DumpRenderTree/ios/PixelDumpSupportIOS.mm:
+        * DumpRenderTree/ios/UIScriptControllerIOS.mm:
+        (WTR::UIScriptController::zoomScale):
+        (WTR::UIScriptController::minimumZoomScale):
+        (WTR::UIScriptController::maximumZoomScale):
+        (WTR::UIScriptController::contentVisibleRect):
+        * DumpRenderTree/mac/DumpRenderTree.mm:
+        (createWebViewAndOffscreenWindow):
+        * DumpRenderTree/mac/DumpRenderTreeMac.h:
+
+2016-08-24  Simon Fraser  <simon.fraser@apple.com>
+
+        Minor project rearrangement.
+
+        * DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj:
+
+2016-08-24  Simon Fraser  <simon.fraser@apple.com>
+
+        [iOS DRT] Make iOS DRT use the same window size as WebKitTestRunner for flexible viewport tests
+        https://bugs.webkit.org/show_bug.cgi?id=161124
+
+        Reviewed by Daniel Bates.
+        
+        Use the main UIScreen's bounds for the size of the window in flexible viewport mode,
+        and remove the 'phoneBrowserAddressBarOffset' hack, so that it matches WTR.
+        
+        Remove some unnecessary zero initializations of static variables.
+        
+        Change _setVisibleSize to use the window size, which should not be a behavior change
+        because this it just used for text autosizing, which is disabled by default,
+        and only the width is consulted.
+
+        * DumpRenderTree/mac/DumpRenderTree.mm:
+        (adjustWebDocumentForFlexibleViewport):
+        (adjustWebDocumentForStandardViewport):
+        (createWebViewAndOffscreenWindow):
+
+2016-08-24  Jonathan Bedard  <jbedard@apple.com>
+
+        WebKit2 needs layoutTestController.setDeferMainResourceDataLoad
+        https://bugs.webkit.org/show_bug.cgi?id=42777
+
+        Reviewed by Darin Adler.
+
+        The setDeferMainResourceDataLoad method was not correctly tested in DumpRenderTree and would pass even when setDeterMainResourceDataLoad was not called.  Removed function and test.
+
+        * DumpRenderTree/TestRunner.cpp:
+        (setDeferMainResourceDataLoadCallback): Removed setDeferMainResourceDataLoad.
+        * DumpRenderTree/TestRunner.h:
+        (TestRunner::deferMainResourceDataLoad): Ditto.
+        (TestRunner::setDeferMainResourceDataLoad): Ditto.
+        * DumpRenderTree/mac/ResourceLoadDelegate.mm: Ditto.
+
+2016-08-23  Anders Carlsson  <andersca@apple.com>
+
+        Add enum traits and use them in the IPC::Decoder
+        https://bugs.webkit.org/show_bug.cgi?id=161103
+
+        Reviewed by Sam Weinig.
+
+        Add API test.
+
+        * TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
+        * TestWebKitAPI/Tests/WTF/EnumTraits.cpp: Added.
+        (TestWebKitAPI::TEST):
+
 2016-08-24  Filip Pizlo  <fpizlo@apple.com>
 
         Unreviewed, roll out r204901, r204897, r204866, r204856, r204854.