[ABANDONED] Add exception handling options to V8StringResource<T>::prepare.

Source/bindings/v8/V8BindingMacros.h

 /*
  * Copyright (C) 2010 Google Inc. 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.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 62 matching lines @@
     {                                                                  \
         v8::TryCatch block;                                            \
         var = (value);                                                 \
         if (UNLIKELY(block.HasCaught()))                               \
             exceptionState.rethrowV8Exception(block.Exception());      \
         if (UNLIKELY(exceptionState.throwIfNeeded()))                  \
             return;                                                    \
     }
 
 #define TONATIVE_DEFAULT_EXCEPTIONSTATE(type, var, value, exceptionState, retVal) \
-    type var;                                                                 \
-    {                                                                         \
-        v8::TryCatch block;                                                   \
-        var = (value);                                                        \
-        if (UNLIKELY(block.HasCaught()))                                      \
-            exceptionState.rethrowV8Exception(block.Exception());             \
-        if (UNLIKELY(exceptionState.throwIfNeeded()))                         \
-            return retVal;                                                    \
+    type var;                                                                     \
+    {                                                                             \
+        v8::TryCatch block;                                                       \
+        var = (value);                                                            \
+        if (UNLIKELY(block.HasCaught()))                                          \
+            exceptionState.rethrowV8Exception(block.Exception());                 \
+        if (UNLIKELY(exceptionState.throwIfNeeded()))                             \
+            return retVal;                                                        \
     }
 
 // type is an instance of class template V8StringResource<>,
 // but Mode argument varies; using type (not Mode) for consistency
 // with other macros and ease of code generation
-#define TOSTRING_VOID(type, var, value) \
-    type var(value);                    \
-    if (UNLIKELY(!var.prepare()))       \
-        return;
+#define TOSTRING_VOID(type, var, value)          \
+    type var(value);                             \
+    if (UNLIKELY(!var.setStringAsPrimitive())) { \

[haraken, 2014/04/21 03:57:02]

Shall we rename as follows?

setStringAsPrimitive => toStringFast
setStringWithToString => toString

[yhirano, 2014/04/21 04:08:06]

Done.

+        v8::TryCatch block;                      \
+        var.setStringWithToString();             \
+        if (UNLIKELY(block.HasCaught())) {       \
+            block.ReThrow();                     \
+            return;                              \
+        }                                        \
+    }
 
 #define TOSTRING_DEFAULT(type, var, value, retVal) \
     type var(value);                               \
-    if (UNLIKELY(!var.prepare()))                  \
-        return retVal;
+    if (UNLIKELY(!var.setStringAsPrimitive())) {   \
+        v8::TryCatch block;                        \
+        var.setStringWithToString();               \
+        if (UNLIKELY(block.HasCaught())) {         \
+            block.ReThrow();                       \
+            return retVal;                         \
+        }                                          \
+    }
 
 } // namespace WebCore
 
 #endif // V8BindingMacros_h