Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(435)

Unified Diff: public/web/WebViewClient.h

Issue 232263003: Remove old version navigator content utils APIs (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 -----------------------------------------------------------
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698