Introduce abstractions to allow embedders to test RenderViewHost

content/browser/tab_contents/test_tab_contents.h

Index: content/browser/tab_contents/test_tab_contents.h
diff --git a/content/browser/tab_contents/test_tab_contents.h b/content/browser/tab_contents/test_tab_contents.h
index 9842038b89f3f383b2c8b3e97b68237ef6f50d29..74096274233cad4bf84716d61bdd110c99b0fecd 100644
--- a/content/browser/tab_contents/test_tab_contents.h
+++ b/content/browser/tab_contents/test_tab_contents.h
@@ -13,18 +13,23 @@
 class SiteInstanceImpl;
 
 namespace content {
+class RenderViewHost;
 class TestRenderViewHost;
 }
 
-// Subclass TabContents to ensure it creates TestRenderViewHosts and does
-// not do anything involving views.
+// Subclass TabContents to ensure it creates TestRenderViewHosts
+// and does not do anything involving views.
 class TestTabContents : public TabContents {
  public:
   TestTabContents(content::BrowserContext* browser_context,
                   content::SiteInstance* instance);
   virtual ~TestTabContents();
 
-  content::TestRenderViewHost* pending_rvh() const;
+  content::RenderViewHost* pending_rvh() const;
+
+  // TODO(joi): Make sure this one is hidden once TestTabContents
+  // hides internal types from embedders.
+  content::TestRenderViewHost* pending_test_rvh() const;
 
   // State accessor.
   bool cross_navigation_pending() {