binding: Introduces ExceptionToPromiseScope.

third_party/WebKit/Source/bindings/core/v8/GeneratedCodeHelper.cpp

Index: third_party/WebKit/Source/bindings/core/v8/GeneratedCodeHelper.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/GeneratedCodeHelper.cpp b/third_party/WebKit/Source/bindings/core/v8/GeneratedCodeHelper.cpp
index e449086ed7acba36288a6a3904f1cc105dae021c..5ac409892eafed661905d79cf026bf8bda02189f 100644
--- a/third_party/WebKit/Source/bindings/core/v8/GeneratedCodeHelper.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/GeneratedCodeHelper.cpp
@@ -4,6 +4,7 @@
 
 #include "bindings/core/v8/GeneratedCodeHelper.h"
 
+#include "bindings/core/v8/ExceptionMessages.h"
 #include "bindings/core/v8/SerializedScriptValue.h"
 #include "bindings/core/v8/V8Binding.h"
 
@@ -26,4 +27,24 @@ v8::Local<v8::Value> v8Deserialize(v8::Isolate* isolate, PassRefPtr<SerializedSc
     return v8::Null(isolate);
 }
 
+void throwMinimumArityError(ExceptionState& exceptionState, unsigned expected, unsigned actual)

[haraken, 2016/09/02 16:24:13]

These helper functions are just one line, so I'd prefer inlining them in the
call sites.

Even if we want to have the helper functions, I want to avoid putting it in
GeneratedCodeHelper since it's used in hand-written bindings as well. (I still
don't understand what GeneratedCodeHelper is.)

[Yuki, 2016/09/05 07:19:25]

I think we'd like to guarantee the following things.
- Arity error is a TypeError.
- The error message is the same among all APIs.

These helpers are useful to make sure of those things.
Do you think V8XxxCustom.cpp is out of scope?

Otherwise, all the clients are generated bindings, and no one in core/ or
modules/ use these helpers, or I don't expect such use cases in future.

[haraken, 2016/09/05 08:24:22]

I don't know. Even if we adopt your definition of generated bindings, I don't
see much benefit of distinguishing helper classes for generated bindings vs.
non-generated bindings.

Tangentically related, but Onion Soup is going to move core/modules-independent
parts of bindings/ into platform/.

+{
+    exceptionState.throwTypeError(ExceptionMessages::notEnoughArguments(expected, actual));
+}
+
+void throwInvalidArityError(ExceptionState& exceptionState, const char* validRange, unsigned actual)
+{
+    exceptionState.throwTypeError(ExceptionMessages::invalidArity(validRange, actual));
+}
+
+void throwMinimumArityErrorForMethod(v8::Isolate* isolate, const char* interface, const char* method, unsigned expected, unsigned actual)
+{
+    V8ThrowException::throwTypeError(isolate, ExceptionMessages::failedToExecute(method, interface, ExceptionMessages::notEnoughArguments(expected, actual)));
+}
+
+void throwMinimumArityErrorForConstructor(v8::Isolate* isolate, const char* interface, unsigned expected, unsigned actual)
+{
+    V8ThrowException::throwTypeError(isolate, ExceptionMessages::failedToConstruct(interface, ExceptionMessages::notEnoughArguments(expected, actual)));
+}
+
 } // namespace blink