Reviewed by Chris.
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 11 Dec 2003 23:30:30 +0000 (23:30 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 11 Dec 2003 23:30:30 +0000 (23:30 +0000)
<rdar://problem/3507859>: Status text should only include "OK", not full "HTTP/1.0 200 OK" status line

        * khtml/ecma/xmlhttprequest.cpp:
        (KJS::XMLHttpRequest::getStatusText): Include only the status
text, not the full status line.

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

WebCore/ChangeLog-2005-08-23
WebCore/khtml/ecma/xmlhttprequest.cpp

index 6db8fda..b80379e 100644 (file)
@@ -1,3 +1,13 @@
+2003-12-11  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Chris.
+
+       <rdar://problem/3507859>: Status text should only include "OK", not full "HTTP/1.0 200 OK" status line
+       
+        * khtml/ecma/xmlhttprequest.cpp:
+        (KJS::XMLHttpRequest::getStatusText): Include only the status
+       text, not the full status line.
+
 === Safari-117 ===
 
 2003-12-11  David Hyatt  <hyatt@apple.com>
index fd6023c..34593e0 100644 (file)
@@ -445,8 +445,16 @@ Value XMLHttpRequest::getStatusText() const
   
   int endOfLine = responseHeaders.find("\n");
   QString firstLine = endOfLine == -1 ? responseHeaders : responseHeaders.left(endOfLine);
+  int codeStart = firstLine.find(" ");
+  int codeEnd = firstLine.find(" ", codeStart + 1);
+
+  if (codeStart == -1 || codeEnd == -1) {
+    return Undefined();
+  }
+  
+  QString statusText = firstLine.mid(codeEnd + 1, endOfLine - (codeEnd + 1)).stripWhiteSpace();
   
-  return String(firstLine);
+  return String(statusText);
 }
 
 #if APPLE_CHANGES