Bug 190153 - Convert tests to use navigation client
Summary: Convert tests to use navigation client
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: Other
Hardware: PC Linux
: P2 Normal
Assignee: Michael Catanzaro
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-10-01 12:58 PDT by Michael Catanzaro
Modified: 2018-10-04 09:56 PDT (History)
5 users (show)

See Also:


Attachments
Patch (4.40 KB, patch)
2018-10-03 06:53 PDT, Michael Catanzaro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2018-10-01 12:58:05 PDT
[4613/4761] Building CXX object Tools/TestWebKitAPI...r/Tests/WebKit/DownloadDecideDestinationCrash.cpp.o
../../Tools/TestWebKitAPI/Tests/WebKit/DownloadDecideDestinationCrash.cpp: In function ‘void TestWebKitAPI::setPagePolicyClient(WKPageRef)’:
../../Tools/TestWebKitAPI/Tests/WebKit/DownloadDecideDestinationCrash.cpp:69:55: warning: ‘void WKPageSetPagePolicyClient(WKPageRef, const WKPagePolicyClientBase*)’ is deprecated: use WKPageSetPageNavigationClient [-Wdeprecated-declarations]
     WKPageSetPagePolicyClient(page, &policyClient.base);
                                                       ^
In file included from DerivedSources/ForwardingHeaders/WebKit/WKPage.h:1,
                 from ../../Source/WebKit/UIProcess/API/C/WebKit2_C.h:53,
                 from DerivedSources/ForwardingHeaders/WebKit/WebKit2_C.h:1,
                 from ../../Tools/TestWebKitAPI/config.h:49,
                 from ../../Tools/TestWebKitAPI/Tests/WebKit/DownloadDecideDestinationCrash.cpp:26:
../../Source/WebKit/UIProcess/API/C/WKPage.h:229:16: note: declared here
 WK_EXPORT void WKPageSetPagePolicyClient(WKPageRef page, const WKPagePolicyClientBase* client) WK_C_API_DEPRECATED_WITH_REPLACEMENT(WKPageSetPageNavigationClient, macosx(WK_MAC_TBA));
                ^~~~~~~~~~~~~~~~~~~~~~~~~
../../Tools/TestWebKitAPI/Tests/WebKit/DownloadDecideDestinationCrash.cpp:69:55: warning: ‘void WKPageSetPagePolicyClient(WKPageRef, const WKPagePolicyClientBase*)’ is deprecated: use WKPageSetPageNavigationClient [-Wdeprecated-declarations]
     WKPageSetPagePolicyClient(page, &policyClient.base);
                                                       ^
In file included from DerivedSources/ForwardingHeaders/WebKit/WKPage.h:1,
                 from ../../Source/WebKit/UIProcess/API/C/WebKit2_C.h:53,
                 from DerivedSources/ForwardingHeaders/WebKit/WebKit2_C.h:1,
                 from ../../Tools/TestWebKitAPI/config.h:49,
                 from ../../Tools/TestWebKitAPI/Tests/WebKit/DownloadDecideDestinationCrash.cpp:26:
../../Source/WebKit/UIProcess/API/C/WKPage.h:229:16: note: declared here
 WK_EXPORT void WKPageSetPagePolicyClient(WKPageRef page, const WKPagePolicyClientBase* client) WK_C_API_DEPRECATED_WITH_REPLACEMENT(WKPageSetPageNavigationClient, macosx(WK_MAC_TBA));
                ^~~~~~~~~~~~~~~~~~~~~~~~~
[4649/4761] Building CXX object Tools/TestWebKitAPI...it/ProvisionalURLAfterWillSendRequestCallback.cpp.o
../../Tools/TestWebKitAPI/Tests/WebKit/ProvisionalURLAfterWillSendRequestCallback.cpp: In member function ‘virtual void TestWebKitAPI::WebKit2_ProvisionalURLAfterWillSendRequestCallback_Test::TestBody()’:
../../Tools/TestWebKitAPI/Tests/WebKit/ProvisionalURLAfterWillSendRequestCallback.cpp:80:65: warning: ‘void WKPageSetPageLoaderClient(WKPageRef, const WKPageLoaderClientBase*)’ is deprecated: use WKPageSetPageNavigationClient [-Wdeprecated-declarations]
     WKPageSetPageLoaderClient(webView.page(), &loaderClient.base);
                                                                 ^
In file included from DerivedSources/ForwardingHeaders/WebKit/WKPage.h:1,
                 from ../../Source/WebKit/UIProcess/API/C/WebKit2_C.h:53,
                 from DerivedSources/ForwardingHeaders/WebKit/WebKit2_C.h:1,
                 from ../../Tools/TestWebKitAPI/config.h:49,
                 from ../../Tools/TestWebKitAPI/Tests/WebKit/ProvisionalURLAfterWillSendRequestCallback.cpp:26:
../../Source/WebKit/UIProcess/API/C/WKPage.h:228:16: note: declared here
 WK_EXPORT void WKPageSetPageLoaderClient(WKPageRef page, const WKPageLoaderClientBase* client) WK_C_API_DEPRECATED_WITH_REPLACEMENT(WKPageSetPageNavigationClient, macosx(WK_MAC_TBA));
                ^~~~~~~~~~~~~~~~~~~~~~~~~
../../Tools/TestWebKitAPI/Tests/WebKit/ProvisionalURLAfterWillSendRequestCallback.cpp:80:65: warning: ‘void WKPageSetPageLoaderClient(WKPageRef, const WKPageLoaderClientBase*)’ is deprecated: use WKPageSetPageNavigationClient [-Wdeprecated-declarations]
     WKPageSetPageLoaderClient(webView.page(), &loaderClient.base);
                                                                 ^
In file included from DerivedSources/ForwardingHeaders/WebKit/WKPage.h:1,
                 from ../../Source/WebKit/UIProcess/API/C/WebKit2_C.h:53,
                 from DerivedSources/ForwardingHeaders/WebKit/WebKit2_C.h:1,
                 from ../../Tools/TestWebKitAPI/config.h:49,
                 from ../../Tools/TestWebKitAPI/Tests/WebKit/ProvisionalURLAfterWillSendRequestCallback.cpp:26:
../../Source/WebKit/UIProcess/API/C/WKPage.h:228:16: note: declared here
 WK_EXPORT void WKPageSetPageLoaderClient(WKPageRef page, const WKPageLoaderClientBase* client) WK_C_API_DEPRECATED_WITH_REPLACEMENT(WKPageSetPageNavigationClient, macosx(WK_MAC_TBA));
                ^~~~~~~~~~~~~~~~~~~~~~~~~
Comment 1 Michael Catanzaro 2018-10-02 09:57:33 PDT
These are all in cross-platform tests.

Alex, I guess we should update the tests rather than silencing the warnings.
Comment 2 Alex Christensen 2018-10-02 10:03:25 PDT
Definitely
Comment 3 Michael Catanzaro 2018-10-03 06:53:48 PDT
Created attachment 351515 [details]
Patch
Comment 4 WebKit Commit Bot 2018-10-04 09:55:56 PDT
Comment on attachment 351515 [details]
Patch

Clearing flags on attachment: 351515

Committed r236833: <https://trac.webkit.org/changeset/236833>
Comment 5 WebKit Commit Bot 2018-10-04 09:55:57 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Radar WebKit Bug Importer 2018-10-04 09:56:46 PDT
<rdar://problem/45011949>