Get rid of static TabContentsView::Create function since the interface is in content, but the imp...

content/browser/site_instance_unittest.cc

 // Copyright (c) 2011 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 "base/compiler_specific.h"
 #include "base/stl_util.h"
 #include "base/string16.h"
 #include "chrome/test/base/testing_browser_process_test.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/browser/browser_thread.h"
@@ skipping 22 matching lines @@
                               const GURL& url) const OVERRIDE {
     return HasWebUIScheme(url);
   }
   virtual bool HasWebUIScheme(const GURL& url) const OVERRIDE {
     return url.SchemeIs(chrome::kChromeUIScheme);
   }
 };
 
 class SiteInstanceTestBrowserClient : public content::MockContentBrowserClient {
  public:
+  SiteInstanceTestBrowserClient() : old_browser_client_(NULL) {
+  }
+
+  virtual TabContentsView* CreateTabContentsView(TabContents* tab_contents) {
+    return old_browser_client_->CreateTabContentsView(tab_contents);
+  }
+
   virtual content::WebUIFactory* GetWebUIFactory() OVERRIDE {
     return &factory_;
   }
 
   virtual bool ShouldUseProcessPerSite(content::BrowserContext* browser_context,
                                        const GURL& effective_url) OVERRIDE {
     return false;
   }
 
   virtual bool IsURLSameAsAnySiteInstance(const GURL& url) OVERRIDE {
     return url == GURL(kSameAsAnyInstanceURL) ||
            url == GURL(chrome::kAboutCrashURL);
   }
 
   virtual GURL GetEffectiveURL(content::BrowserContext* browser_context,
                                const GURL& url) OVERRIDE {
     return url;
   }
 
+  void SetOriginalClient(content::ContentBrowserClient* old_browser_client) {
+    old_browser_client_ = old_browser_client;
+  }
+
  private:
   SiteInstanceTestWebUIFactory factory_;
+  content::ContentBrowserClient* old_browser_client_;
 };
 
 class SiteInstanceTest : public TestingBrowserProcessTest {
  public:
   SiteInstanceTest()
       : ui_thread_(BrowserThread::UI, &message_loop_),
         old_browser_client_(NULL) {
   }
 
   virtual void SetUp() {
     old_browser_client_ = content::GetContentClient()->browser();
+    browser_client_.SetOriginalClient(old_browser_client_);
     content::GetContentClient()->set_browser(&browser_client_);
   }
 
   virtual void TearDown() {
     content::GetContentClient()->set_browser(old_browser_client_);
   }
 
  private:
   MessageLoopForUI message_loop_;
   BrowserThread ui_thread_;
@@ skipping 429 matching lines @@
   // Make sure none of differing privilege processes are mixed.
   EXPECT_NE(extension1_instance->GetProcess(), webui1_instance->GetProcess());
 
   for (size_t i = 0; i < content::kMaxRendererProcessCount; ++i) {
     EXPECT_NE(extension1_instance->GetProcess(), hosts[i]);
     EXPECT_NE(webui1_instance->GetProcess(), hosts[i]);
   }
 
   STLDeleteContainerPointers(hosts.begin(), hosts.end());
 }