Changeset 80249 in webkit

Timestamp:

Mar 3, 2011 9:36:04 AM (13 years ago)

Author:

Adam Roben

Message:

Work around a bug in KURL's parsing of Windows-style absolute file: URLs

Fixes <​http://webkit.org/b/55674> Many tests fail in WebKit2 mode on Windows due to
incorrect parsing of absolute Windows-style file: URLs

Reviewed by Tim Hatcher.

• WebKitTestRunner/TestInvocation.cpp:

(WTR::createWKURL): Add a "localhost" host to file: URLs we create to work around
<​http://webkit.org/b/55683>.

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• WebKitTestRunner/TestInvocation.cpp (modified) (1 diff)

trunk/Tools/ChangeLog

@@ +1 @@
+2011-03-03  Adam Roben  <aroben@apple.com>
+
+        Work around a bug in KURL's parsing of Windows-style absolute file: URLs
+
+        Fixes <http://webkit.org/b/55674> Many tests fail in WebKit2 mode on Windows due to
+        incorrect parsing of absolute Windows-style file: URLs
+
+        Reviewed by Tim Hatcher.
+
+        * WebKitTestRunner/TestInvocation.cpp:
+        (WTR::createWKURL): Add a "localhost" host to file: URLs we create to work around
+        <http://webkit.org/b/55683>.
+
 2011-03-03  Tony Chang  <tony@chromium.org>
 

trunk/Tools/WebKitTestRunner/TestInvocation.cpp

@@ -59 +59 @@
         return 0;
 
-    const char* filePrefix = "file://";
+    // FIXME: Remove the "localhost/" suffix once <http://webkit.org/b/55683> is fixed.
+    const char* filePrefix = "file://localhost/";
     static const size_t prefixLength = strlen(filePrefix);
 #if OS(WINDOWS)