Reviewed by Adam Roben.
authorap@webkit.org <ap@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 10 Jan 2008 21:33:19 +0000 (21:33 +0000)
committerap@webkit.org <ap@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 10 Jan 2008 21:33:19 +0000 (21:33 +0000)
        <rdar://problem/5667003> fast/dom/xmlhttprequest-html-response-encoding.html is failing

        * xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::send): Do not disable sniffing for file://
        requests, as CFNetwork doesn't perform extension to MIME type mapping then.

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

LayoutTests/ChangeLog
LayoutTests/platform/win/Skipped
WebCore/ChangeLog
WebCore/xml/XMLHttpRequest.cpp

index ba199ea72777b481c54c34eecd840272f3815c70..c1a0aa1413a6d595c8209cb3c1954743f98fbe70 100644 (file)
@@ -1,3 +1,11 @@
+2008-01-10  Alexey Proskuryakov  <ap@webkit.org>
+
+        Reviewed by Adam Roben.
+
+        <rdar://problem/5667003> fast/dom/xmlhttprequest-html-response-encoding.html is failing
+
+        * platform/win/Skipped: Remove the fixed test.
+
 2008-01-10  Kevin McCullough  <kmccullough@apple.com>
 
         Reviewed by Darin, Sam, and Adam.
index f13bfa7815e9992102948ee3820e65ae2259981f..ac9ba16e4b375a2e9b96633f5c4d85b6711d981f 100644 (file)
@@ -364,9 +364,6 @@ http/tests/security/dataURL/xss-DENIED-to-data-url-sub-frame-uppercase.html
 # <rdar://5666926>
 svg/custom/use-css-no-effect-on-shadow-tree.svg
 
-# <rdar://5667003> Undefined value exception
-fast/dom/xmlhttprequest-html-response-encoding.html
-
 # <rdar://5667007> 2 SVG text tests failing (due to bad rounding?)
 svg/batik/text/textOnPath.svg
 svg/batik/text/verticalTextOnPath.svg
index 173e0863d278cdbc67cac05642a453f1ba6c6e63..cbba8be9e2478963d0db463211cc7bdcb32116ce 100644 (file)
@@ -1,3 +1,12 @@
+2008-01-10  Alexey Proskuryakov  <ap@webkit.org>
+
+        Reviewed by Adam Roben.
+
+        <rdar://problem/5667003> fast/dom/xmlhttprequest-html-response-encoding.html is failing
+
+        * xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::send): Do not disable sniffing for file://
+        requests, as CFNetwork doesn't perform extension to MIME type mapping then.
+
 2008-01-10  Adam Roben  <aroben@apple.com>
 
         Fixes to allow multiple FrameViews on Windows
index 254020fe2d039015a24a199964a4ad27cd86fd1f..51c52ec6a671526befd0fa507740e20124ae980d 100644 (file)
@@ -474,10 +474,13 @@ void XMLHttpRequest::send(const String& body, ExceptionCode& ec)
         gcProtectNullTolerant(KJS::ScriptInterpreter::getDOMObject(this));
     }
   
-    // create can return null here, for example if we're no longer attached to a page.
-    // this is true while running onunload handlers
-    // FIXME: Maybe create can return false for other reasons too?
-    m_loader = SubresourceLoader::create(m_doc->frame(), this, request, false, true, false);
+    // SubresourceLoader::create can return null here, for example if we're no longer attached to a page.
+    // This is true while running onunload handlers.
+    // FIXME: We need to be able to send XMLHttpRequests from onunload, <http://bugs.webkit.org/show_bug.cgi?id=10904>.
+    // FIXME: Maybe create can return null for other reasons too?
+    // We need to keep content sniffing enabled for local files due to CFNetwork not providing a MIME type
+    // for local files otherwise, <rdar://problem/5671813>.
+    m_loader = SubresourceLoader::create(m_doc->frame(), this, request, false, true, request.url().isLocalFile());
 }
 
 void XMLHttpRequest::abort()