Chromium Code Reviews| Index: public/web/WebViewClient.h |
| diff --git a/public/web/WebViewClient.h b/public/web/WebViewClient.h |
| index 1a248f601844f07d00c46c4371ec141773179e22..ee70cf9d6b2d9a763a171c3b25f7310cfc662d75 100644 |
| --- a/public/web/WebViewClient.h |
| +++ b/public/web/WebViewClient.h |
| @@ -329,32 +329,19 @@ public: |
| const WebURL& url, |
| const WebString& title) |
| { |
| - registerProtocolHandler(scheme, baseUrl.string(), url.string(), title); |
| + registerProtocolHandler(scheme, baseUrl, url, title); |
|
jochen (gone - plz use gerrit)
2014/04/10 07:29:06
you can drop this call entirely, no?
gyuyoung-inactive
2014/04/10 07:46:52
As far as I know, the content module only override
|
| } |
| // Unregisters a given URL handler for the given protocol. |
| virtual void unregisterProtocolHandler(const WebString& scheme, const WebURL& baseUrl, const WebURL& url) |
| { |
| - unregisterProtocolHandler(scheme, baseUrl.string(), url.string()); |
| + unregisterProtocolHandler(scheme, baseUrl, url); |
| } |
| // Check if a given URL handler is registered for the given protocol. |
| virtual WebCustomHandlersState isProtocolHandlerRegistered(const WebString& scheme, const WebURL& baseUrl, const WebURL& url) |
| { |
| - return isProtocolHandlerRegistered(scheme, baseUrl.string(), url.string()); |
| - } |
| - |
| - // These old version APIs need to be removed after synching with chrome side. |
| - virtual void registerProtocolHandler(const WebString& scheme, |
| - const WebString& baseUrl, |
| - const WebString& url, |
| - const WebString& title) { } |
| - |
| - virtual void unregisterProtocolHandler(const WebString& scheme, const WebString& baseUrl, const WebString& url) { } |
| - |
| - virtual WebCustomHandlersState isProtocolHandlerRegistered(const WebString& scheme, const WebString& baseUrl, const WebString& url) |
| - { |
| - return WebCustomHandlersNew; |
| + return isProtocolHandlerRegistered(scheme, baseUrl, url; |
| } |
| // Visibility ----------------------------------------------------------- |