Title: [260205] trunk/Source/WebCore
Revision
260205
Author
commit-qu...@webkit.org
Date
2020-04-16 11:38:20 -0700 (Thu, 16 Apr 2020)

Log Message

Remove outdated comment from FrameLoader
https://bugs.webkit.org/show_bug.cgi?id=210607

Patch by Rob Buis <rb...@igalia.com> on 2020-04-16
Reviewed by Darin Adler.

Remove comment from FrameLoader that is not valid/important anymore because
addExtraFieldsToRequest does not set the Origin header since r259036.

* loader/FrameLoader.cpp:
(WebCore::FrameLoader::loadDifferentDocumentItem):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (260204 => 260205)


--- trunk/Source/WebCore/ChangeLog	2020-04-16 18:38:18 UTC (rev 260204)
+++ trunk/Source/WebCore/ChangeLog	2020-04-16 18:38:20 UTC (rev 260205)
@@ -1,3 +1,16 @@
+2020-04-16  Rob Buis  <rb...@igalia.com>
+
+        Remove outdated comment from FrameLoader
+        https://bugs.webkit.org/show_bug.cgi?id=210607
+
+        Reviewed by Darin Adler.
+
+        Remove comment from FrameLoader that is not valid/important anymore because
+        addExtraFieldsToRequest does not set the Origin header since r259036.
+
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::loadDifferentDocumentItem):
+
 2020-04-16  Chris Fleizach  <cfleiz...@apple.com>
 
         AX: Need method for setting selected range from NSRange

Modified: trunk/Source/WebCore/loader/FrameLoader.cpp (260204 => 260205)


--- trunk/Source/WebCore/loader/FrameLoader.cpp	2020-04-16 18:38:18 UTC (rev 260204)
+++ trunk/Source/WebCore/loader/FrameLoader.cpp	2020-04-16 18:38:20 UTC (rev 260205)
@@ -3756,8 +3756,6 @@
             request.setHTTPOrigin(origin);
         }
 
-        // Make sure to add extra fields to the request after the Origin header is added for the FormData case.
-        // See https://bugs.webkit.org/show_bug.cgi?id=22194 for more discussion.
         addExtraFieldsToRequest(request, IsMainResource::Yes, loadType);
         
         // FIXME: Slight hack to test if the NSURL cache contains the page we're going to.
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to