Remove WebContents::GetRoutingID().

content/public/browser/web_contents_observer.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/public/browser/web_contents_observer.h"
 
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/public/browser/navigation_details.h"
 #include "content/public/browser/render_view_host.h"
 
@@ skipping 42 matching lines @@
 
 bool WebContentsObserver::Send(IPC::Message* message) {
   if (!web_contents_) {
     delete message;
     return false;
   }
 
   return web_contents_->Send(message);
 }
 
 int WebContentsObserver::routing_id() const {

[alexmos, 2016/11/22 01:29:06]

Interesting.  This looks like another bad API that we might want to remove in a
follow-up CL, or at least audit its uses.  At the very least I think it should
be named something like render_view_routing_id().

   if (!web_contents_)
     return MSG_ROUTING_NONE;
 
-  return web_contents_->GetRoutingID();
+  return web_contents_->GetRenderViewHost()->GetRoutingID();
 }
 
 void WebContentsObserver::ResetWebContents() {
   web_contents_->RemoveObserver(this);
   web_contents_ = nullptr;
 }
 
 }  // namespace content