2006-10-20 Anders Carlsson <acarlsson@apple.com>
authorandersca <andersca@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 20 Oct 2006 21:05:39 +0000 (21:05 +0000)
committerandersca <andersca@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 20 Oct 2006 21:05:39 +0000 (21:05 +0000)
        Reviewed by Darin and Geoff.

        Remove debugging output.
        * xml/xmlhttprequest.cpp:
        (WebCore::XMLHttpRequest::send):

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

WebCore/ChangeLog
WebCore/xml/xmlhttprequest.cpp

index 9be79498856feb6aeb82fa3090a1c5ee45e28d1f..add98d213d9b26fd2f940b2bf79959cad5d3c247 100644 (file)
@@ -1,3 +1,11 @@
+2006-10-20  Anders Carlsson  <acarlsson@apple.com>
+
+        Reviewed by Darin and Geoff.
+
+        Remove debugging output.
+        * xml/xmlhttprequest.cpp:
+        (WebCore::XMLHttpRequest::send):
+
 2006-10-20  David Harrison  <harrison@apple.com>
 
         Reviewed by John.
index 4b16247dcedf6f538d4724bd3f6ea76c2381b3ed..a1264c8df120703903dc371f2f164198f06165ec 100644 (file)
@@ -351,10 +351,11 @@ void XMLHttpRequest::send(const String& body, ExceptionCode& ec)
         gcProtectNullTolerant(KJS::ScriptInterpreter::getDOMObject(this));
     }
   
-    if (!m_loader->start(m_doc->docLoader())) {
-        LOG_ERROR("Failed to send an XMLHttpRequest for %s", m_url.url().ascii());
+    // start can return false here, for example if we're no longer attached to a page.
+    // this is true while running onunload handlers
+    // FIXME: Maybe start can return false for other reasons too?
+    if (!m_loader->start(m_doc->docLoader()))
         m_loader = 0;
-    }
 }
 
 void XMLHttpRequest::abort()