Bug 174238

Summary: [WebIDL] Move plugin object customization into the generator
Product: WebKit Reporter: Sam Weinig <sam>
Component: New BugsAssignee: Sam Weinig <sam>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, cdumez, commit-queue, darin
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch none

Description Sam Weinig 2017-07-06 20:47:43 PDT 
[WebIDL] Move plugin object customization into the generator
Comment 1 Sam Weinig 2017-07-06 20:48:22 PDT Comment hidden (obsolete)
Comment 2 Sam Weinig 2017-07-06 22:17:22 PDT Comment hidden (obsolete)
Comment 3 Sam Weinig 2017-07-07 06:43:41 PDT Comment hidden (obsolete)
Comment 4 Sam Weinig 2017-07-07 13:21:07 PDT 
Created attachment 314869 [details]
Patch
Comment 5 Chris Dumez 2017-07-09 10:17:06 PDT 
Comment on attachment 314869 [details]
Patch

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

> Source/WebCore/bindings/js/JSCSSStyleDeclarationCustom.cpp:358
> +bool JSCSSStyleDeclaration::put(JSCell* cell, ExecState* state, PropertyName propertyName, JSValue value, PutPropertySlot& putPropertySlot)

This is more custom code now.

> Source/WebCore/bindings/js/JSLocationCustom.cpp:126
> +bool JSLocation::put(JSCell* cell, ExecState* state, PropertyName propertyName, JSValue value, PutPropertySlot& putPropertySlot)

This is more custom code now.
Comment 6 Sam Weinig 2017-07-09 18:34:56 PDT 
(In reply to Chris Dumez from comment #5)
> Comment on attachment 314869 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=314869&action=review
> 
> > Source/WebCore/bindings/js/JSCSSStyleDeclarationCustom.cpp:358
> > +bool JSCSSStyleDeclaration::put(JSCell* cell, ExecState* state, PropertyName propertyName, JSValue value, PutPropertySlot& putPropertySlot)
> 
> This is more custom code now.
> 
> > Source/WebCore/bindings/js/JSLocationCustom.cpp:126
> > +bool JSLocation::put(JSCell* cell, ExecState* state, PropertyName propertyName, JSValue value, PutPropertySlot& putPropertySlot)
> 
> This is more custom code now.

It is a tiny bit more code, but it allows us to get rid of having two ways to override put, which is confusing and error prone. In reality, Location is custom enough that having the generator do things it doesn't expect is probably worse.
Comment 7 WebKit Commit Bot 2017-07-10 11:27:30 PDT 
Comment on attachment 314869 [details]
Patch

Clearing flags on attachment: 314869

Committed r219302: <http://trac.webkit.org/changeset/219302>
Comment 8 WebKit Commit Bot 2017-07-10 11:27:31 PDT 
All reviewed patches have been landed.  Closing bug.