Use V8TRYCATCH_FOR_V8STRINGRESOURCE() macro instead of toCoreStringWithUndefinedOrNullCheck()

Source/bindings/v8/custom/V8JavaScriptCallFrameCustom.cpp

Index: Source/bindings/v8/custom/V8JavaScriptCallFrameCustom.cpp
diff --git a/Source/bindings/v8/custom/V8JavaScriptCallFrameCustom.cpp b/Source/bindings/v8/custom/V8JavaScriptCallFrameCustom.cpp
index 0f1a60ad709c3960b7ad0a259d763b0cff910aff..e062efaef2300d1944dd7b37f992a6c18320f17d 100644
--- a/Source/bindings/v8/custom/V8JavaScriptCallFrameCustom.cpp
+++ b/Source/bindings/v8/custom/V8JavaScriptCallFrameCustom.cpp
@@ -37,8 +37,8 @@ namespace WebCore {
 
 void V8JavaScriptCallFrame::evaluateMethodCustom(const v8::FunctionCallbackInfo<v8::Value>& info)
 {
+    V8TRYCATCH_FOR_V8STRINGRESOURCE_VOID(V8StringResource<WithUndefinedOrNullCheck>, expression, info[0]);

[Inactive, 2013/12/12 15:59:26]

For e.g. I understand that this is related to the inspector. However, these are
bindings and this macro/behavior is what would be generated by our bindings
generator script if this wasn't custom code.

[pfeldman, 2013/12/12 16:11:24]

This is only called from within InjectedScriptSource, should be always string as
well.

     JavaScriptCallFrame* impl = V8JavaScriptCallFrame::toNative(info.Holder());
-    String expression = toCoreStringWithUndefinedOrNullCheck(info[0]);
     v8SetReturnValue(info, impl->evaluate(expression));
 }