Diff
Modified: trunk/Source/WebCore/ChangeLog (239726 => 239727)
--- trunk/Source/WebCore/ChangeLog 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebCore/ChangeLog 2019-01-08 18:08:55 UTC (rev 239727)
@@ -1,3 +1,25 @@
+2019-01-08 Alex Christensen <achristen...@webkit.org>
+
+ Stop using NetworkStorageSession in WebProcess
+ https://bugs.webkit.org/show_bug.cgi?id=193236
+
+ Reviewed by Don Olmstead.
+
+ No change in behavior. Some code was only used for ResourceHandle, which isn't used in modern WebKit,
+ and for cookies, which are handled in the NetworkProcess in modern WebKit.
+
+ * loader/CookieJar.cpp:
+ (WebCore::storageSession):
+ * loader/EmptyClients.cpp:
+ * platform/network/NetworkingContext.h:
+ * platform/network/mac/ResourceHandleMac.mm:
+ (WebCore::ResourceHandle::createNSURLConnection):
+ (WebCore::ResourceHandle::start):
+ (WebCore::ResourceHandle::platformLoadResourceSynchronously):
+ (WebCore::ResourceHandle::willSendRequest):
+ (WebCore::ResourceHandle::tryHandlePasswordBasedAuthentication):
+ (WebCore::ResourceHandle::receivedCredential):
+
2019-01-08 Chris Dumez <cdu...@apple.com>
Regression(PSON-r239182): Blank view when navigating back and forth between google.com and stack overflow
Modified: trunk/Source/WebCore/loader/CookieJar.cpp (239726 => 239727)
--- trunk/Source/WebCore/loader/CookieJar.cpp 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebCore/loader/CookieJar.cpp 2019-01-08 18:08:55 UTC (rev 239727)
@@ -54,7 +54,7 @@
inline NetworkStorageSession& storageSession(const Document& document)
{
NetworkingContext* context = networkingContext(document);
- return context ? context->storageSession() : NetworkStorageSession::defaultStorageSession();
+ return context ? *context->storageSession() : NetworkStorageSession::defaultStorageSession();
}
static IncludeSecureCookies shouldIncludeSecureCookies(const Document& document, const URL& url)
Modified: trunk/Source/WebCore/loader/EmptyClients.cpp (239726 => 239727)
--- trunk/Source/WebCore/loader/EmptyClients.cpp 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebCore/loader/EmptyClients.cpp 2019-01-08 18:08:55 UTC (rev 239727)
@@ -287,7 +287,7 @@
EmptyFrameNetworkingContext();
bool shouldClearReferrerOnHTTPSToHTTPRedirect() const { return true; }
- NetworkStorageSession& storageSession() const final { return NetworkStorageSession::defaultStorageSession(); }
+ NetworkStorageSession* storageSession() const final { return &NetworkStorageSession::defaultStorageSession(); }
#if PLATFORM(COCOA)
bool localFileContentSniffingEnabled() const { return false; }
Modified: trunk/Source/WebCore/platform/network/NetworkingContext.h (239726 => 239727)
--- trunk/Source/WebCore/platform/network/NetworkingContext.h 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebCore/platform/network/NetworkingContext.h 2019-01-08 18:08:55 UTC (rev 239727)
@@ -58,7 +58,7 @@
virtual String sourceApplicationIdentifier() const { return emptyString(); }
- virtual NetworkStorageSession& storageSession() const = 0;
+ virtual NetworkStorageSession* storageSession() const = 0;
#if PLATFORM(WIN)
virtual ResourceError blockedError(const ResourceRequest&) const = 0;
Modified: trunk/Source/WebCore/platform/network/cf/ResourceHandleCFNet.cpp (239726 => 239727)
--- trunk/Source/WebCore/platform/network/cf/ResourceHandleCFNet.cpp 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebCore/platform/network/cf/ResourceHandleCFNet.cpp 2019-01-08 18:08:55 UTC (rev 239727)
@@ -149,12 +149,12 @@
if (d->m_user.isEmpty() && d->m_pass.isEmpty()) {
// <rdar://problem/7174050> - For URLs that match the paths of those previously challenged for HTTP Basic authentication,
// try and reuse the credential preemptively, as allowed by RFC 2617.
- d->m_initialCredential = d->m_context->storageSession().credentialStorage().get(partition, firstRequest().url());
+ d->m_initialCredential = d->m_context->storageSession()->credentialStorage().get(partition, firstRequest().url());
} else {
// If there is already a protection space known for the URL, update stored credentials before sending a request.
// This makes it possible to implement logout by sending an XMLHttpRequest with known incorrect credentials, and aborting it immediately
// (so that an authentication dialog doesn't pop up).
- d->m_context->storageSession().credentialStorage().set(partition, Credential(d->m_user, d->m_pass, CredentialPersistenceNone), firstRequest().url());
+ d->m_context->storageSession()->credentialStorage().set(partition, Credential(d->m_user, d->m_pass, CredentialPersistenceNone), firstRequest().url());
}
}
@@ -261,7 +261,7 @@
if (!d->m_context->isValid())
return false;
- d->m_storageSession = d->m_context->storageSession().platformSession();
+ d->m_storageSession = d->m_context->storageSession()->platformSession();
bool shouldUseCredentialStorage = !client() || client()->shouldUseCredentialStorage(this);
@@ -307,7 +307,7 @@
// Only consider applying authentication credentials if this is actually a redirect and the redirect
// URL didn't include credentials of its own.
if (d->m_user.isEmpty() && d->m_pass.isEmpty() && !redirectResponse.isNull()) {
- Credential credential = d->m_context->storageSession().credentialStorage().get(partition, request.url());
+ Credential credential = d->m_context->storageSession()->credentialStorage().get(partition, request.url());
if (!credential.isEmpty()) {
d->m_initialCredential = credential;
@@ -382,7 +382,7 @@
URL urlToStore;
if (challenge.failureResponse().httpStatusCode() == 401)
urlToStore = challenge.failureResponse().url();
- d->m_context->storageSession().credentialStorage().set(partition, credential, challenge.protectionSpace(), urlToStore);
+ d->m_context->storageSession()->credentialStorage().set(partition, credential, challenge.protectionSpace(), urlToStore);
CFURLConnectionUseCredential(d->m_connection.get(), cfCredential.get(), challenge.cfURLAuthChallengeRef());
d->m_user = String();
@@ -396,16 +396,16 @@
// The stored credential wasn't accepted, stop using it.
// There is a race condition here, since a different credential might have already been stored by another ResourceHandle,
// but the observable effect should be very minor, if any.
- d->m_context->storageSession().credentialStorage().remove(partition, challenge.protectionSpace());
+ d->m_context->storageSession()->credentialStorage().remove(partition, challenge.protectionSpace());
}
if (!challenge.previousFailureCount()) {
- Credential credential = d->m_context->storageSession().credentialStorage().get(partition, challenge.protectionSpace());
+ Credential credential = d->m_context->storageSession()->credentialStorage().get(partition, challenge.protectionSpace());
if (!credential.isEmpty() && credential != d->m_initialCredential) {
ASSERT(credential.persistence() == CredentialPersistenceNone);
if (challenge.failureResponse().httpStatusCode() == 401) {
// Store the credential back, possibly adding it as a default for this directory.
- d->m_context->storageSession().credentialStorage().set(partition, credential, challenge.protectionSpace(), challenge.failureResponse().url());
+ d->m_context->storageSession()->credentialStorage().set(partition, credential, challenge.protectionSpace(), challenge.failureResponse().url());
}
#if PLATFORM(COCOA)
CFURLConnectionUseCredential(d->m_connection.get(), credential.cfCredential(), challenge.cfURLAuthChallengeRef());
@@ -455,7 +455,7 @@
if (challenge.failureResponse().httpStatusCode() == 401)
urlToStore = challenge.failureResponse().url();
- d->m_context->storageSession().credentialStorage().set(firstRequest().cachePartition(), webCredential, challenge.protectionSpace(), urlToStore);
+ d->m_context->storageSession()->credentialStorage().set(firstRequest().cachePartition(), webCredential, challenge.protectionSpace(), urlToStore);
if (d->m_connection) {
#if PLATFORM(COCOA)
@@ -571,7 +571,7 @@
bool shouldContentEncodingSniff = true;
RefPtr<ResourceHandle> handle = adoptRef(new ResourceHandle(context, request, &client, defersLoading, shouldContentSniff, shouldContentEncodingSniff));
- handle->d->m_storageSession = context->storageSession().platformSession();
+ handle->d->m_storageSession = context->storageSession()->platformSession();
if (handle->d->m_scheduledFailureType != NoFailure) {
error = context->blockedError(request);
Modified: trunk/Source/WebCore/platform/network/mac/ResourceHandleMac.mm (239726 => 239727)
--- trunk/Source/WebCore/platform/network/mac/ResourceHandleMac.mm 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebCore/platform/network/mac/ResourceHandleMac.mm 2019-01-08 18:08:55 UTC (rev 239727)
@@ -157,12 +157,12 @@
if (d->m_user.isEmpty() && d->m_pass.isEmpty()) {
// <rdar://problem/7174050> - For URLs that match the paths of those previously challenged for HTTP Basic authentication,
// try and reuse the credential preemptively, as allowed by RFC 2617.
- d->m_initialCredential = d->m_context->storageSession().credentialStorage().get(firstRequest().cachePartition(), firstRequest().url());
+ d->m_initialCredential = d->m_context->storageSession()->credentialStorage().get(firstRequest().cachePartition(), firstRequest().url());
} else {
// If there is already a protection space known for the URL, update stored credentials before sending a request.
// This makes it possible to implement logout by sending an XMLHttpRequest with known incorrect credentials, and aborting it immediately
// (so that an authentication dialog doesn't pop up).
- d->m_context->storageSession().credentialStorage().set(firstRequest().cachePartition(), Credential(d->m_user, d->m_pass, CredentialPersistenceNone), firstRequest().url());
+ d->m_context->storageSession()->credentialStorage().set(firstRequest().cachePartition(), Credential(d->m_user, d->m_pass, CredentialPersistenceNone), firstRequest().url());
}
}
@@ -240,7 +240,7 @@
if (!d->m_context->isValid())
return false;
- d->m_storageSession = d->m_context->storageSession().platformSession();
+ d->m_storageSession = d->m_context->storageSession()->platformSession();
// FIXME: Do not use the sync version of shouldUseCredentialStorage when the client returns true from usesAsyncCallbacks.
bool shouldUseCredentialStorage = !client() || client()->shouldUseCredentialStorage(this);
@@ -367,7 +367,7 @@
bool shouldContentEncodingSniff = true;
RefPtr<ResourceHandle> handle = adoptRef(new ResourceHandle(context, request, &client, defersLoading, shouldContentSniff, shouldContentEncodingSniff));
- handle->d->m_storageSession = context->storageSession().platformSession();
+ handle->d->m_storageSession = context->storageSession()->platformSession();
if (context && handle->d->m_scheduledFailureType != NoFailure) {
error = context->blockedError(request);
@@ -449,7 +449,7 @@
// Only consider applying authentication credentials if this is actually a redirect and the redirect
// URL didn't include credentials of its own.
if (d->m_user.isEmpty() && d->m_pass.isEmpty() && !redirectResponse.isNull()) {
- Credential credential = d->m_context->storageSession().credentialStorage().get(request.cachePartition(), request.url());
+ Credential credential = d->m_context->storageSession()->credentialStorage().get(request.cachePartition(), request.url());
if (!credential.isEmpty()) {
d->m_initialCredential = credential;
@@ -540,16 +540,16 @@
// The stored credential wasn't accepted, stop using it.
// There is a race condition here, since a different credential might have already been stored by another ResourceHandle,
// but the observable effect should be very minor, if any.
- d->m_context->storageSession().credentialStorage().remove(d->m_partition, challenge.protectionSpace());
+ d->m_context->storageSession()->credentialStorage().remove(d->m_partition, challenge.protectionSpace());
}
if (!challenge.previousFailureCount()) {
- Credential credential = d->m_context->storageSession().credentialStorage().get(d->m_partition, challenge.protectionSpace());
+ Credential credential = d->m_context->storageSession()->credentialStorage().get(d->m_partition, challenge.protectionSpace());
if (!credential.isEmpty() && credential != d->m_initialCredential) {
ASSERT(credential.persistence() == CredentialPersistenceNone);
if (challenge.failureResponse().httpStatusCode() == 401) {
// Store the credential back, possibly adding it as a default for this directory.
- d->m_context->storageSession().credentialStorage().set(d->m_partition, credential, challenge.protectionSpace(), challenge.failureResponse().url());
+ d->m_context->storageSession()->credentialStorage().set(d->m_partition, credential, challenge.protectionSpace(), challenge.failureResponse().url());
}
[challenge.sender() useCredential:credential.nsCredential() forAuthenticationChallenge:mac(challenge)];
return true;
@@ -591,7 +591,7 @@
URL urlToStore;
if (challenge.failureResponse().httpStatusCode() == 401)
urlToStore = challenge.failureResponse().url();
- d->m_context->storageSession().credentialStorage().set(d->m_partition, webCredential, ProtectionSpace([d->m_currentMacChallenge protectionSpace]), urlToStore);
+ d->m_context->storageSession()->credentialStorage().set(d->m_partition, webCredential, ProtectionSpace([d->m_currentMacChallenge protectionSpace]), urlToStore);
[[d->m_currentMacChallenge sender] useCredential:webCredential.nsCredential() forAuthenticationChallenge:d->m_currentMacChallenge];
} else
[[d->m_currentMacChallenge sender] useCredential:credential.nsCredential() forAuthenticationChallenge:d->m_currentMacChallenge];
Modified: trunk/Source/WebKit/ChangeLog (239726 => 239727)
--- trunk/Source/WebKit/ChangeLog 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/ChangeLog 2019-01-08 18:08:55 UTC (rev 239727)
@@ -1,5 +1,41 @@
2019-01-08 Alex Christensen <achristen...@webkit.org>
+ Stop using NetworkStorageSession in WebProcess
+ https://bugs.webkit.org/show_bug.cgi?id=193236
+
+ Reviewed by Don Olmstead.
+
+ * Shared/WebProcessCreationParameters.cpp:
+ (WebKit::WebProcessCreationParameters::encode const):
+ (WebKit::WebProcessCreationParameters::decode):
+ * Shared/WebProcessCreationParameters.h:
+ * Shared/WebsitePoliciesData.cpp:
+ (WebKit::WebsitePoliciesData::applyToDocumentLoader):
+ * UIProcess/WebProcessPool.cpp:
+ (WebKit::WebProcessPool::setAnyPageGroupMightHavePrivateBrowsingEnabled):
+ (WebKit::WebProcessPool::tryTakePrewarmedProcess):
+ (WebKit::WebProcessPool::initializeNewWebProcess):
+ (WebKit::WebProcessPool::pageBeginUsingWebsiteDataStore):
+ (WebKit::WebProcessPool::pageEndUsingWebsiteDataStore):
+ (WebKit::WebProcessPool::processForNavigationInternal):
+ * WebProcess/InjectedBundle/InjectedBundle.cpp:
+ (WebKit::InjectedBundle::setPrivateBrowsingEnabled):
+ * WebProcess/WebCoreSupport/mac/WebFrameNetworkingContext.h:
+ * WebProcess/WebCoreSupport/mac/WebFrameNetworkingContext.mm:
+ (WebKit::WebFrameNetworkingContext::ensureWebsiteDataStoreSession): Deleted.
+ (WebKit::WebFrameNetworkingContext::storageSession const): Deleted.
+ * WebProcess/WebPage/WebPage.cpp:
+ (WebKit::WebPage::setSessionID):
+ * WebProcess/WebProcess.cpp:
+ (WebKit::WebProcess::initializeWebProcess):
+ (WebKit::WebProcess::fetchWebsiteData):
+ (WebKit::WebProcess::addWebsiteDataStore): Deleted.
+ (WebKit::WebProcess::destroySession): Deleted.
+ * WebProcess/WebProcess.h:
+ * WebProcess/WebProcess.messages.in:
+
+2019-01-08 Alex Christensen <achristen...@webkit.org>
+
Always call CompletionHandler in Cache::store
https://bugs.webkit.org/show_bug.cgi?id=193237
Modified: trunk/Source/WebKit/Shared/WebProcessCreationParameters.cpp (239726 => 239727)
--- trunk/Source/WebKit/Shared/WebProcessCreationParameters.cpp 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/Shared/WebProcessCreationParameters.cpp 2019-01-08 18:08:55 UTC (rev 239727)
@@ -72,7 +72,6 @@
encoder << shouldCaptureVideoInUIProcess;
encoder << shouldCaptureDisplayInUIProcess;
#endif
- encoder << shouldUseTestingNetworkSession;
encoder << urlSchemesRegisteredAsEmptyDocument;
encoder << urlSchemesRegisteredAsSecure;
encoder << urlSchemesRegisteredAsBypassingContentSecurityPolicy;
@@ -267,8 +266,6 @@
if (!decoder.decode(parameters.shouldCaptureDisplayInUIProcess))
return false;
#endif
- if (!decoder.decode(parameters.shouldUseTestingNetworkSession))
- return false;
if (!decoder.decode(parameters.urlSchemesRegisteredAsEmptyDocument))
return false;
if (!decoder.decode(parameters.urlSchemesRegisteredAsSecure))
Modified: trunk/Source/WebKit/Shared/WebProcessCreationParameters.h (239726 => 239727)
--- trunk/Source/WebKit/Shared/WebProcessCreationParameters.h 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/Shared/WebProcessCreationParameters.h 2019-01-08 18:08:55 UTC (rev 239727)
@@ -124,7 +124,6 @@
double defaultRequestTimeoutInterval { INT_MAX };
- bool shouldUseTestingNetworkSession { false };
bool shouldAlwaysUseComplexTextCodePath { false };
bool shouldEnableMemoryPressureReliefLogging { false };
bool shouldSuppressMemoryPressureHandler { false };
Modified: trunk/Source/WebKit/Shared/WebsitePoliciesData.cpp (239726 => 239727)
--- trunk/Source/WebKit/Shared/WebsitePoliciesData.cpp 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/Shared/WebsitePoliciesData.cpp 2019-01-08 18:08:55 UTC (rev 239727)
@@ -161,11 +161,8 @@
if (websitePolicies.websiteDataStoreParameters) {
if (auto* frame = documentLoader.frame()) {
- if (auto* page = frame->page()) {
- auto sessionID = websitePolicies.websiteDataStoreParameters->networkSessionParameters.sessionID;
- WebProcess::singleton().addWebsiteDataStore(WTFMove(*websitePolicies.websiteDataStoreParameters));
- page->setSessionID(sessionID);
- }
+ if (auto* page = frame->page())
+ page->setSessionID(websitePolicies.websiteDataStoreParameters->networkSessionParameters.sessionID);
}
}
}
Modified: trunk/Source/WebKit/UIProcess/WebProcessPool.cpp (239726 => 239727)
--- trunk/Source/WebKit/UIProcess/WebProcessPool.cpp 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/UIProcess/WebProcessPool.cpp 2019-01-08 18:08:55 UTC (rev 239727)
@@ -689,10 +689,8 @@
{
if (privateBrowsingEnabled) {
sendToNetworkingProcess(Messages::NetworkProcess::AddWebsiteDataStore(WebsiteDataStoreParameters::legacyPrivateSessionParameters()));
- sendToAllProcesses(Messages::WebProcess::AddWebsiteDataStore(WebsiteDataStoreParameters::legacyPrivateSessionParameters()));
} else {
networkProcess()->removeSession(PAL::SessionID::legacyPrivateSessionID());
- sendToAllProcesses(Messages::WebProcess::DestroySession(PAL::SessionID::legacyPrivateSessionID()));
}
}
@@ -765,8 +763,6 @@
ASSERT(m_prewarmedProcess->isPrewarmed());
m_prewarmedProcess->markIsNoLongerInPrewarmedPool();
- if (&m_prewarmedProcess->websiteDataStore() != &websiteDataStore)
- m_prewarmedProcess->send(Messages::WebProcess::AddWebsiteDataStore(websiteDataStore.parameters()), 0);
return std::exchange(m_prewarmedProcess, nullptr);
}
@@ -854,8 +850,6 @@
SandboxExtension::createHandleWithoutResolvingPath(parameters._javascript_ConfigurationDirectory, SandboxExtension::Type::ReadWrite, parameters._javascript_ConfigurationDirectoryExtensionHandle);
}
- parameters.shouldUseTestingNetworkSession = m_shouldUseTestingNetworkSession;
-
parameters.cacheModel = cacheModel();
parameters.languages = userPreferredLanguages();
@@ -942,9 +936,6 @@
process.send(Messages::WebProcess::SetQOS(webProcessLatencyQOS(), webProcessThroughputQOS()), 0);
#endif
- if (WebPreferences::anyPagesAreUsingPrivateBrowsing())
- process.send(Messages::WebProcess::AddWebsiteDataStore(WebsiteDataStoreParameters::legacyPrivateSessionParameters()), 0);
-
if (m_automationSession)
process.send(Messages::WebProcess::EnsureAutomationSessionProxy(m_automationSession->sessionIdentifier()), 0);
@@ -1188,12 +1179,10 @@
ASSERT(page.websiteDataStore().parameters().networkSessionParameters.sessionID == sessionID);
if (m_networkProcess)
m_networkProcess->addSession(makeRef(page.websiteDataStore()));
- page.process().send(Messages::WebProcess::AddWebsiteDataStore(WebsiteDataStoreParameters::privateSessionParameters(sessionID)), 0);
page.websiteDataStore().clearPendingCookies();
} else if (sessionID != PAL::SessionID::defaultSessionID()) {
if (m_networkProcess)
m_networkProcess->addSession(makeRef(page.websiteDataStore()));
- page.process().send(Messages::WebProcess::AddWebsiteDataStore(page.websiteDataStore().parameters()), 0);
page.websiteDataStore().clearPendingCookies();
}
@@ -1224,7 +1213,6 @@
// The last user of this non-default PAL::SessionID is gone, so clean it up in the child processes.
if (networkProcess())
networkProcess()->removeSession(sessionID);
- page.process().send(Messages::WebProcess::DestroySession(sessionID), 0);
}
}
@@ -2238,9 +2226,6 @@
if (&(*it)->page() == &page)
m_suspendedPages.remove(it);
- if (&process->websiteDataStore() != &page.websiteDataStore())
- process->send(Messages::WebProcess::AddWebsiteDataStore(page.websiteDataStore().parameters()), 0);
-
return completionHandler(WTFMove(process), nullptr, reason);
}
Modified: trunk/Source/WebKit/WebProcess/InjectedBundle/InjectedBundle.cpp (239726 => 239727)
--- trunk/Source/WebKit/WebProcess/InjectedBundle/InjectedBundle.cpp 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/WebProcess/InjectedBundle/InjectedBundle.cpp 2019-01-08 18:08:55 UTC (rev 239727)
@@ -357,7 +357,6 @@
ASSERT(!hasProcessPrivilege(ProcessPrivilege::CanAccessRawCookies));
if (enabled) {
WebProcess::singleton().ensureLegacyPrivateBrowsingSessionInNetworkProcess();
- WebFrameNetworkingContext::ensureWebsiteDataStoreSession(WebsiteDataStoreParameters::legacyPrivateSessionParameters());
} else
SessionTracker::destroySession(PAL::SessionID::legacyPrivateSessionID());
Modified: trunk/Source/WebKit/WebProcess/WebCoreSupport/curl/WebFrameNetworkingContext.cpp (239726 => 239727)
--- trunk/Source/WebKit/WebProcess/WebCoreSupport/curl/WebFrameNetworkingContext.cpp 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/WebProcess/WebCoreSupport/curl/WebFrameNetworkingContext.cpp 2019-01-08 18:08:55 UTC (rev 239727)
@@ -56,15 +56,6 @@
{
}
-NetworkStorageSession& WebFrameNetworkingContext::storageSession() const
-{
- if (frame()) {
- if (auto* storageSession = NetworkStorageSession::storageSession(frame()->page()->sessionID()))
- return *storageSession;
- }
- return NetworkStorageSession::defaultStorageSession();
-}
-
WebFrameLoaderClient* WebFrameNetworkingContext::webFrameLoaderClient() const
{
if (!frame())
Modified: trunk/Source/WebKit/WebProcess/WebCoreSupport/curl/WebFrameNetworkingContext.h (239726 => 239727)
--- trunk/Source/WebKit/WebProcess/WebCoreSupport/curl/WebFrameNetworkingContext.h 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/WebProcess/WebCoreSupport/curl/WebFrameNetworkingContext.h 2019-01-08 18:08:55 UTC (rev 239727)
@@ -53,7 +53,7 @@
private:
WebFrameNetworkingContext(WebFrame*);
- WebCore::NetworkStorageSession& storageSession() const override;
+ WebCore::NetworkStorageSession* storageSession() const override { return nullptr; }
};
}
Modified: trunk/Source/WebKit/WebProcess/WebCoreSupport/mac/WebFrameNetworkingContext.h (239726 => 239727)
--- trunk/Source/WebKit/WebProcess/WebCoreSupport/mac/WebFrameNetworkingContext.h 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/WebProcess/WebCoreSupport/mac/WebFrameNetworkingContext.h 2019-01-08 18:08:55 UTC (rev 239727)
@@ -56,7 +56,7 @@
RetainPtr<CFDataRef> sourceApplicationAuditData() const override;
String sourceApplicationIdentifier() const override;
WebCore::ResourceError blockedError(const WebCore::ResourceRequest&) const override;
- WebCore::NetworkStorageSession& storageSession() const override;
+ WebCore::NetworkStorageSession* storageSession() const override { return nullptr; }
};
}
Modified: trunk/Source/WebKit/WebProcess/WebCoreSupport/mac/WebFrameNetworkingContext.mm (239726 => 239727)
--- trunk/Source/WebKit/WebProcess/WebCoreSupport/mac/WebFrameNetworkingContext.mm 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/WebProcess/WebCoreSupport/mac/WebFrameNetworkingContext.mm 2019-01-08 18:08:55 UTC (rev 239727)
@@ -43,20 +43,6 @@
namespace WebKit {
using namespace WebCore;
-void WebFrameNetworkingContext::ensureWebsiteDataStoreSession(WebsiteDataStoreParameters&& parameters)
-{
- ASSERT(!hasProcessPrivilege(ProcessPrivilege::CanAccessRawCookies));
- auto sessionID = parameters.networkSessionParameters.sessionID;
- if (NetworkStorageSession::storageSession(sessionID))
- return;
-
- String base = WebProcess::singleton().uiProcessBundleIdentifier();
- if (base.isNull())
- base = [[NSBundle mainBundle] bundleIdentifier];
-
- NetworkStorageSession::ensureSession(sessionID, base + '.' + String::number(sessionID.sessionID()));
-}
-
bool WebFrameNetworkingContext::localFileContentSniffingEnabled() const
{
return frame() && frame()->settings().localFileContentSniffingEnabled();
@@ -84,19 +70,6 @@
return frame()->loader().client().blockedError(request);
}
-NetworkStorageSession& WebFrameNetworkingContext::storageSession() const
-{
- ASSERT(RunLoop::isMain());
- ASSERT(!hasProcessPrivilege(ProcessPrivilege::CanAccessRawCookies));
- if (frame()) {
- if (auto* storageSession = WebCore::NetworkStorageSession::storageSession(frame()->page()->sessionID()))
- return *storageSession;
- // Some requests may still be coming shortly after WebProcess was told to destroy its session.
- LOG_ERROR("WEB Invalid session ID. Please file a bug unless you just disabled private browsing, in which case it's an expected race.");
- }
- return NetworkStorageSession::defaultStorageSession();
-}
-
WebFrameLoaderClient* WebFrameNetworkingContext::webFrameLoaderClient() const
{
if (!frame())
Modified: trunk/Source/WebKit/WebProcess/WebCoreSupport/soup/WebFrameNetworkingContext.cpp (239726 => 239727)
--- trunk/Source/WebKit/WebProcess/WebCoreSupport/soup/WebFrameNetworkingContext.cpp 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/WebProcess/WebCoreSupport/soup/WebFrameNetworkingContext.cpp 2019-01-08 18:08:55 UTC (rev 239727)
@@ -58,15 +58,6 @@
{
}
-NetworkStorageSession& WebFrameNetworkingContext::storageSession() const
-{
- if (frame()) {
- if (auto* storageSession = NetworkStorageSession::storageSession(frame()->page()->sessionID()))
- return *storageSession;
- }
- return NetworkStorageSession::defaultStorageSession();
-}
-
WebFrameLoaderClient* WebFrameNetworkingContext::webFrameLoaderClient() const
{
if (!frame())
Modified: trunk/Source/WebKit/WebProcess/WebCoreSupport/soup/WebFrameNetworkingContext.h (239726 => 239727)
--- trunk/Source/WebKit/WebProcess/WebCoreSupport/soup/WebFrameNetworkingContext.h 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/WebProcess/WebCoreSupport/soup/WebFrameNetworkingContext.h 2019-01-08 18:08:55 UTC (rev 239727)
@@ -51,7 +51,7 @@
private:
WebFrameNetworkingContext(WebFrame*);
- WebCore::NetworkStorageSession& storageSession() const override;
+ WebCore::NetworkStorageSession* storageSession() const override { return nullptr; }
};
}
Modified: trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp (239726 => 239727)
--- trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp 2019-01-08 18:08:55 UTC (rev 239727)
@@ -3043,8 +3043,6 @@
void WebPage::setSessionID(PAL::SessionID sessionID)
{
- if (sessionID.isEphemeral())
- WebProcess::singleton().addWebsiteDataStore(WebsiteDataStoreParameters::privateSessionParameters(sessionID));
m_page->setSessionID(sessionID);
}
Modified: trunk/Source/WebKit/WebProcess/WebProcess.cpp (239726 => 239727)
--- trunk/Source/WebKit/WebProcess/WebProcess.cpp 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/WebProcess/WebProcess.cpp 2019-01-08 18:08:55 UTC (rev 239727)
@@ -381,9 +381,6 @@
setShouldUseFontSmoothing(parameters.shouldUseFontSmoothing);
- if (parameters.shouldUseTestingNetworkSession)
- NetworkStorageSession::switchToNewTestingSession();
-
ensureNetworkProcessConnection();
setTerminationTimeout(parameters.terminationTimeout);
@@ -540,16 +537,6 @@
m_fullKeyboardAccessEnabled = fullKeyboardAccessEnabled;
}
-void WebProcess::addWebsiteDataStore(WebsiteDataStoreParameters&& parameters)
-{
- WebFrameNetworkingContext::ensureWebsiteDataStoreSession(WTFMove(parameters));
-}
-
-void WebProcess::destroySession(PAL::SessionID sessionID)
-{
- SessionTracker::destroySession(sessionID);
-}
-
void WebProcess::ensureLegacyPrivateBrowsingSessionInNetworkProcess()
{
ensureNetworkProcessConnection().connection().send(Messages::NetworkConnectionToWebProcess::EnsureLegacyPrivateBrowsingSession(), 0);
@@ -1294,11 +1281,6 @@
for (auto& origin : MemoryCache::singleton().originsWithCache(sessionID))
websiteData.entries.append(WebsiteData::Entry { origin->data(), WebsiteDataType::MemoryCache, 0 });
}
-
- if (websiteDataTypes.contains(WebsiteDataType::Credentials)) {
- if (NetworkStorageSession::storageSession(sessionID))
- websiteData.originsWithCredentials = NetworkStorageSession::storageSession(sessionID)->credentialStorage().originsWithCredentials();
- }
}
void WebProcess::deleteWebsiteData(PAL::SessionID sessionID, OptionSet<WebsiteDataType> websiteDataTypes, WallTime modifiedSince)
Modified: trunk/Source/WebKit/WebProcess/WebProcess.h (239726 => 239727)
--- trunk/Source/WebKit/WebProcess/WebProcess.h 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/WebProcess/WebProcess.h 2019-01-08 18:08:55 UTC (rev 239727)
@@ -178,8 +178,6 @@
void setCacheModel(CacheModel);
void ensureLegacyPrivateBrowsingSessionInNetworkProcess();
- void addWebsiteDataStore(WebsiteDataStoreParameters&&);
- void destroySession(PAL::SessionID);
void pageDidEnterWindow(uint64_t pageID);
void pageWillLeaveWindow(uint64_t pageID);
Modified: trunk/Source/WebKit/WebProcess/WebProcess.messages.in (239726 => 239727)
--- trunk/Source/WebKit/WebProcess/WebProcess.messages.in 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKit/WebProcess/WebProcess.messages.in 2019-01-08 18:08:55 UTC (rev 239727)
@@ -53,9 +53,6 @@
ClearCachedCredentials()
- AddWebsiteDataStore(struct WebKit::WebsiteDataStoreParameters websiteDataStoreParameters);
- DestroySession(PAL::SessionID sessionID)
-
# Plug-ins.
DidAddPlugInAutoStartOriginHash(uint32_t hash, WallTime expirationTime, PAL::SessionID sessionID)
ResetPlugInAutoStartOriginDefaultHashes(HashMap<uint32_t,WallTime> hashes)
Modified: trunk/Source/WebKitLegacy/mac/ChangeLog (239726 => 239727)
--- trunk/Source/WebKitLegacy/mac/ChangeLog 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKitLegacy/mac/ChangeLog 2019-01-08 18:08:55 UTC (rev 239727)
@@ -1,3 +1,17 @@
+2019-01-08 Alex Christensen <achristen...@webkit.org>
+
+ Stop using NetworkStorageSession in WebProcess
+ https://bugs.webkit.org/show_bug.cgi?id=193236
+
+ Reviewed by Don Olmstead.
+
+ * WebCoreSupport/WebFrameNetworkingContext.h:
+ * WebCoreSupport/WebFrameNetworkingContext.mm:
+ (WebFrameNetworkingContext::storageSession const):
+ * WebView/WebView.mm:
+ (-[WebView _cachedResponseForURL:]):
+ (-[WebView _clearCredentials]):
+
2019-01-07 David Kilzer <ddkil...@apple.com>
Prefer RetainPtr<NSObject> to RetainPtr<NSObject *>
Modified: trunk/Source/WebKitLegacy/mac/WebCoreSupport/WebFrameNetworkingContext.h (239726 => 239727)
--- trunk/Source/WebKitLegacy/mac/WebCoreSupport/WebFrameNetworkingContext.h 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKitLegacy/mac/WebCoreSupport/WebFrameNetworkingContext.h 2019-01-08 18:08:55 UTC (rev 239727)
@@ -49,5 +49,5 @@
RetainPtr<CFDataRef> sourceApplicationAuditData() const override;
String sourceApplicationIdentifier() const override;
WebCore::ResourceError blockedError(const WebCore::ResourceRequest&) const override;
- WebCore::NetworkStorageSession& storageSession() const override;
+ WebCore::NetworkStorageSession* storageSession() const override;
};
Modified: trunk/Source/WebKitLegacy/mac/WebCoreSupport/WebFrameNetworkingContext.mm (239726 => 239727)
--- trunk/Source/WebKitLegacy/mac/WebCoreSupport/WebFrameNetworkingContext.mm 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKitLegacy/mac/WebCoreSupport/WebFrameNetworkingContext.mm 2019-01-08 18:08:55 UTC (rev 239727)
@@ -90,14 +90,14 @@
return frame()->loader().client().blockedError(request);
}
-NetworkStorageSession& WebFrameNetworkingContext::storageSession() const
+NetworkStorageSession* WebFrameNetworkingContext::storageSession() const
{
ASSERT(isMainThread());
if (frame() && frame()->page() && frame()->page()->sessionID().isEphemeral()) {
if (auto* session = NetworkStorageSession::storageSession(PAL::SessionID::legacyPrivateSessionID()))
- return *session;
+ return session;
// Some requests may still be coming shortly before WebCore updates the session ID and after WebKit destroys the private browsing session.
LOG_ERROR("Invalid session ID. Please file a bug unless you just disabled private browsing, in which case it's an expected race.");
}
- return NetworkStorageSession::defaultStorageSession();
+ return &NetworkStorageSession::defaultStorageSession();
}
Modified: trunk/Source/WebKitLegacy/mac/WebView/WebView.mm (239726 => 239727)
--- trunk/Source/WebKitLegacy/mac/WebView/WebView.mm 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKitLegacy/mac/WebView/WebView.mm 2019-01-08 18:08:55 UTC (rev 239727)
@@ -3668,7 +3668,7 @@
if (!_private->page)
return nil;
- if (CFURLStorageSessionRef storageSession = _private->page->mainFrame().loader().networkingContext()->storageSession().platformSession())
+ if (auto storageSession = _private->page->mainFrame().loader().networkingContext()->storageSession()->platformSession())
cachedResponse = cachedResponseForRequest(storageSession, request.get());
else
cachedResponse = [[NSURLCache sharedURLCache] cachedResponseForRequest:request.get()];
@@ -9251,7 +9251,7 @@
if (!networkingContext)
return;
- networkingContext->storageSession().credentialStorage().clearCredentials();
+ networkingContext->storageSession()->credentialStorage().clearCredentials();
}
- (BOOL)_needsOneShotDrawingSynchronization
Modified: trunk/Source/WebKitLegacy/win/WebCoreSupport/WebFrameNetworkingContext.cpp (239726 => 239727)
--- trunk/Source/WebKitLegacy/win/WebCoreSupport/WebFrameNetworkingContext.cpp 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKitLegacy/win/WebCoreSupport/WebFrameNetworkingContext.cpp 2019-01-08 18:08:55 UTC (rev 239727)
@@ -99,12 +99,12 @@
return frame()->loader().client().blockedError(request);
}
-NetworkStorageSession& WebFrameNetworkingContext::storageSession() const
+NetworkStorageSession* WebFrameNetworkingContext::storageSession() const
{
ASSERT(isMainThread());
if (frame() && frame()->page()->usesEphemeralSession())
- return *NetworkStorageSession::storageSession(PAL::SessionID::legacyPrivateSessionID());
+ return NetworkStorageSession::storageSession(PAL::SessionID::legacyPrivateSessionID());
- return NetworkStorageSession::defaultStorageSession();
+ return &NetworkStorageSession::defaultStorageSession();
}
Modified: trunk/Source/WebKitLegacy/win/WebCoreSupport/WebFrameNetworkingContext.h (239726 => 239727)
--- trunk/Source/WebKitLegacy/win/WebCoreSupport/WebFrameNetworkingContext.h 2019-01-08 17:59:52 UTC (rev 239726)
+++ trunk/Source/WebKitLegacy/win/WebCoreSupport/WebFrameNetworkingContext.h 2019-01-08 18:08:55 UTC (rev 239727)
@@ -47,5 +47,5 @@
}
WebCore::ResourceError blockedError(const WebCore::ResourceRequest&) const override;
- WebCore::NetworkStorageSession& storageSession() const override;
+ WebCore::NetworkStorageSession* storageSession() const override;
};