Introduce unregisterProtocolHandler() and isProtocolHandlerRegistered() webview API

Introduce unregisterProtocolHandler() and isProtocolHandlerRegistered() webview API

As a step to support the custom scheme handler in experimental features, this cl introduce unregisterProtocolHandler() and isProtocolHandlerRegistered(). The content module will be able to deregister existing registered protocol and check if a protocol is registered.

BUG=N/A

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=165276

[gyuyoung-inactive, 2013-06-29 07:00:12]

Work in progress.

[abarth-chromium, 2013-06-29 18:01:20]

Would you be willing to send an "intent to ship" email for this change?

http://dev.chromium.org/blink#launch-process

[gyuyoung-inactive, 2013-06-30 02:47:38]

On 2013/06/29 18:01:20, abarth wrote:
> Would you be willing to send an "intent to ship" email for this change?
> 
> http://dev.chromium.org/blink#launch-process

Ok, I will do that when I'm ready to get review. :)

[gyuyoung-inactive, 2013-07-06 05:42:56]

On 2013/06/30 02:47:38, gyuyoung wrote:
> On 2013/06/29 18:01:20, abarth wrote:
> > Would you be willing to send an "intent to ship" email for this change?
> > 
> > http://dev.chromium.org/blink#launch-process
> 
> Ok, I will do that when I'm ready to get review. :)

I will update this patch after finishing my priority work first.

[gyuyoung-inactive, 2014-01-10 04:01:27]

To support the custom scheme handler feature, I upload a patch in order to add
virtual functions which can be override by content layer. Could you take a look
?

[gyuyoung-inactive, 2014-01-16 03:50:12]

On 2014/01/10 04:01:27, gyuyoung wrote:
> To support the custom scheme handler feature, I upload a patch in order to add
> virtual functions which can be override by content layer. Could you take a
look
> ?

Abarth, could you take a look this patch ?

[abarth-chromium, 2014-01-16 04:46:44]

lgtm

https://codereview.chromium.org/18179009/diff/48001/public/web/WebViewClient.h
File public/web/WebViewClient.h (right):

https://codereview.chromium.org/18179009/diff/48001/public/web/WebViewClient....
public/web/WebViewClient.h:362: virtual void unregisterProtocolHandler(const
WebString& scheme, const WebString& baseUrl, const WebString& url) { }
Why do these use WebString instead of WebURL?

[gyuyoung-inactive, 2014-01-16 05:12:10]

https://codereview.chromium.org/18179009/diff/48001/public/web/WebViewClient.h
File public/web/WebViewClient.h (right):

https://codereview.chromium.org/18179009/diff/48001/public/web/WebViewClient....
public/web/WebViewClient.h:362: virtual void unregisterProtocolHandler(const
WebString& scheme, const WebString& baseUrl, const WebString& url) { }
On 2014/01/16 04:46:45, abarth wrote:
> Why do these use WebString instead of WebURL?

Because registerProtocolHandler() already has used *WebString* for them. IMHO,
it would be good if we change them for registerProtocolHandler(),
unregisterProtocolHandler(), and isProtocolHandlerRegistered() at a time.

[gyuyoung-inactive, 2014-01-16 05:17:41]

https://codereview.chromium.org/18179009/diff/48001/public/web/WebViewClient.h
File public/web/WebViewClient.h (right):

https://codereview.chromium.org/18179009/diff/48001/public/web/WebViewClient....
public/web/WebViewClient.h:362: virtual void unregisterProtocolHandler(const
WebString& scheme, const WebString& baseUrl, const WebString& url) { }
On 2014/01/16 05:12:10, gyuyoung wrote:
> On 2014/01/16 04:46:45, abarth wrote:
> > Why do these use WebString instead of WebURL?
> 
> Because registerProtocolHandler() already has used *WebString* for them. IMHO,
> it would be good if we change them for registerProtocolHandler(),
> unregisterProtocolHandler(), and isProtocolHandlerRegistered() at a time.

If there is no objection, I will upload a patch for it after landing this cl.

[commit-bot: I haz the power, 2014-01-16 05:52:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gyuyoung.kim@samsung.com/18179009/48001

[commit-bot: I haz the power, 2014-01-16 06:37:36]

Retried try job too often on linux_blink for step(s) wtf_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_blin...

[commit-bot: I haz the power, 2014-01-16 07:41:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gyuyoung.kim@samsung.com/18179009/228001

[commit-bot: I haz the power, 2014-01-16 08:32:42]

Retried try job too often on linux_blink for step(s) wtf_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_blin...

[commit-bot: I haz the power, 2014-01-17 05:41:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gyuyoung.kim@samsung.com/18179009/228001

[commit-bot: I haz the power, 2014-01-17 05:56:57]

Change committed as 165276