Change a WebString type of url parameter with WebURL

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 334 matching lines @@
     // Zoom ----------------------------------------------------------------
 
     // Informs the browser that the zoom levels for this frame have changed from
     // the default values.
     virtual void zoomLimitsChanged(double minimumLevel, double maximumLevel) { }
 
     // Informs the browser that the zoom level has changed as a result of an
     // action that wasn't initiated by the client.
     virtual void zoomLevelChanged() { }
 
+    // Navigaor Content Utils  ----------------------------------------------

[jochen (gone - plz use gerrit), 2014/01/21 09:29:19]

Navigator

+
     // Registers a new URL handler for the given protocol.
     virtual void registerProtocolHandler(const WebString& scheme,
-                                         const WebString& baseUrl,
-                                         const WebString& url,
-                                         const WebString& title) { }
+        const WebURL& baseUrl,
+        const WebURL& url,
+        const WebString& title)
+    {
+        registerProtocolHandler(scheme, baseUrl.string(), url.string(), title);

[gyuyoung-inactive, 2014/01/21 09:18:13]

Jochen, I change to let new version call old version. But, I'm not sure if this
intermediate implementation can be accepted.

+    }
 
     // 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());
+    }
+
+    // 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) { }
 
-    // Check if a given URL handler is registered for the given protocol.
     virtual WebCustomHandlersState isProtocolHandlerRegistered(const WebString& scheme, const WebString& baseUrl, const WebString& url)
     {
         return WebCustomHandlersNew;
     }
 
     // Visibility -----------------------------------------------------------
 
     // Returns the current visibility of the WebView.
     virtual WebPageVisibilityState visibilityState() const
     {
@@ skipping 27 matching lines @@
     // Informs the browser that the draggable regions have been updated.
     virtual void draggableRegionsChanged() { }
 
 protected:
     ~WebViewClient() { }
 };
 
 } // namespace blink
 
 #endif