Remove old version navigator content utils APIs

public/web/WebViewClient.h

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 311 matching lines @@
     virtual void zoomLevelChanged() { }
 
     // Navigator Content Utils  --------------------------------------------
 
     // Registers a new URL handler for the given protocol.
     virtual void registerProtocolHandler(const WebString& scheme,
         const WebURL& baseUrl,
         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 this function. I already
changed it in r246326 of content module. If there is no dependency issue between
blink and content, I think we can drop the old APIs, because new APIs were added
to blink and content module 2 months ago, isn't it ?

     }
 
     // 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 -----------------------------------------------------------
 
     // Returns the current visibility of the WebView.
     virtual WebPageVisibilityState visibilityState() const
     {
         return WebPageVisibilityStateVisible;
     }
 
@@ skipping 27 matching lines @@
     // Informs the browser that the draggable regions have been updated.
     virtual void draggableRegionsChanged() { }
 
 protected:
     ~WebViewClient() { }
 };
 
 } // namespace blink
 
 #endif