Bug 72998

Summary: WebIDL: Add support for static for JSC and V8
Product: WebKit Reporter: Erik Arvidsson <arv>
Component: DOMAssignee: Erik Arvidsson <arv>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, code.vineet, dominicc, japhet, kaustubh.ra, ojan, sam, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
URL: http://www.w3.org/TR/WebIDL/#idl-static-operations
Bug Depends on: 73342    
Bug Blocks: 71968, 73365    
Attachments:
Description Flags
Patch
none
Patch abarth: review+

Description Erik Arvidsson 2011-11-22 18:19:38 PST 
http://www.w3.org/TR/WebIDL/#idl-static-operations

We need to add support for static to be able to correctly implement the URL object.

Our implementation of the URL object is flawed. window.webkitURL returns an instance of DOMURL when it should return a URL constructor.

http://www.w3.org/TR/FileAPI/#URL-object
Comment 1 Erik Arvidsson 2011-11-28 12:02:24 PST 
It turns out that CodeGeneratorV8.pm already supports this using [ClassMethod]

I plan to change the code generator to use "static" instead but a as a first step I am going to make CodeGeneratorJS support [ClassMethod] too.
Comment 2 Erik Arvidsson 2011-11-29 14:38:26 PST 
Created attachment 117047 [details]
Patch
Comment 3 Erik Arvidsson 2011-11-29 14:48:01 PST 
Created attachment 117048 [details]
Patch
Comment 4 Erik Arvidsson 2011-11-29 16:40:41 PST 
webkit-patch land failed

Commited as

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