Test extension reloading behavior.

chrome/browser/extensions/extension_process_manager.h

Index: chrome/browser/extensions/extension_process_manager.h
diff --git a/chrome/browser/extensions/extension_process_manager.h b/chrome/browser/extensions/extension_process_manager.h
index d0c747cbb49eab5ca7824eec257f71d1a930ed70..4bc46c7d89999d9c8bd6a63070c5d62f79e7a4f6 100644
--- a/chrome/browser/extensions/extension_process_manager.h
+++ b/chrome/browser/extensions/extension_process_manager.h
@@ -22,6 +22,7 @@ class GURL;
 class Profile;
 
 namespace content {
+class RenderProcessHostFactory;
 class RenderViewHost;
 class SiteInstance;
 };
@@ -138,6 +139,13 @@ class ExtensionProcessManager : public content::NotificationObserver {
   // onSuspendCanceled() event to it.
   void CancelSuspend(const extensions::Extension* extension);
 
+  // Installs |rph_factory| into the SiteInstance shared by all extensions in
+  // this profile.  This must be called before any ExtensionHosts are created
+  // (to avoid having multiple kinds of RenderProcessHosts), and |rph_factory|
+  // must outlive the profile.
+  void SetRenderProcessHostFactoryForTest(

[Paweł Hajdan Jr., 2013/04/04 21:30:05]

Why not just use SiteInstance directly? More layers of indirection make the code
harder to follow.

[Jeffrey Yasskin, 2013/04/05 13:14:01]

i.e. GetSiteInstanceForTest()->SetRenderProcessHostFactory()? ok.

+      content::RenderProcessHostFactory* rph_factory);
+
  protected:
   explicit ExtensionProcessManager(Profile* profile);