Remove custom code for JavaScriptCallFrame.setVariableValue() operation

Remove custom code for JavaScriptCallFrame.setVariableValue() operation

Remove custom code for JavaScriptCallFrame.setVariableValue() operation by using
ScriptValue instead of v8::Value on the implementation side and by marking
the arguments as optional to behave the same way.

TEST=inspector-protocol/debugger/debugger-setVariableValue.html

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

[Inactive, 2013-11-08 21:28:53]

For reference, the generated code looks like:
static void setVariableValueMethod(const v8::FunctionCallbackInfo<v8::Value>&
info)
{
    JavaScriptCallFrame* imp = V8JavaScriptCallFrame::toNative(info.Holder());
    V8TRYCATCH_VOID(int, scopeIndex, toInt32(info[0]));
   
V8TRYCATCH_FOR_V8STRINGRESOURCE_VOID(V8StringResource<WithUndefinedOrNullCheck>,
variableName, info[1]);
    V8TRYCATCH_VOID(ScriptValue, newValue, ScriptValue(info[2],
info.GetIsolate()));
    v8SetReturnValue(info, imp->setVariableValue(scopeIndex, variableName,
newValue).v8Value());
}

It is very close to the custom code, modulo the V8TRYCATCH_*() macros.

[haraken, 2013-11-08 21:56:41]

LGTM.

[commit-bot: I haz the power, 2013-11-11 13:59:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/67373003/1

[commit-bot: I haz the power, 2013-11-12 16:42:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/67373003/1

[commit-bot: I haz the power, 2013-11-12 17:26:10]

Change committed as 161831