[Win][Clang][WebKitLegacy] error: cannot pass object of non-trivial type through...
authorHironori.Fujii@sony.com <Hironori.Fujii@sony.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 18 Sep 2018 02:54:37 +0000 (02:54 +0000)
committerHironori.Fujii@sony.com <Hironori.Fujii@sony.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 18 Sep 2018 02:54:37 +0000 (02:54 +0000)
https://bugs.webkit.org/show_bug.cgi?id=189612

Reviewed by Alex Christensen.

Non-trivial types can't be used as aruguments of LOG_ERROR and
ASSERT_WITH_MESSAGE.

* DefaultPolicyDelegate.cpp:
(DefaultPolicyDelegate::unableToImplementPolicyWithError): Apply
static_cast<BSTR> to convert BString.
* WebLocalizableStrings.cpp:
(copyLocalizedStringFromBundle): Convert 'key' from WTF::String to
char* by using 'key.utf8().data()'.

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

Source/WebKitLegacy/win/ChangeLog
Source/WebKitLegacy/win/DefaultPolicyDelegate.cpp
Source/WebKitLegacy/win/WebLocalizableStrings.cpp

index 65e2ab0..abb2763 100644 (file)
@@ -1,3 +1,20 @@
+2018-09-17  Fujii Hironori  <Hironori.Fujii@sony.com>
+
+        [Win][Clang][WebKitLegacy] error: cannot pass object of non-trivial type through variadic function
+        https://bugs.webkit.org/show_bug.cgi?id=189612
+
+        Reviewed by Alex Christensen.
+
+        Non-trivial types can't be used as aruguments of LOG_ERROR and
+        ASSERT_WITH_MESSAGE.
+
+        * DefaultPolicyDelegate.cpp:
+        (DefaultPolicyDelegate::unableToImplementPolicyWithError): Apply
+        static_cast<BSTR> to convert BString.
+        * WebLocalizableStrings.cpp:
+        (copyLocalizedStringFromBundle): Convert 'key' from WTF::String to
+        char* by using 'key.utf8().data()'.
+
 2018-09-13  Ryan Haddad  <ryanhaddad@apple.com>
 
         Unreviewed, rolling out r235953.
index 3b00fb9..e8e5f5c 100644 (file)
@@ -179,7 +179,7 @@ HRESULT DefaultPolicyDelegate::unableToImplementPolicyWithError(_In_opt_ IWebVie
     BString frameName;
     frame->name(&frameName);
 
-    LOG_ERROR("called unableToImplementPolicyWithError:%S inFrame:%S", errorStr ? errorStr : TEXT(""), frameName ? frameName : TEXT(""));
+    LOG_ERROR("called unableToImplementPolicyWithError:%S inFrame:%S", errorStr ? static_cast<BSTR>(errorStr) : TEXT(""), frameName ? static_cast<BSTR>(frameName) : TEXT(""));
 
     return S_OK;
 }
index c55607f..de0221f 100644 (file)
@@ -161,7 +161,7 @@ static CFStringRef copyLocalizedStringFromBundle(WebLocalizableStringsBundle* st
 
     CFStringRef result = CFCopyLocalizedStringWithDefaultValue(key.createCFString().get(), 0, bundle, notFound, 0);
 
-    ASSERT_WITH_MESSAGE(result != notFound, "could not find localizable string %s in bundle", key);
+    ASSERT_WITH_MESSAGE(result != notFound, "could not find localizable string %s in bundle", key.utf8().data());
     return result;
 }