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

Source/bindings/v8/V8Binding.h

 /*
 * Copyright (C) 2009 Google Inc. All rights reserved.
 * Copyright (C) 2012 Ericsson AB. All rights reserved.
 *
 * Redistribution and use in source and binary forms, with or without
 * modification, are permitted provided that the following conditions are
 * met:
 *
 *     * Redistributions of source code must retain the above copyright
 * notice, this list of conditions and the following disclaimer.
@@ skipping 373 matching lines @@
     WrapperWorldType worldTypeInMainThread(v8::Isolate*);
 
     DOMWrapperWorld* isolatedWorldForIsolate(v8::Isolate*);
 
     template<class T> struct NativeValueTraits;
 
     template<>
     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;
         }
     };
 
     template<>
     struct NativeValueTraits<unsigned> {
         static inline unsigned nativeValue(const v8::Handle<v8::Value>& value)
         {
             return toUInt32(value);
         }
     };
@@ skipping 228 matching lines @@
         return v8::FunctionTemplate::New(function, environment)->GetFunction();
     }
 
     v8::Local<v8::Value> getHiddenValueFromMainWorldWrapper(v8::Isolate*, ScriptWrappable*, v8::Handle<v8::String> key);
 
     v8::Isolate* getIsolateFromScriptExecutionContext(ScriptExecutionContext*);
 
 } // namespace WebCore
 
 #endif // V8Binding_h