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

content/browser/site_instance.h

 // 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.
 
 #ifndef CONTENT_BROWSER_RENDERER_HOST_SITE_INSTANCE_H_
 #define CONTENT_BROWSER_RENDERER_HOST_SITE_INSTANCE_H_
 #pragma once
 
 #include "content/browser/renderer_host/render_process_host_impl.h"
 #include "content/common/content_export.h"
@@ skipping 41 matching lines @@
 // a SiteInstance is only live as long as it is accessible, either from new
 // tabs with no NavigationEntries or in NavigationEntries in the history.
 //
 ///////////////////////////////////////////////////////////////////////////////
 class CONTENT_EXPORT SiteInstance : public base::RefCounted<SiteInstance>,
                                     public content::NotificationObserver {
  public:
   // Returns a unique ID for this SiteInstance.
   int32 id() { return id_; }
 
-  // Get the BrowsingInstance to which this SiteInstance belongs.
-  BrowsingInstance* browsing_instance() { return browsing_instance_; }
-
   // Sets the factory used to create new RenderProcessHosts. This will also be
   // passed on to SiteInstances spawned by this one.
   //
   // The factory must outlive the SiteInstance; ownership is not transferred. It
   // may be NULL, in which case the default BrowserRenderProcessHost will be
   // created (this is the behavior if you don't call this function).
   void set_render_process_host_factory(
       content::RenderProcessHostFactory* rph_factory) {
     render_process_host_factory_ = rph_factory;
   }
@@ skipping 28 matching lines @@
   // class from the rest of the codebase.)
   // TODO(creis): This may be an argument to build a pass_refptr<T> class, as
   // Darin suggests.
   SiteInstance* GetRelatedSiteInstance(const GURL& url);
 
   // Returns whether this SiteInstance has a process that is the wrong type for
   // the given URL.  If so, the browser should force a process swap when
   // navigating to the URL.
   bool HasWrongProcessForURL(const GURL& url) const;
 
+  // Browser context to which this SiteInstance (and all related
+  // SiteInstances) belongs.
+  content::BrowserContext* GetBrowserContext() const;
+
   // Factory method to create a new SiteInstance.  This will create a new
   // new BrowsingInstance, so it should only be used when creating a new tab
   // from scratch (or similar circumstances).  Callers should ensure that
   // this SiteInstance becomes ref counted, by storing it in a scoped_refptr.
   //
   // The render process host factory may be NULL. See SiteInstance constructor.
   //
   // TODO(creis): This may be an argument to build a pass_refptr<T> class, as
   // Darin suggests.
   static SiteInstance* CreateSiteInstance(
@@ skipping 65 matching lines @@
   // will still remain the same even if the process crashes, since in that
   // scenario the RenderProcessHost remains the same.
   content::RenderProcessHost* process_;
 
   // The web site that this SiteInstance is rendering pages for.
   GURL site_;
 
   // Whether SetSite has been called.
   bool has_site_;
 
+  FRIEND_TEST_ALL_PREFIXES(RenderViewHostManagerTest, NewTabPageProcesses);
+
   DISALLOW_COPY_AND_ASSIGN(SiteInstance);
 };
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_SITE_INSTANCE_H_