Title: [257538] trunk/Source/WebKit
Revision
257538
Author
[email protected]
Date
2020-02-26 17:15:35 -0800 (Wed, 26 Feb 2020)

Log Message

Remove WebsiteDataStore setters for allowsCellularAccess, proxyConfiguration, and boundInterfaceIdentifier
https://bugs.webkit.org/show_bug.cgi?id=206390

Reviewed by Chris Dumez.

The WKWebsiteDataStore SPI was deprecated with replacement on _WKWebsiteDataStoreConfiguration,
and the replacements were adopted in rdar://problem/58359766

* UIProcess/API/Cocoa/WKWebsiteDataStore.mm:
(-[WKWebsiteDataStore _setBoundInterfaceIdentifier:]):
(-[WKWebsiteDataStore _boundInterfaceIdentifier]):
(-[WKWebsiteDataStore _setAllowsCellularAccess:]):
(-[WKWebsiteDataStore _allowsCellularAccess]):
(-[WKWebsiteDataStore _setProxyConfiguration:]):
(-[WKWebsiteDataStore _proxyConfiguration]):
* UIProcess/Cocoa/WebPageProxyCocoa.mm:
(WebKit::WebPageProxy::paymentCoordinatorBoundInterfaceIdentifier):
* UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm:
(WebKit::WebsiteDataStore::parameters):
* UIProcess/WebsiteData/WebsiteDataStore.h:
(WebKit::WebsiteDataStore::networkingHasBegun):
(WebKit::WebsiteDataStore::setBoundInterfaceIdentifier): Deleted.
(WebKit::WebsiteDataStore::boundInterfaceIdentifier): Deleted.
(WebKit::WebsiteDataStore::setAllowsCellularAccess): Deleted.
(WebKit::WebsiteDataStore::allowsCellularAccess): Deleted.
(WebKit::WebsiteDataStore::setProxyConfiguration): Deleted.
(WebKit::WebsiteDataStore::proxyConfiguration): Deleted.

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (257537 => 257538)


--- trunk/Source/WebKit/ChangeLog	2020-02-27 01:08:46 UTC (rev 257537)
+++ trunk/Source/WebKit/ChangeLog	2020-02-27 01:15:35 UTC (rev 257538)
@@ -1,3 +1,33 @@
+2020-02-26  Alex Christensen  <[email protected]>
+
+        Remove WebsiteDataStore setters for allowsCellularAccess, proxyConfiguration, and boundInterfaceIdentifier
+        https://bugs.webkit.org/show_bug.cgi?id=206390
+
+        Reviewed by Chris Dumez.
+
+        The WKWebsiteDataStore SPI was deprecated with replacement on _WKWebsiteDataStoreConfiguration,
+        and the replacements were adopted in rdar://problem/58359766
+
+        * UIProcess/API/Cocoa/WKWebsiteDataStore.mm:
+        (-[WKWebsiteDataStore _setBoundInterfaceIdentifier:]):
+        (-[WKWebsiteDataStore _boundInterfaceIdentifier]):
+        (-[WKWebsiteDataStore _setAllowsCellularAccess:]):
+        (-[WKWebsiteDataStore _allowsCellularAccess]):
+        (-[WKWebsiteDataStore _setProxyConfiguration:]):
+        (-[WKWebsiteDataStore _proxyConfiguration]):
+        * UIProcess/Cocoa/WebPageProxyCocoa.mm:
+        (WebKit::WebPageProxy::paymentCoordinatorBoundInterfaceIdentifier):
+        * UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm:
+        (WebKit::WebsiteDataStore::parameters):
+        * UIProcess/WebsiteData/WebsiteDataStore.h:
+        (WebKit::WebsiteDataStore::networkingHasBegun):
+        (WebKit::WebsiteDataStore::setBoundInterfaceIdentifier): Deleted.
+        (WebKit::WebsiteDataStore::boundInterfaceIdentifier): Deleted.
+        (WebKit::WebsiteDataStore::setAllowsCellularAccess): Deleted.
+        (WebKit::WebsiteDataStore::allowsCellularAccess): Deleted.
+        (WebKit::WebsiteDataStore::setProxyConfiguration): Deleted.
+        (WebKit::WebsiteDataStore::proxyConfiguration): Deleted.
+
 2020-02-26  Fujii Hironori  <[email protected]>
 
         Unreviewed unified source builds fix.

Modified: trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebsiteDataStore.mm (257537 => 257538)


--- trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebsiteDataStore.mm	2020-02-27 01:08:46 UTC (rev 257537)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebsiteDataStore.mm	2020-02-27 01:15:35 UTC (rev 257538)
@@ -321,27 +321,24 @@
 
 - (void)_setBoundInterfaceIdentifier:(NSString *)identifier
 {
-    _websiteDataStore->setBoundInterfaceIdentifier(identifier);
 }
 
 - (NSString *)_boundInterfaceIdentifier
 {
-    return _websiteDataStore->boundInterfaceIdentifier();
+    return nil;
 }
 
 - (void)_setAllowsCellularAccess:(BOOL)allows
 {
-    _websiteDataStore->setAllowsCellularAccess(allows ? WebKit::AllowsCellularAccess::Yes : WebKit::AllowsCellularAccess::No);
 }
 
 - (BOOL)_allowsCellularAccess
 {
-    return _websiteDataStore->allowsCellularAccess() == WebKit::AllowsCellularAccess::Yes;
+    return YES;
 }
 
 - (void)_setProxyConfiguration:(NSDictionary *)configuration
 {
-    _websiteDataStore->setProxyConfiguration((__bridge CFDictionaryRef)configuration);
 }
 
 - (NSString *)_sourceApplicationBundleIdentifier
@@ -377,7 +374,7 @@
 
 - (NSDictionary *)_proxyConfiguration
 {
-    return (__bridge NSDictionary *)_websiteDataStore->proxyConfiguration();
+    return nil;
 }
 
 - (NSURL *)_indexedDBDatabaseDirectory

Modified: trunk/Source/WebKit/UIProcess/Cocoa/WebPageProxyCocoa.mm (257537 => 257538)


--- trunk/Source/WebKit/UIProcess/Cocoa/WebPageProxyCocoa.mm	2020-02-27 01:08:46 UTC (rev 257537)
+++ trunk/Source/WebKit/UIProcess/Cocoa/WebPageProxyCocoa.mm	2020-02-27 01:15:35 UTC (rev 257538)
@@ -235,7 +235,7 @@
 
 const String& WebPageProxy::paymentCoordinatorBoundInterfaceIdentifier(const WebPaymentCoordinatorProxy&)
 {
-    return websiteDataStore().boundInterfaceIdentifier();
+    return websiteDataStore().configuration().boundInterfaceIdentifier();
 }
 
 const String& WebPageProxy::paymentCoordinatorSourceApplicationBundleIdentifier(const WebPaymentCoordinatorProxy&)

Modified: trunk/Source/WebKit/UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm (257537 => 257538)


--- trunk/Source/WebKit/UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm	2020-02-27 01:08:46 UTC (rev 257537)
+++ trunk/Source/WebKit/UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm	2020-02-27 01:15:35 UTC (rev 257538)
@@ -150,9 +150,9 @@
     WebsiteDataStoreParameters parameters;
     parameters.networkSessionParameters = {
         m_sessionID,
-        boundInterfaceIdentifier(),
-        allowsCellularAccess(),
-        proxyConfiguration(),
+        configuration().boundInterfaceIdentifier(),
+        configuration().allowsCellularAccess() ? AllowsCellularAccess::Yes : AllowsCellularAccess::No,
+        configuration().proxyConfiguration(),
         sourceApplicationBundleIdentifier(),
         sourceApplicationSecondaryIdentifier(),
         shouldLogCookieInformation,

Modified: trunk/Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.h (257537 => 257538)


--- trunk/Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.h	2020-02-27 01:08:46 UTC (rev 257537)
+++ trunk/Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.h	2020-02-27 01:15:35 UTC (rev 257538)
@@ -224,9 +224,6 @@
     void removePendingCookie(const WebCore::Cookie&);
     void clearPendingCookies();
 
-    void setBoundInterfaceIdentifier(String&& identifier) { m_resolvedConfiguration->setBoundInterfaceIdentifier(WTFMove(identifier)); }
-    const String& boundInterfaceIdentifier() { return m_resolvedConfiguration->boundInterfaceIdentifier(); }
-
     const String& sourceApplicationBundleIdentifier() const { return m_resolvedConfiguration->sourceApplicationBundleIdentifier(); }
     bool setSourceApplicationBundleIdentifier(String&&);
 
@@ -234,15 +231,7 @@
     bool setSourceApplicationSecondaryIdentifier(String&&);
 
     void networkingHasBegun() { m_networkingHasBegun = true; }
-    
-    void setAllowsCellularAccess(AllowsCellularAccess allows) { m_resolvedConfiguration->setAllowsCellularAccess(allows == AllowsCellularAccess::Yes); }
-    AllowsCellularAccess allowsCellularAccess() { return m_resolvedConfiguration->allowsCellularAccess() ? AllowsCellularAccess::Yes : AllowsCellularAccess::No; }
 
-#if PLATFORM(COCOA)
-    void setProxyConfiguration(CFDictionaryRef configuration) { m_resolvedConfiguration->setProxyConfiguration(configuration); }
-    CFDictionaryRef proxyConfiguration() { return m_resolvedConfiguration->proxyConfiguration(); }
-#endif
-
 #if USE(CURL)
     void setNetworkProxySettings(WebCore::CurlProxySettings&&);
     const WebCore::CurlProxySettings& networkProxySettings() const { return m_proxySettings; }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to