Unreviewed, a quick build fix for r232276.
authorjiewen_tan@apple.com <jiewen_tan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 30 May 2018 19:23:57 +0000 (19:23 +0000)
committerjiewen_tan@apple.com <jiewen_tan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 30 May 2018 19:23:57 +0000 (19:23 +0000)
Enabled SecItemShim again as it turns out to be useful for CFNetwork APIs that
query Keychains underneath us.

* NetworkProcess/ios/NetworkProcessIOS.mm:
(WebKit::NetworkProcess::platformInitializeNetworkProcess):
* NetworkProcess/mac/NetworkProcessMac.mm:
(WebKit::NetworkProcess::platformInitializeNetworkProcess):

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

Source/WebKit/ChangeLog
Source/WebKit/NetworkProcess/ios/NetworkProcessIOS.mm
Source/WebKit/NetworkProcess/mac/NetworkProcessMac.mm

index 5abe2d2..c748e88 100644 (file)
@@ -1,3 +1,15 @@
+2018-05-30  Jiewen Tan  <jiewen_tan@apple.com>
+
+        Unreviewed, a quick build fix for r232276.
+
+        Enabled SecItemShim again as it turns out to be useful for CFNetwork APIs that
+        query Keychains underneath us.
+
+        * NetworkProcess/ios/NetworkProcessIOS.mm:
+        (WebKit::NetworkProcess::platformInitializeNetworkProcess):
+        * NetworkProcess/mac/NetworkProcessMac.mm:
+        (WebKit::NetworkProcess::platformInitializeNetworkProcess):
+
 2018-05-30  Jer Noble  <jer.noble@apple.com>
 
         Auto-pip should use main content heuristic.
index 2243721..70f7ca5 100644 (file)
@@ -84,7 +84,8 @@ void NetworkProcess::clearCacheForAllOrigins(uint32_t cachesToClear)
 
 void NetworkProcess::platformInitializeNetworkProcess(const NetworkProcessCreationParameters& parameters)
 {
-#if ENABLE(SEC_ITEM_SHIM) && !HAVE(SEC_KEY_PROXY)
+#if ENABLE(SEC_ITEM_SHIM)
+    // SecItemShim is needed for CFNetwork APIs that query Keychains beneath us.
     initializeSecItemShim(*this);
 #endif
     platformInitializeNetworkProcessCocoa(parameters);
index 8e83346..5fd40da 100644 (file)
@@ -104,7 +104,8 @@ void NetworkProcess::platformInitializeNetworkProcess(const NetworkProcessCreati
 {
     platformInitializeNetworkProcessCocoa(parameters);
 
-#if ENABLE(SEC_ITEM_SHIM) && !HAVE(SEC_KEY_PROXY)
+#if ENABLE(SEC_ITEM_SHIM)
+    // SecItemShim is needed for CFNetwork APIs that query Keychains beneath us.
     initializeSecItemShim(*this);
 #endif