[Webkit-unassigned] [Bug 73177] Add Custom Content Handler

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=73177





--- Comment #14 from Dongwoo Joshua Im (dwim) <dw.im at samsung.com>  2012-06-08 18:45:34 PST ---
(In reply to comment #12)
> (From update of attachment 146229 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=146229&action=review
> 
> > Source/JavaScriptCore/Configurations/FeatureDefines.xcconfig:133
> > 
> The registerProtocolHandler flag is "ENABLE_REGISTER_PROTOCOL_HANDLER" so I'd name this one "ENABLE_REGISTER_CONTENT_HANDLER".

ENABLE_SCHEME_HANDLER flag is also there.
I think ENABLE_SCHEME_HANDLER  and ENABLE_CONTENT_HANDLER are better.

I filed that bug already, https://bugs.webkit.org/show_bug.cgi?id=88614.

> 
> > Source/WebCore/page/NavigatorRegisterContentHandler.cpp:123
> > +    if (!verifyContentHandlerMimeType(mimeType, ec))
> 
> How about just calling iscontentBlacklisted directly here?

That could be.
I will try that.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.