Replace several uses of toWebCoreString() by V8TRYCATCH_FOR_V8STRINGRESOURCE() macro

Source/bindings/v8/V8Binding.h

Index: Source/bindings/v8/V8Binding.h
diff --git a/Source/bindings/v8/V8Binding.h b/Source/bindings/v8/V8Binding.h
index 723840b2b8b762e44993139900967594246186c5..1751f43c6d125d6d74b3df3781dfa071f6e75fff 100644
--- a/Source/bindings/v8/V8Binding.h
+++ b/Source/bindings/v8/V8Binding.h
@@ -391,7 +391,8 @@ namespace WebCore {
     struct NativeValueTraits<String> {
         static inline String nativeValue(const v8::Handle<v8::Value>& value)
         {
-            return toWebCoreString(value);
+            V8TRYCATCH_FOR_V8STRINGRESOURCE_RETURN(V8StringResource<>, stringValue, value, String());

[haraken, 2013/09/12 17:33:52]

This change wouldn't be helpful until you fix all caller sites of nativeValue()
so that the caller sites return immediately when nativeValue() throws
exceptions.

However, I'm afraid that it will mess up the code base. Do you have any good
idea?

[do-not-use, 2013/09/13 14:55:49]

The reason I used V8TRYCATCH_FOR_V8STRINGRESOURCE_RETURN() here even though it
does not change the behavior is because I was planning to have toWebCoreString()
take a v8::String instead of a v8::Value in a follow-up CL.

+            return stringValue;
         }
     };