[webkit-changes] [137810] trunk/Source/WebKit2

Sat, 15 Dec 2012 14:02:27 -0800

Title: [137810] trunk/Source/WebKit2
Revision
137810
Author
[email protected]
Date
2012-12-15 14:04:12 -0800 (Sat, 15 Dec 2012)

Log Message

DownloadProxy should keep a strong reference to its associated web context
https://bugs.webkit.org/show_bug.cgi?id=105107

Reviewed by Alexey Proskuryakov.

While this does create a ref-cycle, it's broken when the download completes, fails, is canceled or
if the process that's doing the download crashes.

* UIProcess/Downloads/DownloadProxy.cpp:
(WebKit::DownloadProxy::processDidClose):
(WebKit::DownloadProxy::didStart):
(WebKit::DownloadProxy::didReceiveAuthenticationChallenge):
(WebKit::DownloadProxy::didReceiveResponse):
(WebKit::DownloadProxy::didReceiveData):
(WebKit::DownloadProxy::shouldDecodeSourceDataOfMIMEType):
(WebKit::DownloadProxy::decideDestinationWithSuggestedFilename):
(WebKit::DownloadProxy::didCreateDestination):
(WebKit::DownloadProxy::didFinish):
(WebKit::DownloadProxy::didFail):
(WebKit::DownloadProxy::didCancel):
* UIProcess/Downloads/DownloadProxy.h:
(DownloadProxy):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (137809 => 137810)


--- trunk/Source/WebKit2/ChangeLog	2012-12-15 21:53:11 UTC (rev 137809)
+++ trunk/Source/WebKit2/ChangeLog	2012-12-15 22:04:12 UTC (rev 137810)
@@ -1,5 +1,30 @@
 2012-12-15  Anders Carlsson  <[email protected]>
 
+        DownloadProxy should keep a strong reference to its associated web context
+        https://bugs.webkit.org/show_bug.cgi?id=105107
+
+        Reviewed by Alexey Proskuryakov.
+
+        While this does create a ref-cycle, it's broken when the download completes, fails, is canceled or
+        if the process that's doing the download crashes.
+
+        * UIProcess/Downloads/DownloadProxy.cpp:
+        (WebKit::DownloadProxy::processDidClose):
+        (WebKit::DownloadProxy::didStart):
+        (WebKit::DownloadProxy::didReceiveAuthenticationChallenge):
+        (WebKit::DownloadProxy::didReceiveResponse):
+        (WebKit::DownloadProxy::didReceiveData):
+        (WebKit::DownloadProxy::shouldDecodeSourceDataOfMIMEType):
+        (WebKit::DownloadProxy::decideDestinationWithSuggestedFilename):
+        (WebKit::DownloadProxy::didCreateDestination):
+        (WebKit::DownloadProxy::didFinish):
+        (WebKit::DownloadProxy::didFail):
+        (WebKit::DownloadProxy::didCancel):
+        * UIProcess/Downloads/DownloadProxy.h:
+        (DownloadProxy):
+
+2012-12-15  Anders Carlsson  <[email protected]>
+
         Remove WebContext::downloadFinished
         https://bugs.webkit.org/show_bug.cgi?id=105105

Modified: trunk/Source/WebKit2/UIProcess/Downloads/DownloadProxy.cpp (137809 => 137810)


--- trunk/Source/WebKit2/UIProcess/Downloads/DownloadProxy.cpp	2012-12-15 21:53:11 UTC (rev 137809)
+++ trunk/Source/WebKit2/UIProcess/Downloads/DownloadProxy.cpp	2012-12-15 22:04:12 UTC (rev 137810)
@@ -82,7 +82,7 @@
     if (!m_webContext)
         return;
 
-    m_webContext->downloadClient().processDidCrash(m_webContext, this);
+    m_webContext->downloadClient().processDidCrash(m_webContext.get(), this);
 }
 
 void DownloadProxy::didReceiveMessage(CoreIPC::Connection* connection, CoreIPC::MessageID messageID, CoreIPC::MessageDecoder& decoder)
@@ -102,7 +102,7 @@
     if (!m_webContext)
         return;
 
-    m_webContext->downloadClient().didStart(m_webContext, this);
+    m_webContext->downloadClient().didStart(m_webContext.get(), this);
 }
 
 void DownloadProxy::didReceiveAuthenticationChallenge(const AuthenticationChallenge& authenticationChallenge, uint64_t challengeID)
@@ -113,7 +113,7 @@
     // FIXME (Multi-WebProcess): <rdar://problem/12239483> Downloads shouldn't be handled in the web process.
     // Once this is fixed, remove WebContext::deprecatedSharedProcess().
     RefPtr<AuthenticationChallengeProxy> authenticationChallengeProxy = AuthenticationChallengeProxy::create(authenticationChallenge, challengeID, m_webContext->deprecatedSharedProcess());
-    m_webContext->downloadClient().didReceiveAuthenticationChallenge(m_webContext, this, authenticationChallengeProxy.get());
+    m_webContext->downloadClient().didReceiveAuthenticationChallenge(m_webContext.get(), this, authenticationChallengeProxy.get());
 }
 
 void DownloadProxy::didReceiveResponse(const ResourceResponse& response)
@@ -121,7 +121,7 @@
     if (!m_webContext)
         return;
 
-    m_webContext->downloadClient().didReceiveResponse(m_webContext, this, response);
+    m_webContext->downloadClient().didReceiveResponse(m_webContext.get(), this, response);
 }
 
 void DownloadProxy::didReceiveData(uint64_t length)
@@ -129,7 +129,7 @@
     if (!m_webContext)
         return;
 
-    m_webContext->downloadClient().didReceiveData(m_webContext, this, length);
+    m_webContext->downloadClient().didReceiveData(m_webContext.get(), this, length);
 }
 
 void DownloadProxy::shouldDecodeSourceDataOfMIMEType(const String& mimeType, bool& result)
@@ -137,7 +137,7 @@
     if (!m_webContext)
         return;
 
-    result = m_webContext->downloadClient().shouldDecodeSourceDataOfMIMEType(m_webContext, this, mimeType);
+    result = m_webContext->downloadClient().shouldDecodeSourceDataOfMIMEType(m_webContext.get(), this, mimeType);
 }
 
 void DownloadProxy::decideDestinationWithSuggestedFilename(const String& filename, String& destination, bool& allowOverwrite, SandboxExtension::Handle& sandboxExtensionHandle)
@@ -145,7 +145,7 @@
     if (!m_webContext)
         return;
 
-    destination = m_webContext->downloadClient().decideDestinationWithSuggestedFilename(m_webContext, this, filename, allowOverwrite);
+    destination = m_webContext->downloadClient().decideDestinationWithSuggestedFilename(m_webContext.get(), this, filename, allowOverwrite);
 
     if (!destination.isNull())
         SandboxExtension::createHandle(destination, SandboxExtension::WriteOnly, sandboxExtensionHandle);
@@ -156,7 +156,7 @@
     if (!m_webContext)
         return;
 
-    m_webContext->downloadClient().didCreateDestination(m_webContext, this, path);
+    m_webContext->downloadClient().didCreateDestination(m_webContext.get(), this, path);
 }
 
 void DownloadProxy::didFinish()
@@ -164,7 +164,7 @@
     if (!m_webContext)
         return;
 
-    m_webContext->downloadClient().didFinish(m_webContext, this);
+    m_webContext->downloadClient().didFinish(m_webContext.get(), this);
 
     // This can cause the DownloadProxy object to be deleted.
     m_downloadProxyMap.downloadFinished(this);
@@ -185,7 +185,7 @@
 
     m_resumeData = createWebData(resumeData);
 
-    m_webContext->downloadClient().didFail(m_webContext, this, error);
+    m_webContext->downloadClient().didFail(m_webContext.get(), this, error);
 
     // This can cause the DownloadProxy object to be deleted.
     m_downloadProxyMap.downloadFinished(this);
@@ -195,7 +195,7 @@
 {
     m_resumeData = createWebData(resumeData);
 
-    m_webContext->downloadClient().didCancel(m_webContext, this);
+    m_webContext->downloadClient().didCancel(m_webContext.get(), this);
 
     // This can cause the DownloadProxy object to be deleted.
     m_downloadProxyMap.downloadFinished(this);

Modified: trunk/Source/WebKit2/UIProcess/Downloads/DownloadProxy.h (137809 => 137810)


--- trunk/Source/WebKit2/UIProcess/Downloads/DownloadProxy.h	2012-12-15 21:53:11 UTC (rev 137809)
+++ trunk/Source/WebKit2/UIProcess/Downloads/DownloadProxy.h	2012-12-15 22:04:12 UTC (rev 137810)
@@ -91,7 +91,7 @@
     void didCancel(const CoreIPC::DataReference& resumeData);
 
     DownloadProxyMap& m_downloadProxyMap;
-    WebContext* m_webContext;
+    RefPtr<WebContext> m_webContext;
     uint64_t m_downloadID;
 
     RefPtr<WebData> m_resumeData;

_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to