Attachment 358665 Details for Bug 193271 – Patch

[patch] Patch

bug-193271-20190109134532.patch (text/plain), 2.05 KB, created by Fujii Hironori on 2019-01-08 20:45:33 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Fujii Hironori

Created: 2019-01-08 20:45:33 PST

Size: 2.05 KB

patch

obsolete

>Subversion Revision: 239762
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index f21329fea6d2c2af3e1b0d42ccb8e22a24adbc23..a45193e0e492f632fc238cbc3e17256ee9d96e8a 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,17 @@
>+2019-01-08  Fujii Hironori  <Hironori.Fujii@sony.com>
>+
>+        [Win][MiniBrowser] wchar_t strings shouldn't be treated as BSTR
>+        https://bugs.webkit.org/show_bug.cgi?id=193271
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        BSTR is a special data structure created by SysAllocString, not a
>+        wchar_t null-terminated string.
>+
>+        * MiniBrowser/win/MiniBrowserWebHost.cpp:
>+        (MiniBrowserWebHost::didFinishLoadForFrame): Use _bstr_t to create
>+        BSTR from wide string literals.
>+
> 2019-01-08  Dean Jackson  <dino@apple.com>
> 
>         Blob references for System Previews don't get a correct file extension
>diff --git a/Tools/MiniBrowser/win/MiniBrowserWebHost.cpp b/Tools/MiniBrowser/win/MiniBrowserWebHost.cpp
>index 7fe73246745ef9fc3c74a7d645053f2ad941f7eb..a604b8ce12f9a791d8efa609ee7a48c6ba3d2e26 100644
>--- a/Tools/MiniBrowser/win/MiniBrowserWebHost.cpp
>+++ b/Tools/MiniBrowser/win/MiniBrowserWebHost.cpp
>@@ -167,7 +167,8 @@ HRESULT MiniBrowserWebHost::didFinishLoadForFrame(_In_opt_ IWebView* webView, _I
>         m_client->showLastVisitedSites(*webView);
> 
>     // The following is for the test page:
>-    HRESULT hr = doc->getElementById(L"webkit logo", &element.GetInterfacePtr());
>+    static _bstr_t id = L"webkit logo";
>+    HRESULT hr = doc->getElementById(id, &element.GetInterfacePtr());
>     if (!SUCCEEDED(hr))
>         return hr;
> 
>@@ -175,7 +176,9 @@ HRESULT MiniBrowserWebHost::didFinishLoadForFrame(_In_opt_ IWebView* webView, _I
>     if (!SUCCEEDED(hr))
>         return hr;
> 
>-    hr = target->addEventListener(L"click", new SimpleEventListener (L"webkit logo click"), FALSE);
>+    static _bstr_t eventName = L"click";
>+    static _bstr_t eventType = L"webkit logo click";
>+    hr = target->addEventListener(eventName, new SimpleEventListener(eventType), FALSE);
>     if (!SUCCEEDED(hr))
>         return hr;
>

Actions: View | Formatted Diff | Diff

Attachments on bug 193271: 358665 | 358668