Test extension reloading behavior.

chrome/browser/extensions/extension_process_manager.h

 // 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_PROCESS_MANAGER_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_PROCESS_MANAGER_H_
 
 #include <map>
 #include <set>
 #include <string>
 
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/weak_ptr.h"
 #include "base/time.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "extensions/common/view_type.h"
 
 class Browser;
 class GURL;
 class Profile;
 
 namespace content {
+class RenderProcessHostFactory;
 class RenderViewHost;
 class SiteInstance;
 };
 
 namespace extensions {
 class Extension;
 class ExtensionHost;
 }
 
 // Manages dynamic state of running Chromium extensions. There is one instance
@@ skipping 96 matching lines @@
 
   // Tracks network requests for a given RenderViewHost, used to know
   // when network activity is idle for lazy background pages.
   void OnNetworkRequestStarted(content::RenderViewHost* render_view_host);
   void OnNetworkRequestDone(content::RenderViewHost* render_view_host);
 
   // Prevents |extension|'s background page from being closed and sends the
   // onSuspendCanceled() event to it.
   void CancelSuspend(const extensions::Extension* extension);
 
+  // Retrieves the base SiteInstance that is used to create the SiteInstances
+  // for all extensions in this profile.
+  content::SiteInstance* GetSiteInstanceForTest() {

[Paweł Hajdan Jr., 2013/04/08 20:03:51]

nit: I don't think this needs to be inline. Please move the implementation to
.cc file (or convert to hacker_style for a simple accessor).

[Jeffrey Yasskin, 2013/04/08 22:09:38]

You're saying that if I call it "site_instance_for_test()", it can stay in the
header?

[Paweł Hajdan Jr., 2013/04/08 22:17:38]

Yes, that would be fine for me.

[Jeffrey Yasskin, 2013/04/08 22:38:02]

Done.

+    return site_instance_.get();
+  }
+
  protected:
   explicit ExtensionProcessManager(Profile* profile);
 
   // Called just after |host| is created so it can be registered in our lists.
   void OnExtensionHostCreated(extensions::ExtensionHost* host,
                               bool is_background);
 
   // Called on browser shutdown to close our extension hosts.
   void CloseBackgroundHosts();
 
@@ skipping 69 matching lines @@
   // The time to delay between sending a ShouldSuspend message and
   // sending a Suspend message; read from command-line switch.
   base::TimeDelta event_page_suspending_time_;
 
   base::WeakPtrFactory<ExtensionProcessManager> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionProcessManager);
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_PROCESS_MANAGER_H_