Bug 111693

Summary: Custom Elements: should support non-HTML namespaces.
Product: WebKit Reporter: Hajime Morrita <morrita>
Component: DOMAssignee: Web Components Team <webcomponents-bugzilla>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, ericbidelman, esprehn+autocc, haraken, japhet, ojan.autocc, webcomponents-bugzilla, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 99688    
Attachments:
Description Flags
Patch
none
Patch none

Description Hajime Morrita 2013-03-07 00:23:36 PST
Currently it only supports SVG. It should also support non-HTML namespaces like SVG.
Comment 1 Hajime Morrita 2013-03-26 22:24:57 PDT
Created attachment 195224 [details]
Patch
Comment 2 Dimitri Glazkov (Google) 2013-03-27 09:28:48 PDT
Comment on attachment 195224 [details]
Patch

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

> Source/WebCore/bindings/v8/CustomElementHelpers.h:59
> +    static bool isValidPrototypeParameter(const ScriptValue&, ScriptState*, AtomicString* namespaceURI = 0);

Can we not pass by ref here? The ptr thing looks a bit weird.
Comment 3 Hajime Morrita 2013-03-27 19:00:15 PDT
Created attachment 195456 [details]
Patch
Comment 4 Hajime Morrita 2013-03-27 19:05:01 PDT
(In reply to comment #2)
> > Source/WebCore/bindings/v8/CustomElementHelpers.h:59
> > +    static bool isValidPrototypeParameter(const ScriptValue&, ScriptState*, AtomicString* namespaceURI = 0);
> 
> Can we not pass by ref here? The ptr thing looks a bit weird.

Right, fixed.
Comment 5 WebKit Review Bot 2013-03-28 07:32:36 PDT
Comment on attachment 195456 [details]
Patch

Clearing flags on attachment: 195456

Committed r147113: <http://trac.webkit.org/changeset/147113>
Comment 6 WebKit Review Bot 2013-03-28 07:32:40 PDT
All reviewed patches have been landed.  Closing bug.