Remove WebContents::GetRoutingID().

content/public/browser/web_contents.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 245 matching lines @@
   // breadth-first traversal order.
   virtual std::vector<RenderFrameHost*> GetAllFrames() = 0;
 
   // Sends the given IPC to all live frames in this WebContents and returns the
   // number of sent messages (i.e. the number of processed frames).
   virtual int SendToAllFrames(IPC::Message* message) = 0;
 
   // Gets the current RenderViewHost for this tab.
   virtual RenderViewHost* GetRenderViewHost() const = 0;
 
-  // Gets the current RenderViewHost's routing id. Returns
-  // MSG_ROUTING_NONE when there is no RenderViewHost.
-  virtual int GetRoutingID() const = 0;

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

Interestingly, the old code handled the case where the RenderViewHost was null,
but I don't think that's actually possible, and regardless, both old and new
code would crash on GetRoutingID() if the RenderViewHost was ever null.

-
   // Returns the currently active RenderWidgetHostView. This may change over
   // time and can be nullptr (during setup and teardown).
   virtual RenderWidgetHostView* GetRenderWidgetHostView() const = 0;
 
   // Returns the outermost RenderWidgetHostView. This will return the platform
   // specific RenderWidgetHostView (as opposed to
   // RenderWidgetHostViewChildFrame), which can be used to create context
   // menus.
   virtual RenderWidgetHostView* GetTopLevelRenderWidgetHostView() = 0;
 
@@ skipping 489 matching lines @@
 
  private:
   // This interface should only be implemented inside content.
   friend class WebContentsImpl;
   WebContents() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_