API functions returning Promises should not throw exceptions.

Source/bindings/v8/V8BindingMacros.h

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 39 matching lines @@
     type var;                              \
     {                                      \
         v8::TryCatch block;                \
         var = (value);                     \
         if (UNLIKELY(block.HasCaught())) { \
             block.ReThrow();               \
             return;                        \
         }                                  \
     }
 
+#define TONATIVE_VOID_ASYNC(type, var, value, info)                                    \
+    type var;                                                                          \
+    {                                                                                  \
+        v8::TryCatch block;                                                            \
+        var = (value);                                                                 \
+        if (UNLIKELY(block.HasCaught())) {                                             \
+            v8::Isolate* isolate = info.GetIsolate();                                  \
+            ScriptPromise promise = ScriptPromise::reject(block.Exception(), isolate); \
+            v8SetReturnValue(info, promise.v8Value());                                 \
+            return;                                                                    \
+        }                                                                              \
+    }
+
 #define TONATIVE_BOOL(type, var, value, retVal) \
     type var;                                     \
     {                                             \
         v8::TryCatch block;                       \
         var = (value);                            \
         if (UNLIKELY(block.HasCaught())) {        \
             block.ReThrow();                      \
             return retVal;                        \
         }                                         \
     }
 
 #define TONATIVE_VOID_EXCEPTIONSTATE(type, var, value, exceptionState) \
     type var;                                                            \
     {                                                                    \
         v8::TryCatch block;                                              \
         var = (value);                                                   \
         if (UNLIKELY(block.HasCaught()))                                 \
             exceptionState.rethrowV8Exception(block.Exception());        \
         if (UNLIKELY(exceptionState.throwIfNeeded()))                    \
             return;                                                      \
     }
 
+#define TONATIVE_VOID_EXCEPTIONSTATE_ASYNC(type, var, value, exceptionState, info) \
+    type var;                                                                      \
+    {                                                                              \
+        v8::TryCatch block;                                                        \
+        var = (value);                                                             \
+        if (UNLIKELY(block.HasCaught()))                                           \
+            exceptionState.rethrowV8Exception(block.Exception());                  \
+        if (UNLIKELY(exceptionState.hadException())) {                             \
+            v8SetReturnValue(info, exceptionState.reject().v8Value());             \
+            return;                                                                \
+        }                                                                          \
+    }
+
 #define TONATIVE_BOOL_EXCEPTIONSTATE(type, var, value, exceptionState, retVal) \

[pneubeck (no reviews), 2014/04/17 09:27:04]

As discussed with haraken@, this (and the other _BOOL_ macros) should be
_DEFAULT_ instead of _BOOL_, because there are/can be usages where retVal is not
boolean.

[Nils Barth (inactive), 2014/04/18 05:27:56]

Fix posted:
Rename macros TONATIVE_BOOL* and TOSTRING_BOOL to *_DEFAULT
https://codereview.chromium.org/242223004/

[yhirano, 2014/04/18 16:55:54]

Thanks!

-    type var;                                                                 \
-    {                                                                         \
-        v8::TryCatch block;                                                   \
-        var = (value);                                                        \
-        if (UNLIKELY(block.HasCaught()))                                      \
-            exceptionState.rethrowV8Exception(block.Exception());             \
-        if (UNLIKELY(exceptionState.throwIfNeeded()))                         \
-            return retVal;                                                    \
+    type var;                                                                  \
+    {                                                                          \
+        v8::TryCatch block;                                                    \
+        var = (value);                                                         \
+        if (UNLIKELY(block.HasCaught()))                                       \
+            exceptionState.rethrowV8Exception(block.Exception());              \
+        if (UNLIKELY(exceptionState.throwIfNeeded()))                          \
+            return retVal;                                                     \
     }
 
 // type is an instance of class template V8StringResource<>,
 // but Mode argument varies; using type (not Mode) for consistency
 // with other macros and ease of code generation
-#define TOSTRING_VOID(type, var, value) \
-    type var(value);                    \
-    if (UNLIKELY(!var.prepare()))       \
-        return;
+#define TOSTRING_VOID(type, var, value)    \
+    type var(value);                       \
+    {                                      \
+        v8::TryCatch block;                \
+        var.prepare();                     \
+        if (UNLIKELY(block.HasCaught())) { \
+            block.ReThrow();               \
+            return;                        \
+        }                                  \
+    }
 
 #define TOSTRING_BOOL(type, var, value, retVal) \
-    type var(value);                                 \
-    if (UNLIKELY(!var.prepare()))                    \
-        return retVal;
+    type var(value);                            \
+    {                                           \
+        v8::TryCatch block;                     \
+        var.prepare();                          \
+        if (UNLIKELY(block.HasCaught())) {      \
+            block.ReThrow();                    \
+            return retVal;                      \
+        }                                       \
+    }
+
+#define TOSTRING_VOID_ASYNC(type, var, value, info)                                    \
+    type var(value);                                                                   \
+    {                                                                                  \
+        v8::TryCatch block;                                                            \
+        var.prepare();                                                                 \
+        if (UNLIKELY(block.HasCaught())) {                                             \
+            v8::Isolate* isolate = info.GetIsolate();                                  \
+            ScriptPromise promise = ScriptPromise::reject(block.Exception(), isolate); \
+            v8SetReturnValue(info, promise.v8Value());                                 \
+            return;                                                                    \
+        }                                                                              \
+    }
 
 } // namespace WebCore
 
 #endif // V8BindingMacros_h