Factor out the test web contents from the WebContents unit test so that it ca...

chrome/browser/tab_contents/site_instance.h

 // Copyright (c) 2006-2008 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 CHROME_BROWSER_TAB_CONTENTS_SITE_INSTANCE_H_
 #define CHROME_BROWSER_TAB_CONTENTS_SITE_INSTANCE_H_
 
 #include "chrome/browser/browsing_instance.h"
 #include "chrome/browser/renderer_host/render_process_host.h"
 #include "googleurl/src/gurl.h"
@@ skipping 33 matching lines @@
 //
 ///////////////////////////////////////////////////////////////////////////////
 class SiteInstance : public base::RefCounted<SiteInstance> {
  public:
   // Virtual to allow tests to extend it.
   virtual ~SiteInstance();
 
   // 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(RenderProcessHostFactory* rph_factory) {
+    render_process_host_factory_ = rph_factory;
+  }
+
   // Set / Get the host ID for this SiteInstance's current RenderProcessHost.
   void set_process_host_id(int process_host_id) {
     process_host_id_ = process_host_id;
   }
   int process_host_id() const { return process_host_id_; }
 
   // Update / Get the max page ID for this SiteInstance.
   void UpdateMaxPageID(int32 page_id) {
     if (page_id > max_page_id_)
       max_page_id_ = page_id;
@@ skipping 26 matching lines @@
   // a scoped_refptr.  (By having this method, we can hide the BrowsingInstance
   // 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);
 
   // 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(Profile* profile);
 
   // Returns the site for the given URL, which includes only the scheme and
   // registered domain.  Returns an empty GURL if the URL has no host.
   static GURL GetSiteForURL(const GURL& url);
 
   // Return whether both URLs are part of the same web site, for the purpose of
   // assigning them to processes accordingly.  The decision is currently based
   // on the registered domain of the URLs (google.com, bbc.co.uk), as well as
   // the scheme (https, http).  This ensures that two pages will be in
   // the same process if they can communicate with other via JavaScript.
   // (e.g., docs.google.com and mail.google.com have DOM access to each other
   // if they both set their document.domain properties to google.com.)
   static bool IsSameWebSite(const GURL& url1, const GURL& url2);
 
  protected:
   friend class BrowsingInstance;
 
   // Create a new SiteInstance.  Protected to give access to BrowsingInstance
   // and tests; most callers should use CreateSiteInstance or
   // GetRelatedSiteInstance instead.
   SiteInstance(BrowsingInstance* browsing_instance)
       : browsing_instance_(browsing_instance),
+        render_process_host_factory_(NULL),
         process_host_id_(-1),
         max_page_id_(-1),
         has_site_(false) {
     DCHECK(browsing_instance);
   }
 
  private:
   // BrowsingInstance to which this SiteInstance belongs.
   scoped_refptr<BrowsingInstance> browsing_instance_;
 
+  // Factory for new RenderProcessHosts, not owned by this class. NULL indiactes
+  // that the default BrowserRenderProcessHost should be created.
+  const RenderProcessHostFactory* render_process_host_factory_;
+
   // Current host ID for the RenderProcessHost that is rendering pages for this
   // SiteInstance.  If the rendering process dies, this host ID can be
   // replaced when a new process is created, without losing the association
   // between all pages in this SiteInstance.
   int process_host_id_;
 
   // The current max_page_id in the SiteInstance's RenderProcessHost.  If the
   // rendering process dies, its replacement should start issuing page IDs that
   // are larger than this value.
   int32 max_page_id_;
 
   // The web site that this SiteInstance is rendering pages for.
   GURL site_;
 
   // Whether SetSite has been called.
   bool has_site_;
 
   DISALLOW_EVIL_CONSTRUCTORS(SiteInstance);
 };
 
 #endif  // CHROME_BROWSER_TAB_CONTENTS_SITE_INSTANCE_H_