[bindings] Pass NormalConversion to to[U]Int{8/16/32/64}(...) in v8_types.py and reduce conversion …

Source/bindings/core/v8/PrivateScriptRunner.cpp

Index: Source/bindings/core/v8/PrivateScriptRunner.cpp
diff --git a/Source/bindings/core/v8/PrivateScriptRunner.cpp b/Source/bindings/core/v8/PrivateScriptRunner.cpp
index bfac52528535af061d8d5062074962fa9f47266a..be3c61035d114ac48b9fbd764ac37d077054b130 100644
--- a/Source/bindings/core/v8/PrivateScriptRunner.cpp
+++ b/Source/bindings/core/v8/PrivateScriptRunner.cpp
@@ -242,7 +242,7 @@ void rethrowExceptionInPrivateScript(v8::Isolate* isolate, v8::TryCatch& block,
         RELEASE_ASSERT(!code.IsEmpty() && code->IsInt32());
         ScriptState::Scope scope(scriptStateInUserScript);
         ExceptionState exceptionState(errorContext, propertyName, interfaceName, scriptStateInUserScript->context()->Global(), scriptStateInUserScript->isolate());
-        exceptionState.throwDOMException(toInt32(isolate, code), messageString);
+        exceptionState.throwDOMException(toInt32(isolate, code, NormalConversion, exceptionState), messageString);

[Jens Widell, 2015/03/17 18:03:13]

Actually, technically it's wrong to use the existing exceptionState here, I
think. It's for throwing an exception in a different context, is it not?

(It won't matter much since toInt32() won't throw anything; the value is known
to be an int32 here.)

[haraken, 2015/03/18 01:23:13]

Ah, you're right; sorry, let's revert it back to the NonThrowableExceptionState.

Also it would be better to call toInt32() just after the code->isInt32() check.

         exceptionState.throwIfNeeded();
         return;
     }