API functions returning Promises should not throw exceptions.

Source/bindings/core/v8/V8BindingMacros.h

Index: Source/bindings/core/v8/V8BindingMacros.h
diff --git a/Source/bindings/core/v8/V8BindingMacros.h b/Source/bindings/core/v8/V8BindingMacros.h
index 165d5b7b0144f1c7756a018f4110c2bc2a598063..2756f1d76c74f25a306b99ad4320e4c046461f32 100644
--- a/Source/bindings/core/v8/V8BindingMacros.h
+++ b/Source/bindings/core/v8/V8BindingMacros.h
@@ -70,6 +70,23 @@ namespace blink {
         }                                          \
     }
 
+#define TONATIVE_VOID_ASYNC_INTERNAL(var, value, info)                                          \
+    var = (value);                                                                              \
+    if (UNLIKELY(block.HasCaught())) {                                                          \
+        v8SetReturnValue(info, ScriptPromise::rejectRaw(info.GetIsolate(), block.Exception())); \
+        rethrow.cancel();                                                                       \
+        return;                                                                                 \
+    }
+
+#define TONATIVE_VOID_ASYNC(type, var, value, info)     \
+    type var;                                           \
+    {                                                   \
+        v8::TryCatch block;                             \
+        V8RethrowTryCatchScope rethrow(block);          \

[haraken, 2014/07/31 07:54:47]

Do we need this? It looks redundant to set a V8RethrowTryCatchScope and cancel
it later.

[yhirano, 2014/08/01 02:27:57]

We need this to reuse the x_ASYNC_INTERNAL macro in a x_ASYNC macro.
I can delete it if I expand the x_ASYNC_INTERNAL macro.
What do you think?

[haraken, 2014/08/01 02:42:40]

Makes sense. Let's keep the x_ASYNC_INTERNAL macro as is for consistency with
other INTERNAL macros.

+        TONATIVE_VOID_ASYNC_INTERNAL(var, value, info); \
+    }
+
+
 #define TONATIVE_VOID_EXCEPTIONSTATE_INTERNAL(var, value, exceptionState) \
     var = (value);                                                        \
     if (UNLIKELY(block.HasCaught() || exceptionState.throwIfNeeded()))    \
@@ -93,6 +110,26 @@ namespace blink {
             return retVal;                                                        \
     }
 
+#define TONATIVE_VOID_EXCEPTIONSTATE_ASYNC_INTERNAL(scriptState, var, value, exceptionState, info) \
+    var = (value);                                                                                 \
+    if (UNLIKELY(block.HasCaught())) {                                                             \
+        v8SetReturnValue(info, ScriptPromise::rejectRaw(info.GetIsolate(), block.Exception()));    \
+        rethrow.cancel();                                                                          \

[haraken, 2014/07/31 07:54:47]

Shall we add a comment and mention why we need to cancel it?

[yhirano, 2014/08/01 02:27:57]

Done.

+        return;                                                                                    \
+    }                                                                                              \
+    if (UNLIKELY(exceptionState.hadException())) {                                                 \
+        v8SetReturnValue(info, exceptionState.reject(scriptState).v8Value());                      \
+        return;                                                                                    \
+    }
+
+#define TONATIVE_VOID_EXCEPTIONSTATE_ASYNC(scriptState, type, var, value, exceptionState, info) \
+    type var;                                                                                   \
+    {                                                                                           \
+        v8::TryCatch block;                                                                     \
+        V8RethrowTryCatchScope rethrow(block);                                                  \
+        TONATIVE_VOID_EXCEPTIONSTATE_INTERNAL(scriptState, var, value, exceptionState, info);   \

[haraken, 2014/07/31 07:54:47]

Why doesn't this call TONATIVE_VOID_EXCEPTIONSTATE_ASYNC_INTERNAL?

[yhirano, 2014/08/01 02:27:57]

You're right. Thanks, done.

+    }
+
 // 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
@@ -111,6 +148,22 @@ namespace blink {
     if (UNLIKELY(!var.prepare()))                  \
         return retVal;
 
+#define TOSTRING_VOID_ASYNC_INTERNAL(var, value, info)                                             \
+    var = (value);                                                                                 \
+    if (UNLIKELY(!var.prepare()))  {                                                               \
+        rethrow.cancel();                                                                          \

[haraken, 2014/07/31 07:54:47]

Just for consistency, let's move rethrow.cancel() to below line 155.

[yhirano, 2014/08/01 02:27:57]

Done.

+        info.GetReturnValue().Set(ScriptPromise::rejectRaw(info.GetIsolate(), block.Exception())); \
+        return;                                                                                    \
+    }
+
+#define TOSTRING_VOID_ASYNC(type, var, value, info)           \
+    type var;                                                 \
+    {                                                         \
+        v8::TryCatch block;                                   \
+        V8RethrowTryCatchScope rethrow(block);                \
+        TOSTRING_VOID_ASYNC_INTERNAL(type, var, value, info); \
+    }
+
 } // namespace blink
 
 #endif // V8BindingMacros_h