Test extension reloading behavior.

content/browser/site_instance_impl.h

Index: content/browser/site_instance_impl.h
diff --git a/content/browser/site_instance_impl.h b/content/browser/site_instance_impl.h
index fda0d3743a3c7ad382a53e3e8d1e5dae5885fc7e..732facdc49056429a9391e5a57447ea22fd900dc 100644
--- a/content/browser/site_instance_impl.h
+++ b/content/browser/site_instance_impl.h
@@ -26,6 +26,8 @@ class CONTENT_EXPORT SiteInstanceImpl : public SiteInstance,
   virtual SiteInstance* GetRelatedSiteInstance(const GURL& url) OVERRIDE;
   virtual bool IsRelatedSiteInstance(const SiteInstance* instance) OVERRIDE;
   virtual BrowserContext* GetBrowserContext() const OVERRIDE;
+  virtual void set_render_process_host_factory(
+      const RenderProcessHostFactory* rph_factory) OVERRIDE;
 
   // Set the web site that this SiteInstance is rendering pages for.
   // This includes the scheme and registered domain, but not the port.  If the
@@ -44,15 +46,6 @@ class CONTENT_EXPORT SiteInstanceImpl : public SiteInstance,
   // navigating to the URL.
   bool HasWrongProcessForURL(const GURL& url);
 
-  // Sets the factory used to create new RenderProcessHosts. This will also be
-  // passed on to SiteInstances spawned by this one.

[Jeffrey Yasskin, 2013/04/04 09:34:35]

This wasn't actually true, but I've made it true with the change to
SiteInstanceImpl::GetRelatedSiteInstance() in this CL.

-  // 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;
-  }
-
  protected:
   friend class BrowsingInstance;
   friend class SiteInstance;