[BlackBerry] We should update the status in NetworkJob if there is a new one from...
authorjason.liu@torchmobile.com.cn <jason.liu@torchmobile.com.cn@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Jul 2012 15:04:17 +0000 (15:04 +0000)
committerjason.liu@torchmobile.com.cn <jason.liu@torchmobile.com.cn@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Jul 2012 15:04:17 +0000 (15:04 +0000)
https://bugs.webkit.org/show_bug.cgi?id=91475

Reviewed by Yong Li.

Libcurl sometimes sends multiple status messages, we need to keep the last
one in NetworkJob.
We originally had the m_statusReceived check, then we found out that libcurl
sometimes sent additional 401 codes and added the 401 exception to the check,
and now we're removing the whole check(so we don't need the exception either).

RIM PR# 163172
Reviewed internally by Joe Mason.

No new tests. This is caused by libcurl's multiple status messages.
So we don't need to write a test case for webkit.

* platform/network/blackberry/NetworkJob.cpp:
(WebCore::NetworkJob::handleNotifyStatusReceived):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@122972 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/network/blackberry/NetworkJob.cpp

index 1b67de6..31d3b63 100644 (file)
@@ -1,3 +1,25 @@
+2012-07-18  Jason Liu  <jason.liu@torchmobile.com.cn>
+
+        [BlackBerry] We should update the status in NetworkJob if there is a new one from libcurl.
+        https://bugs.webkit.org/show_bug.cgi?id=91475
+
+        Reviewed by Yong Li.
+
+        Libcurl sometimes sends multiple status messages, we need to keep the last 
+        one in NetworkJob.
+        We originally had the m_statusReceived check, then we found out that libcurl 
+        sometimes sent additional 401 codes and added the 401 exception to the check, 
+        and now we're removing the whole check(so we don't need the exception either).
+
+        RIM PR# 163172
+        Reviewed internally by Joe Mason.
+
+        No new tests. This is caused by libcurl's multiple status messages.
+        So we don't need to write a test case for webkit.
+
+        * platform/network/blackberry/NetworkJob.cpp:
+        (WebCore::NetworkJob::handleNotifyStatusReceived):
+
 2012-07-18  Kevin Ellis  <kevers@chromium.org>
 
         TOUCH_ADJUSTMENT is too aggressive when snapping to large elements.
index 56b8760..687e4d7 100644 (file)
@@ -192,7 +192,7 @@ void NetworkJob::notifyStatusReceived(int status, const char* message)
 void NetworkJob::handleNotifyStatusReceived(int status, const String& message)
 {
     // Check for messages out of order or after cancel.
-    if ((m_statusReceived && m_extendedStatusCode != 401) || m_responseSent || m_cancelled)
+    if (m_responseSent || m_cancelled)
         return;
 
     if (isInfo(status))