Title: [230076] trunk/Source/WebKit
- Revision
- 230076
- Author
- bfulg...@apple.com
- Date
- 2018-03-29 09:28:03 -0700 (Thu, 29 Mar 2018)
Log Message
REGRESSION(r229480): ERROR: Unhandled web process message 'WebCookieManager:SetHTTPCookieAcceptPolicy'
https://bugs.webkit.org/show_bug.cgi?id=184124
<rdar://problem/38998971>
Reviewed by Chris Dumez.
Cookie accept policy messages were still being sent to the WebContent process after
I removed cookie access in r229480. The WebContent process no longer recognizes these
messages, and generates logging to that effect.
This patch stops sending these unnecessary messages to the WebContent process. Only the
Network process needs to receive this information.
* UIProcess/WebCookieManagerProxy.cpp:
(WebKit::WebCookieManagerProxy::setHTTPCookieAcceptPolicy):
Modified Paths
Diff
Modified: trunk/Source/WebKit/ChangeLog (230075 => 230076)
--- trunk/Source/WebKit/ChangeLog 2018-03-29 15:38:54 UTC (rev 230075)
+++ trunk/Source/WebKit/ChangeLog 2018-03-29 16:28:03 UTC (rev 230076)
@@ -1,3 +1,21 @@
+2018-03-29 Brent Fulgham <bfulg...@apple.com>
+
+ REGRESSION(r229480): ERROR: Unhandled web process message 'WebCookieManager:SetHTTPCookieAcceptPolicy'
+ https://bugs.webkit.org/show_bug.cgi?id=184124
+ <rdar://problem/38998971>
+
+ Reviewed by Chris Dumez.
+
+ Cookie accept policy messages were still being sent to the WebContent process after
+ I removed cookie access in r229480. The WebContent process no longer recognizes these
+ messages, and generates logging to that effect.
+
+ This patch stops sending these unnecessary messages to the WebContent process. Only the
+ Network process needs to receive this information.
+
+ * UIProcess/WebCookieManagerProxy.cpp:
+ (WebKit::WebCookieManagerProxy::setHTTPCookieAcceptPolicy):
+
2018-03-29 Carlos Eduardo Ramalho <cadubent...@gmail.com>
[WPE] Floating point exception in WebEventFactory::createWebWheelEvent
Modified: trunk/Source/WebKit/UIProcess/WebCookieManagerProxy.cpp (230075 => 230076)
--- trunk/Source/WebKit/UIProcess/WebCookieManagerProxy.cpp 2018-03-29 15:38:54 UTC (rev 230075)
+++ trunk/Source/WebKit/UIProcess/WebCookieManagerProxy.cpp 2018-03-29 16:28:03 UTC (rev 230076)
@@ -252,11 +252,6 @@
processPool()->setInitialHTTPCookieAcceptPolicy(policy);
#endif
- // The policy is not sent to newly created processes (only Soup does that via setInitialHTTPCookieAcceptPolicy()). This is not a serious problem, because:
- // - When testing, we only have one WebProcess and one NetworkProcess, and WebKitTestRunner never restarts them;
- // - When not testing, Cocoa has the policy persisted, and thus new processes use it (even for ephemeral sessions).
- processPool()->sendToAllProcesses(Messages::WebCookieManager::SetHTTPCookieAcceptPolicy(policy, OptionalCallbackID()));
-
auto callbackID = m_callbacks.put(WTFMove(callbackFunction));
processPool()->sendToNetworkingProcess(Messages::WebCookieManager::SetHTTPCookieAcceptPolicy(policy, OptionalCallbackID(callbackID)));
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes