Hide BrowsingInstance from all but SiteInstance, as intended by design.

content/browser/site_instance_unittest.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 "base/compiler_specific.h"
 #include "base/stl_util.h"
 #include "base/string16.h"
 #include "content/browser/browser_thread_impl.h"
 #include "content/browser/browsing_instance.h"
 #include "content/browser/child_process_security_policy.h"
@@ skipping 115 matching lines @@
   // Overrides BrowsingInstance::ShouldUseProcessPerSite so that we can test
   // both alternatives without using command-line switches.
   bool ShouldUseProcessPerSite(const GURL& url) {
     return use_process_per_site_;
   }
 
   void set_use_process_per_site(bool use_process_per_site) {
     use_process_per_site_ = use_process_per_site;
   }
 
+  // Make a few methods public for tests.
+  using BrowsingInstance::ShouldUseProcessPerSite;
+  using BrowsingInstance::browser_context;
+  using BrowsingInstance::HasSiteInstance;
+  using BrowsingInstance::GetSiteInstanceForURL;
+  using BrowsingInstance::RegisterSiteInstance;
+  using BrowsingInstance::UnregisterSiteInstance;
+
  private:
   virtual ~TestBrowsingInstance() {
     (*delete_counter_)++;
   }
 
   // Set by individual tests.
   bool use_process_per_site_;
 
   int* delete_counter_;
 };
@@ skipping 428 matching lines @@
   host.reset(instance->GetProcess());
   EXPECT_TRUE(host.get() != NULL);
   EXPECT_TRUE(instance->HasProcess());
 
   EXPECT_FALSE(instance->HasWrongProcessForURL(GURL("http://evernote.com")));
   EXPECT_FALSE(instance->HasWrongProcessForURL(
       GURL("javascript:alert(document.location.href);")));
 
   EXPECT_TRUE(instance->HasWrongProcessForURL(GURL("chrome://settings")));
 }