[Webkit-unassigned] [Bug 73176] Add 'isProtocolHandlerRegistered' and 'unregisterProtocolHandler'.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Mar 15 10:52:19 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=73176
--- Comment #21 from Adam Barth <abarth at webkit.org> 2012-03-15 10:52:16 PST ---
(From update of attachment 121962)
View in context: https://bugs.webkit.org/attachment.cgi?id=121962&action=review
> Source/WebCore/page/Chrome.cpp:374
> +String Chrome::isProtocolHandlerRegistered(const String& scheme, const String& baseURL, const String& url)
> +{
> + return m_client->isProtocolHandlerRegistered(scheme, baseURL, url);
> +}
> +
> +void Chrome::unregisterProtocolHandler(const String& scheme, const String& baseURL, const String& url)
> +{
> + m_client->unregisterProtocolHandler(scheme, baseURL, url);
> +}
This functions are unnecessary. The ChromeClient is visible publicly. You can just call methods on it directly. (Feel free to remove Chrome::registerProtocolHandler as well.)
> Source/WebCore/page/Navigator.cpp:289
> +String Navigator::isProtocolHandlerRegistered(const String& scheme, const String& url, ExceptionCode& ec)
I thought we moved Navigator::registerProtocolHandler out of Navigator.cpp.
> Source/WebCore/page/Navigator.cpp:292
> + return "declined";
This should be a constant so you don't need to repeat it four times.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list