Bug 153667

Summary: Move more 'constructor' properties to the prototype
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: BindingsAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: barraclough, cgarcia, commit-queue, darin, ggaren, rniwa, sam, webkit-bug-importer
Priority: P2 Keywords: InRadar, WebExposed
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
URL: http://heycam.github.io/webidl/#interface-prototype-object
See Also: https://bugs.webkit.org/show_bug.cgi?id=148092
https://bugs.webkit.org/show_bug.cgi?id=149412
Attachments:
Description Flags
WIP Patch
none
Patch none

Chris Dumez
Reported 2016-01-29 13:30:05 PST
Move more 'constructor' properties to the prototype. Currently, we keep the 'constructor' on the instance for interfaces that have an indexed / named property getter because our getOwnPropertySlot() implementation used to be wrong for such interfaces. However, getOwnPropertySlot() should be correct after <https://bugs.webkit.org/show_bug.cgi?id=148092> so we should be able to move the 'constructor' up to the prototype now.
Attachments
WIP Patch (32.50 KB, patch)
2016-01-29 13:31 PST, Chris Dumez
no flags
Patch (36.74 KB, patch)
2016-01-29 15:53 PST, Chris Dumez
no flags
Chris Dumez
Comment 1 2016-01-29 13:31:15 PST
Created attachment 270248 [details] WIP Patch
Chris Dumez
Comment 2 2016-01-29 15:53:08 PST
Radar WebKit Bug Importer
Comment 3 2016-01-29 15:56:46 PST
Radar WebKit Bug Importer
Comment 4 2016-01-29 15:57:05 PST
Chris Dumez
Comment 5 2016-01-30 12:10:25 PST
Comment on attachment 270268 [details] Patch Clearing flags on attachment: 270268 Committed r195904: <http://trac.webkit.org/changeset/195904>
Chris Dumez
Comment 6 2016-01-30 12:10:31 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.