Attachment 350136 Details for Bug 189679 – PATCH

[patch] PATCH

189679.diff (text/plain), 1.91 KB, created by Basuke Suzuki on 2018-09-19 11:35:41 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Basuke Suzuki

Created: 2018-09-19 11:35:41 PDT

Size: 1.91 KB

patch

obsolete

>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index d424f6e6c51..0f3dd631fb7 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,15 @@
>+2018-09-19  Basuke Suzuki  <Basuke.Suzuki@sony.com>
>+
>+        [Win] TestRunner::queueLoad() fails to generate correct url for some urls.
>+        https://bugs.webkit.org/show_bug.cgi?id=189679
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Replace the implementation with correct API call.
>+
>+        * DumpRenderTree/win/TestRunnerWin.cpp:
>+        (TestRunner::queueLoad): Call UrlCombineW.
>+
> 2018-09-19  Michael Catanzaro  <mcatanzaro@igalia.com>
> 
>         [WPE][GTK] Unreviewed, fix that typo in more places
>diff --git a/Tools/DumpRenderTree/win/TestRunnerWin.cpp b/Tools/DumpRenderTree/win/TestRunnerWin.cpp
>index b0fbe3cf38e..c99ef7bcc7a 100644
>--- a/Tools/DumpRenderTree/win/TestRunnerWin.cpp
>+++ b/Tools/DumpRenderTree/win/TestRunnerWin.cpp
>@@ -339,15 +339,18 @@ void TestRunner::queueLoad(JSStringRef url, JSStringRef target)
>         return;
>     wstring responseURL(responseURLBSTR, responseURLBSTR.length());
> 
>-    // FIXME: We should do real relative URL resolution here.
>-    int lastSlash = responseURL.rfind('/');
>-    if (lastSlash != -1)
>-        responseURL = responseURL.substr(0, lastSlash);
>-
>     wstring wURL = jsStringRefToWString(url);
>-    wstring wAbsoluteURL = responseURL + TEXT("/") + wURL;
>-    auto jsAbsoluteURL = adopt(JSStringCreateWithCharacters(wAbsoluteURL.data(), wAbsoluteURL.length()));
> 
>+    std::vector<wchar_t> buffer;
>+    DWORD bufferSize = 128;
>+    while (true) {
>+        buffer.resize(bufferSize);
>+        auto result = UrlCombineW(responseURL.data(), wURL.data(), buffer.data(), &bufferSize, 0);
>+        if (result == S_OK)
>+            break;
>+        bufferSize *= 2;
>+    }
>+    auto jsAbsoluteURL = adopt(JSStringCreateWithCharacters(buffer.data(), bufferSize));
>     WorkQueue::singleton().queue(new LoadItem(jsAbsoluteURL.get(), target));
> }
>

Actions: View | Formatted Diff | Diff

Attachments on bug 189679: 350136 | 350144 | 350232