Add IPC::Sender and GetRoutingID() to WebContents for convenience and safety.

content/public/browser/web_contents.h

Index: content/public/browser/web_contents.h
diff --git a/content/public/browser/web_contents.h b/content/public/browser/web_contents.h
index c6f55f59b1b7b899feca2007186f235a17867578..d8f00c9474d458c6343f2be8e1c9539a87617507 100644
--- a/content/public/browser/web_contents.h
+++ b/content/public/browser/web_contents.h
@@ -14,6 +14,7 @@
 #include "content/public/browser/page_navigator.h"
 #include "content/public/browser/save_page_type.h"
 #include "content/public/browser/web_ui.h"
+#include "ipc/ipc_sender.h"
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/gfx/native_widget_types.h"
 #include "webkit/glue/window_open_disposition.h"
@@ -45,7 +46,7 @@ class WebContentsView;
 struct RendererPreferences;
 
 // Describes what goes in the main content area of a tab.
-class WebContents : public PageNavigator {
+class WebContents : public PageNavigator, IPC::Sender {

[jam, 2012/08/15 16:22:02]

nit: I think the convention is another public before IPC::Sender

[Jói, 2012/08/15 19:13:24]

Done.

  public:
   // |base_web_contents| is used if we want to size the new WebContents's view
   // based on the view of an existing WebContents.  This can be NULL if not
@@ -110,6 +111,9 @@ class WebContents : public PageNavigator {
   // Gets the current RenderViewHost for this tab.
   virtual RenderViewHost* GetRenderViewHost() const = 0;
 
+  // For convenience, returns MSG_ROUTING_NONE when there is no RenderViewHost.

[jam, 2012/08/15 16:22:02]

nit: to be extra clear (since there could be multiple RVHs), can you say

// Gets the current RenderViewHost's routing id. Returns MSG_ROUTING_NONE when
there is no RenderViewHost.

[Jói, 2012/08/15 19:13:24]

Done.

+  virtual int GetRoutingID() const = 0;
+
   // Returns the currently active RenderWidgetHostView. This may change over
   // time and can be NULL (during setup and teardown).
   virtual content::RenderWidgetHostView* GetRenderWidgetHostView() const = 0;