Bug 32844 – [V8] Generate header declaration for property accessor getters/setters

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Nate Chapin]

Part of https://bugs.webkit.org/show_bug.cgi?id=32638, the quest to remove all the manually declared custom bindings from V8CustomBinding.h.

[Nate Chapin]

Created attachment 45359 [details]
Generate property accessor getter/setter declarations.

I'm on my way out the door for the holidays, so this review is decidedly non-urgent.

[WebKit Review Bot]

style-queue ran check-webkit-style on attachment 45359 [details] without any errors.

[Dimitri Glazkov (Google)]

Comment on attachment 45359 [details]
Generate property accessor getter/setter declarations.

r=me with notes:

Gah! Why is this custom for V8 bindings? Need a FIXME and a bug for this.

> +        readonly attribute [V8CustomGetter] DOMString appVersion;

Ditto.

> +                 attribute [Replaceable, V8CustomGetter] WorkerContext self;

This special-casing bonanza needs FIXMEs and bugs. We should work to remove these.

> +    if ($dataNode->name eq "Event") {
> +        push(@headerContent, "  static v8::Handle<v8::Value> dataTransferAccessorGetter(v8::Local<v8::String> name, const v8::AccessorInfo& info);\n");
> +        push(@headerContent, "  static void valueAccessorSetter(v8::Local<v8::String> name, v8::Local<v8::Value> value, const v8::AccessorInfo& info);\n");
> +    }
> +    if ($dataNode->name eq "Location") {
> +        push(@headerContent, "  static v8::Handle<v8::Value> assignAccessorGetter(v8::Local<v8::String> name, const v8::AccessorInfo& info);\n");
> +        push(@headerContent, "  static v8::Handle<v8::Value> reloadAccessorGetter(v8::Local<v8::String> name, const v8::AccessorInfo& info);\n");
> +        push(@headerContent, "  static v8::Handle<v8::Value> replaceAccessorGetter(v8::Local<v8::String> name, const v8::AccessorInfo& info);\n");
> +    }
>  }

[Nate Chapin]

Created attachment 45683 [details]
patch2 - removed .idl modifications

The .idl modifications were originally necessary because of some v8 only custom bindings that didn't really need to be custom.  Those bindings are now generated, which resulted in the removal of V8NavigatorCustom.cpp.

[WebKit Review Bot]

style-queue ran check-webkit-style on attachment 45683 [details] without any errors.

[Dimitri Glazkov (Google)]

Comment on attachment 45683 [details]
patch2 - removed .idl modifications

ROCK!

[Nate Chapin]

http://trac.webkit.org/changeset/52672