Changeset 86799 in webkit

Timestamp:

May 18, 2011 3:17:13 PM (13 years ago)

Author:

brettw@chromium.org

Message:

2011-05-18 Brett Wilson <​brettw@chromium.org>

Reviewed by Adam Barth.

Don't create empty file objects if no download file path is specified.
​https://bugs.webkit.org/show_bug.cgi?id=60798

• src/WebURLResponse.cpp: (WebKit::WebURLResponse::setDownloadFilePath):

Location:

trunk/Source/WebKit/chromium

Files:

• ChangeLog (modified) (1 diff)
• src/WebURLResponse.cpp (modified) (1 diff)

trunk/Source/WebKit/chromium/ChangeLog

@@ +1 @@
+2011-05-18  Brett Wilson  <brettw@chromium.org>
+
+        Reviewed by Adam Barth.
+
+        Don't create empty file objects if no download file path is specified.
+        https://bugs.webkit.org/show_bug.cgi?id=60798
+
+        * src/WebURLResponse.cpp:
+        (WebKit::WebURLResponse::setDownloadFilePath):
+
 2011-05-17  Mikhail Naganov  <mnaganov@chromium.org>
 

trunk/Source/WebKit/chromium/src/WebURLResponse.cpp

@@ -374 +374 @@
 void WebURLResponse::setDownloadFilePath(const WebString& downloadFilePath)
 {
-    m_private->m_resourceResponse->setDownloadedFile(File::create(downloadFilePath));
+    m_private->m_resourceResponse->setDownloadedFile(downloadFilePath.isEmpty() ? 0 : File::create(downloadFilePath));
 }