Preparation for removing same origin check as blink checks for the same

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index 7cd6639666a7d3829a4e12e8a404986819ae02c1..59ec8c42352431e6dad02732aca4ff52588845af 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -3954,6 +3954,26 @@ void RenderViewImpl::unregisterProtocolHandler(const WebString& scheme,
                                                  user_gesture));
 }
 
+void RenderViewImpl::registerProtocolHandler(const WebString& scheme,

[jamesr, 2014/08/22 05:34:36]

annotate with a comment or something that the registerProtocolHandler versions
that take a base_url are going away, otherwise this looks like needlessly
duplicate code.  a good way would be to add a comment like this:

// TODO(your user name): Remove once blink patch lands. http://crbug.com/21345

to the functions you plan to delete soon

+                                             const WebURL& url,
+                                             const WebString& title) {
+  bool user_gesture = WebUserGestureIndicator::isProcessingUserGesture();
+  Send(new ViewHostMsg_RegisterProtocolHandler(routing_id_,
+                                               base::UTF16ToUTF8(scheme),
+                                               url,
+                                               title,
+                                               user_gesture));
+}
+
+void RenderViewImpl::unregisterProtocolHandler(const WebString& scheme,
+                                               const WebURL& url) {
+  bool user_gesture = WebUserGestureIndicator::isProcessingUserGesture();
+  Send(new ViewHostMsg_UnregisterProtocolHandler(routing_id_,
+                                                 base::UTF16ToUTF8(scheme),
+                                                 url,
+                                                 user_gesture));
+}
+
 blink::WebPageVisibilityState RenderViewImpl::visibilityState() const {
   blink::WebPageVisibilityState current_state = is_hidden() ?
       blink::WebPageVisibilityStateHidden :