Title: [125497] trunk/Source/WebCore
Revision
125497
Author
[email protected]
Date
2012-08-13 19:10:43 -0700 (Mon, 13 Aug 2012)

Log Message

Unreviewed. Simple refactoring that I forgot to apply in r125495.

* bindings/v8/V8ThrowException.cpp:
(WebCore::domExceptionStackGetter):
(WebCore::domExceptionStackSetter):
(WebCore::V8ThrowException::setDOMException):
* bindings/v8/V8ThrowException.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (125496 => 125497)


--- trunk/Source/WebCore/ChangeLog	2012-08-14 02:09:14 UTC (rev 125496)
+++ trunk/Source/WebCore/ChangeLog	2012-08-14 02:10:43 UTC (rev 125497)
@@ -1,3 +1,13 @@
+2012-08-13  Kentaro Hara  <[email protected]>
+
+        Unreviewed. Simple refactoring that I forgot to apply in r125495.
+
+        * bindings/v8/V8ThrowException.cpp:
+        (WebCore::domExceptionStackGetter):
+        (WebCore::domExceptionStackSetter):
+        (WebCore::V8ThrowException::setDOMException):
+        * bindings/v8/V8ThrowException.h:
+
 2012-08-13  Ryuan Choi  <[email protected]>
 
         [CMAKE][EFL] Fix build break with --tiled-backing-store

Modified: trunk/Source/WebCore/bindings/v8/V8ThrowException.cpp (125496 => 125497)


--- trunk/Source/WebCore/bindings/v8/V8ThrowException.cpp	2012-08-14 02:09:14 UTC (rev 125496)
+++ trunk/Source/WebCore/bindings/v8/V8ThrowException.cpp	2012-08-14 02:10:43 UTC (rev 125497)
@@ -31,13 +31,13 @@
 
 namespace WebCore {
 
-static v8::Handle<v8::Value> DOMExceptionStackGetter(v8::Local<v8::String> name, const v8::AccessorInfo& info)
+static v8::Handle<v8::Value> domExceptionStackGetter(v8::Local<v8::String> name, const v8::AccessorInfo& info)
 {
     ASSERT(info.Data()->IsObject());
     return info.Data()->ToObject()->Get(v8String("stack", info.GetIsolate()));
 }
 
-static void DOMExceptionStackSetter(v8::Local<v8::String> name, v8::Local<v8::Value> value, const v8::AccessorInfo& info)
+static void domExceptionStackSetter(v8::Local<v8::String> name, v8::Local<v8::Value> value, const v8::AccessorInfo& info)
 {
     ASSERT(info.Data()->IsObject());
     info.Data()->ToObject()->Set(v8String("stack", info.GetIsolate()), value);
@@ -53,10 +53,8 @@
     if (ec <= 0 || v8::V8::IsExecutionTerminating())
         return v8Undefined();
 
-    if (ec == NATIVE_TYPE_ERR) {
-        const char* message = 0;
-        return V8ThrowException::throwTypeError(message, isolate);
-    }
+    if (ec == NATIVE_TYPE_ERR)
+        return V8ThrowException::throwTypeError(0, isolate);
 
     ExceptionCodeDescription description(ec);
 
@@ -72,7 +70,7 @@
     v8::Handle<v8::Value> error = v8::Exception::Error(v8String(description.description, isolate));
     ASSERT(!error.IsEmpty());
     ASSERT(exception->IsObject());
-    exception->ToObject()->SetAccessor(v8String("stack", isolate), DOMExceptionStackGetter, DOMExceptionStackSetter, error);
+    exception->ToObject()->SetAccessor(v8String("stack", isolate), domExceptionStackGetter, domExceptionStackSetter, error);
 
     return v8::ThrowException(exception);
 }

Modified: trunk/Source/WebCore/bindings/v8/V8ThrowException.h (125496 => 125497)


--- trunk/Source/WebCore/bindings/v8/V8ThrowException.h	2012-08-14 02:09:14 UTC (rev 125496)
+++ trunk/Source/WebCore/bindings/v8/V8ThrowException.h	2012-08-14 02:10:43 UTC (rev 125497)
@@ -29,7 +29,6 @@
 
 namespace WebCore {
 
-// The types of _javascript_ errors that can be thrown.
 enum ErrorType {
     RangeError,
     ReferenceError,
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to