Title: [212987] trunk/Source/WebCore
Revision
212987
Author
[email protected]
Date
2017-02-24 19:23:10 -0800 (Fri, 24 Feb 2017)

Log Message

Unreviewed, follow-up fix after r212972.

Fixes a few assertions on the debug build bots.
URL needs to be exactly the same as the parsed one given
that we are calling the ParsedURLString constructor.

* platform/network/ResourceResponseBase.cpp:
(WebCore::ResourceResponseBase::sanitizeSuggestedFilename):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (212986 => 212987)


--- trunk/Source/WebCore/ChangeLog	2017-02-25 02:14:40 UTC (rev 212986)
+++ trunk/Source/WebCore/ChangeLog	2017-02-25 03:23:10 UTC (rev 212987)
@@ -1,3 +1,14 @@
+2017-02-24  Chris Dumez  <[email protected]>
+
+        Unreviewed, follow-up fix after r212972.
+
+        Fixes a few assertions on the debug build bots.
+        URL needs to be exactly the same as the parsed one given
+        that we are calling the ParsedURLString constructor.
+
+        * platform/network/ResourceResponseBase.cpp:
+        (WebCore::ResourceResponseBase::sanitizeSuggestedFilename):
+
 2017-02-24  Zalan Bujtas  <[email protected]>
 
         Simple line layout: Re-adjust paginated lines when block height changes.

Modified: trunk/Source/WebCore/platform/network/ResourceResponseBase.cpp (212986 => 212987)


--- trunk/Source/WebCore/platform/network/ResourceResponseBase.cpp	2017-02-25 02:14:40 UTC (rev 212986)
+++ trunk/Source/WebCore/platform/network/ResourceResponseBase.cpp	2017-02-25 03:23:10 UTC (rev 212987)
@@ -229,7 +229,7 @@
     if (suggestedFilename.isEmpty())
         return suggestedFilename;
 
-    ResourceResponse response(URL(ParsedURLString, "http://example.com"), String(), -1, String());
+    ResourceResponse response(URL(ParsedURLString, "http://example.com/"), String(), -1, String());
     response.setHTTPStatusCode(200);
     String escapedSuggestedFilename = String(suggestedFilename).replace('\"', "\\\"");
     String value = makeString("attachment; filename=\"", escapedSuggestedFilename, '"');
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to