UserMedia test should use WKPageNavigationClient
authormcatanzaro@igalia.com <mcatanzaro@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 3 Jan 2019 18:20:57 +0000 (18:20 +0000)
committermcatanzaro@igalia.com <mcatanzaro@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 3 Jan 2019 18:20:57 +0000 (18:20 +0000)
https://bugs.webkit.org/show_bug.cgi?id=192913

Reviewed by Youenn Fablet.

The test currently uses the deprecated WKPageLoaderClient. It should use
WKPageNavigationClient instead, which provides the same callback under a similar name.

* TestWebKitAPI/Tests/WebKit/UserMedia.cpp:
(TestWebKitAPI::TEST):

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

Tools/ChangeLog
Tools/TestWebKitAPI/Tests/WebKit/UserMedia.cpp

index 490a0ea..3ea765f 100644 (file)
@@ -1,3 +1,16 @@
+2019-01-03  Michael Catanzaro  <mcatanzaro@igalia.com>
+
+        UserMedia test should use WKPageNavigationClient
+        https://bugs.webkit.org/show_bug.cgi?id=192913
+
+        Reviewed by Youenn Fablet.
+
+        The test currently uses the deprecated WKPageLoaderClient. It should use
+        WKPageNavigationClient instead, which provides the same callback under a similar name.
+
+        * TestWebKitAPI/Tests/WebKit/UserMedia.cpp:
+        (TestWebKitAPI::TEST):
+
 2019-01-02  Alex Christensen  <achristensen@webkit.org>
 
         Homograph with LATIN SMALL LETTER R WITH FISHHOOK
index 2f0cbe2..91c2786 100644 (file)
@@ -132,11 +132,11 @@ TEST(WebKit, OnDeviceChangeCrash)
     // Load a second page in same process.
     PlatformWebView webView2(context.get(), pageGroup.get());
     WKPageSetPageUIClient(webView2.page(), &uiClient.base);
-    WKPageLoaderClientV0 loaderClient;
-    memset(&loaderClient, 0, sizeof(loaderClient));
-    loaderClient.base.version = 0;
-    loaderClient.processDidCrash = didCrashCallback;
-    WKPageSetPageLoaderClient(webView2.page(), &loaderClient.base);
+    WKPageNavigationClientV0 navigationClient;
+    memset(&navigationClient, 0, sizeof(navigationClient));
+    navigationClient.base.version = 0;
+    navigationClient.webProcessDidCrash = didCrashCallback;
+    WKPageSetPageNavigationClient(webView2.page(), &navigationClient.base);
 
     wasPrompted = false;
     url = adoptWK(Util::createURLForResource("getUserMedia", "html"));