Keep lazy background page alive while there are pending network requests or

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_
 #pragma once
 
 #include <map>
 #include <set>
@@ skipping 79 matching lines @@
   bool HasExtensionHost(ExtensionHost* host) const;
 
   // Getter and setter for the lazy background page's keepalive count. This is
   // the count of how many outstanding "things" are keeping the page alive.
   // When this reaches 0, we will begin the process of shutting down the page.
   // "Things" include pending events, resource loads, and API calls.
   int GetLazyKeepaliveCount(const Extension* extension);
   int IncrementLazyKeepaliveCount(const Extension* extension);
   int DecrementLazyKeepaliveCount(const Extension* extension);
 
-  // These are called when the extension transitions between idle and active.
-  // They control the process of closing the background page when idle.
-  void OnLazyBackgroundPageIdle(const std::string& extension_id);
-  void OnLazyBackgroundPageActive(const std::string& extension_id);
-
-  // Handle a response to the ShouldClose message, used for lazy background
+  // Handles a response to the ShouldClose message, used for lazy background
   // pages.
   void OnShouldCloseAck(const std::string& extension_id, int sequence_id);
 
+  // Tracks network requests for a given RenderViewHost, used to know
+  // when network activity is idle for lazy background pages.
+  void OnNetworkRequestStarted(RenderViewHost* render_view_host);
+  void OnNetworkRequestDone(RenderViewHost* render_view_host);
+
   typedef std::set<ExtensionHost*> ExtensionHostSet;
   typedef ExtensionHostSet::const_iterator const_iterator;
   const_iterator begin() const { return all_hosts_.begin(); }
   const_iterator end() const { return all_hosts_.end(); }
 
  protected:
   explicit ExtensionProcessManager(Profile* profile);
 
   // Called just after |host| is created so it can be registered in our lists.
   void OnExtensionHostCreated(ExtensionHost* host, bool is_background);
@@ skipping 18 matching lines @@
   // The set of running viewless background extensions.
   ExtensionHostSet background_hosts_;
 
   // A SiteInstance related to the SiteInstance for all extensions in
   // this profile.  We create it in such a way that a new
   // browsing instance is created.  This controls process grouping.
   scoped_refptr<content::SiteInstance> site_instance_;
 
  private:
   // Contains all extension-related RenderViewHost instances for all extensions.
-  typedef std::set<RenderViewHost*> RenderViewHostSet;
-  RenderViewHostSet all_extension_views_;
+  // We also keep a cache of the host's view type, because that information
+  // is not accessible at registration/deregistration time.
+  typedef std::map<RenderViewHost*, content::ViewType> ExtensionRenderViews;

[Yoyo Zhou, 2012/03/02 03:10:46]

I think this typename should have "Map".

[Matt Perry, 2012/03/02 20:25:21]

I disagree. I don't like putting container types in the type names, because if
you decide to change the container type, you have to rename the typedef (like I
had to do with this CL).

[Yoyo Zhou, 2012/03/02 20:31:58]

I don't think that's a significant cost. As someone reading the code, I
shouldn't have to find the typedef to understand what kind of type it really is.

Although no type name is better than a wrong one (I think ExtensionSet had
something that claimed to be a set but was actually a map, e.g.).

+  ExtensionRenderViews all_extension_views_;
 
   // Close the given |host| iff it's a background page.
   void CloseBackgroundHost(ExtensionHost* host);
 
-  // Excludes background page.
-  bool HasVisibleViews(const std::string& extension_id);
+  // These are called when the extension transitions between idle and active.
+  // They control the process of closing the background page when idle.
+  void OnLazyBackgroundPageIdle(const std::string& extension_id);
+  void OnLazyBackgroundPageActive(const std::string& extension_id);
+
+  // Updates a potentially-registered RenderViewHost once it has been
+  // associated with a WebContents. This allows us to gather information that
+  // was not available when the host was first registered.
+  void UpdateRegisteredRenderView(RenderViewHost* render_view_host);
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionProcessManager);
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_PROCESS_MANAGER_H_