IDL: Drop value conversion (V8 -> C++) macros from generated code

Source/bindings/core/v8/ExceptionState.h

Index: Source/bindings/core/v8/ExceptionState.h
diff --git a/Source/bindings/core/v8/ExceptionState.h b/Source/bindings/core/v8/ExceptionState.h
index 3acd126e8b3af60582f566143500b7c7a8bcb119..9683f5ded5f3fbad29e60aef5fd9df5315ba4ff5 100644
--- a/Source/bindings/core/v8/ExceptionState.h
+++ b/Source/bindings/core/v8/ExceptionState.h
@@ -96,6 +96,8 @@ public:
         return true;
     }
 
+    void throwException();

[yhirano, 2015/02/24 03:51:24]

Can you tell me why you want this function in addition to throwIfNeeded?
I don't like this function name, because it is very confusing (what
throwTypeError does is very different from what throwException does). I think
this function should not be public.

[Jens Widell, 2015/02/24 07:31:43]

I use it in a situation where I know an exception was thrown, in the "stored in
this object" sense, and just need it thrown in the "throw in V8" sense. Using
throwIfNeeded() would have worked, but would have been somewhat "dishonest"
since there's no question that throwing is needed.

I agree that the name of this function is unfortunate. But the same problem
exists in "throwIfNeeded", I think, since the problem is that "throw" is
overloaded with two different meanings, and "throwIfNeeded" and "throwException"
use it the same way.

The best naming scheme I can think of would be to change "throwTypeError" (and
friends) to "scheduleTypeError", leaving these the only "throw"ing functions.

Personally, I'm not convinced we must change the naming, or that this function
must be private. It's name may be somewhat misleading, but it's fairly obvious
from its signature (no arguments) that it can't be used to "schedule" a new
exception. (It also asserts if you try to use it that way.)

[Jens Widell, 2015/02/24 15:34:56]

I've reverted this change, as it wasn't really required.

+
     // This method clears out the exception which |this| has.
     ScriptPromise reject(ScriptState*);
 
@@ -121,7 +123,6 @@ protected:
 
 private:
     void setException(v8::Handle<v8::Value>);
-    void throwException();
 
     String addExceptionContext(const String&) const;