Title: [86799] trunk/Source/WebKit/chromium
- Revision
- 86799
- Author
- [email protected]
- Date
- 2011-05-18 15:17:13 -0700 (Wed, 18 May 2011)
Log Message
2011-05-18 Brett Wilson <[email protected]>
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):
Modified Paths
Diff
Modified: trunk/Source/WebKit/chromium/ChangeLog (86798 => 86799)
--- trunk/Source/WebKit/chromium/ChangeLog 2011-05-18 22:10:55 UTC (rev 86798)
+++ trunk/Source/WebKit/chromium/ChangeLog 2011-05-18 22:17:13 UTC (rev 86799)
@@ -1,3 +1,13 @@
+2011-05-18 Brett Wilson <[email protected]>
+
+ 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 <[email protected]>
Reviewed by Pavel Feldman.
Modified: trunk/Source/WebKit/chromium/src/WebURLResponse.cpp (86798 => 86799)
--- trunk/Source/WebKit/chromium/src/WebURLResponse.cpp 2011-05-18 22:10:55 UTC (rev 86798)
+++ trunk/Source/WebKit/chromium/src/WebURLResponse.cpp 2011-05-18 22:17:13 UTC (rev 86799)
@@ -373,7 +373,7 @@
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));
}
WebString WebURLResponse::remoteIPAddress() const
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
