[V8] Factor out exception related methods of V8Proxy

• Component: WebCore JavaScript
• Status: RESOLVED FIXED

[Kentaro Hara]

To remove V8Proxy, we can factor out exception related methods of V8Proxy to a separate file.

[Kentaro Hara]

Created attachment 157919 [details]
Patch

[WebKit Review Bot]

Comment on attachment 157919 [details]
Patch

Attachment 157919 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/13475983

New failing tests:
http/tests/security/document-all.html

[WebKit Review Bot]

Created attachment 157924 [details]
Archive of layout-test-results from gce-cr-linux-06

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: gce-cr-linux-06  Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'>  Platform: Linux-2.6.39-gcg-201203291735-x86_64-with-Ubuntu-10.04-lucid

[Kentaro Hara]

Created attachment 157937 [details]
Patch

[Adam Barth]

Comment on attachment 157937 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=157937&action=review

> Source/WebCore/bindings/v8/V8ThrowException.cpp:34
> +static v8::Handle<v8::Value> DOMExceptionStackGetter(v8::Local<v8::String> name, const v8::AccessorInfo& info)

DOMExceptionStackGetter -> domExceptionStackGetter

> Source/WebCore/bindings/v8/V8ThrowException.cpp:40
> +static void DOMExceptionStackSetter(v8::Local<v8::String> name, v8::Local<v8::Value> value, const v8::AccessorInfo& info)

DOMExceptionStackSetter -> domExceptionStackSetter

> Source/WebCore/bindings/v8/V8ThrowException.cpp:58
> +        const char* message = 0;
> +        return V8ThrowException::throwTypeError(message, isolate);

I would have combined these lines.

> Source/WebCore/bindings/v8/V8ThrowException.h:32
> +// The types of javascript errors that can be thrown.

This comment seems useless and probably should be removed.

[Kentaro Hara]

Committed r125495: <http://trac.webkit.org/changeset/125495>

[Kentaro Hara]

(In reply to comment #5)
> > Source/WebCore/bindings/v8/V8ThrowException.cpp:34
> > +static v8::Handle<v8::Value> DOMExceptionStackGetter(v8::Local<v8::String> name, const v8::AccessorInfo& info)
> 
> DOMExceptionStackGetter -> domExceptionStackGetter
> 
> > Source/WebCore/bindings/v8/V8ThrowException.cpp:40
> > +static void DOMExceptionStackSetter(v8::Local<v8::String> name, v8::Local<v8::Value> value, const v8::AccessorInfo& info)
> 
> DOMExceptionStackSetter -> domExceptionStackSetter
> 
> > Source/WebCore/bindings/v8/V8ThrowException.cpp:58
> > +        const char* message = 0;
> > +        return V8ThrowException::throwTypeError(message, isolate);
> 
> I would have combined these lines.
> 
> > Source/WebCore/bindings/v8/V8ThrowException.h:32
> > +// The types of javascript errors that can be thrown.
> 
> This comment seems useless and probably should be removed.

Fixed. Thank you very much for a bunch of reviews!