Fix some builds after r234516
authorachristensen@apple.com <achristensen@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 2 Aug 2018 22:31:48 +0000 (22:31 +0000)
committerachristensen@apple.com <achristensen@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 2 Aug 2018 22:31:48 +0000 (22:31 +0000)
​https://bugs.webkit.org/show_bug.cgi?id=188250

* Platform/IPC/ArgumentCoders.h:
This assertion was comparing integers of different signs.
Since we don't allow exception throwing in WebKit, valueless_by_exception won't be true,
so this assertion will always be true.  I'll just remove it.

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

Source/WebKit/ChangeLog
Source/WebKit/Platform/IPC/ArgumentCoders.h

index e157581..98efbe6 100644 (file)
@@ -1,3 +1,13 @@
+2018-08-02  Alex Christensen  <achristensen@webkit.org>
+
+        Fix some builds after r234516
+        ​https://bugs.webkit.org/show_bug.cgi?id=188250
+
+        * Platform/IPC/ArgumentCoders.h:
+        This assertion was comparing integers of different signs.
+        Since we don't allow exception throwing in WebKit, valueless_by_exception won't be true,
+        so this assertion will always be true.  I'll just remove it.
+
 2018-08-02  Sihui Liu  <sihui_liu@apple.com>
 
         httpCookieStore.getAllCookies() does not always call completion block
index 6cadb1c..daea626 100644 (file)
@@ -561,7 +561,6 @@ struct VariantCoder<0, Types...> {
 template<typename... Types> struct ArgumentCoder<WTF::Variant<Types...>> {
     static void encode(Encoder& encoder, const WTF::Variant<Types...>& variant)
     {
-        ASSERT(static_cast<unsigned>(variant.index()) == variant.index());
         unsigned i = variant.index();
         encoder << i;
         VariantCoder<sizeof...(Types) - 1, Types...>::encode(encoder, variant, i);