Title: [207856] trunk/Source/WebKit2
Revision
207856
Author
[email protected]
Date
2016-10-25 16:42:43 -0700 (Tue, 25 Oct 2016)

Log Message

[SOUP] Network process assertion in NetworkDataTask::continueHTTPRedirection
https://bugs.webkit.org/show_bug.cgi?id=163941

Patch by Carlos Garcia Campos <[email protected]> on 2016-10-25
Reviewed by Michael Catanzaro.

equalLettersIgnoringASCIICase() expects the second argument to be the letters to compare in lowercase, so use
"get" instead of "GET".

* NetworkProcess/soup/NetworkDataTaskSoup.cpp:
(WebKit::NetworkDataTaskSoup::continueHTTPRedirection):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (207855 => 207856)


--- trunk/Source/WebKit2/ChangeLog	2016-10-25 23:22:48 UTC (rev 207855)
+++ trunk/Source/WebKit2/ChangeLog	2016-10-25 23:42:43 UTC (rev 207856)
@@ -1,3 +1,16 @@
+2016-10-25  Carlos Garcia Campos  <[email protected]>
+
+        [SOUP] Network process assertion in NetworkDataTask::continueHTTPRedirection
+        https://bugs.webkit.org/show_bug.cgi?id=163941
+
+        Reviewed by Michael Catanzaro.
+
+        equalLettersIgnoringASCIICase() expects the second argument to be the letters to compare in lowercase, so use
+        "get" instead of "GET".
+
+        * NetworkProcess/soup/NetworkDataTaskSoup.cpp:
+        (WebKit::NetworkDataTaskSoup::continueHTTPRedirection):
+
 2016-10-25  Eric Carlson  <[email protected]>
 
         [MediaStream] Add "has capture device" bit to media state flags

Modified: trunk/Source/WebKit2/NetworkProcess/soup/NetworkDataTaskSoup.cpp (207855 => 207856)


--- trunk/Source/WebKit2/NetworkProcess/soup/NetworkDataTaskSoup.cpp	2016-10-25 23:22:48 UTC (rev 207855)
+++ trunk/Source/WebKit2/NetworkProcess/soup/NetworkDataTaskSoup.cpp	2016-10-25 23:42:43 UTC (rev 207856)
@@ -619,7 +619,7 @@
         request.clearHTTPReferrer();
 
     bool isCrossOrigin = !protocolHostAndPortAreEqual(m_firstRequest.url(), request.url());
-    if (!equalLettersIgnoringASCIICase(request.httpMethod(), "GET")) {
+    if (!equalLettersIgnoringASCIICase(request.httpMethod(), "get")) {
         // Change newRequest method to GET if change was made during a previous redirection or if current redirection says so.
         if (m_soupMessage->method == SOUP_METHOD_GET || !request.url().protocolIsInHTTPFamily() || shouldRedirectAsGET(m_soupMessage.get(), isCrossOrigin)) {
             request.setHTTPMethod("GET");
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to