IDL: Make SerializedScriptValue arguments less of a special case

IDL: Make SerializedScriptValue arguments less of a special case

Remove the special handling of SerializedScriptValue arguments from the
methods.cpp template, and instead fix the already existing handling in
v8_types.py.

The conversion uses an ExceptionState object to report exceptions, and is
now handled like any other conversion that does that.

BUG=413257
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=182613

[Jens Widell, 2014-09-24 14:59:07]

jl@opera.com changed reviewers:
+ haraken@chromium.org

[Jens Widell, 2014-09-24 14:59:07]

PTAL

It's worth pointing out that all uses of SerializedScriptValue in our IDL files
(aside from core/testing/Internals.idl) are for read-only attributes with
[Custom=getter] or as arguments to methods with [Custom]. IOW, how we handle the
type in the generated code doesn't matter much at all. We could drop support for
it entirely if we removed it from core/testing/Internals.idl.

https://codereview.chromium.org/600863002/diff/1/Source/bindings/tests/result...
File Source/bindings/tests/results/core/V8TestObject.cpp (left):

https://codereview.chromium.org/600863002/diff/1/Source/bindings/tests/result...
Source/bindings/tests/results/core/V8TestObject.cpp:997:
TONATIVE_VOID(RefPtr<SerializedScriptValue>, cppValue,
SerializedScriptValue::create(v8Value, info.GetIsolate()));
Note: The SerializedScriptValue::create(v8::Handle<v8::Value>, v8::Isolate*)
overload previously called here doesn't actually exist.

Only SerializedScriptValue method arguments had special handling, and only that
special handling was correct.

[haraken, 2014-09-24 15:11:14]

LGTM

[Jens Widell, 2014-09-24 15:12:46]

The CQ bit was checked by jl@opera.com

[commit-bot: I haz the power, 2014-09-24 15:13:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/600863002/1

[commit-bot: I haz the power, 2014-09-24 15:33:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-24 15:34:00]

Try jobs failed on following builders:
  mac_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_compile_dbg/bu...)

[Jens Widell, 2014-09-24 15:44:17]

The CQ bit was checked by jl@opera.com

[commit-bot: I haz the power, 2014-09-24 15:45:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/600863002/1

[commit-bot: I haz the power, 2014-09-24 16:33:24]

Committed patchset #1 (id:1) as 182613