Include App pages in chrome://extensions "active views" list.

This changes the implementation such that the extensions_ui now iterates over all the RVHs in the process, rather than maintain an explicit list of ExtensionFunctionDispatchers (live app pages don't have EFDs).

TEST=none
BUG=42605

Review URL: http://codereview.chromium.org/1827002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46295 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 10 insertions, 1 deletions

diff --git a/chrome/browser/extensions/extensions_ui.h b/chrome/browser/extensions/extensions_ui.h
index b7c05ce..449c8da 100644
--- a/chrome/browser/extensions/extensions_ui.h
+++ b/chrome/browser/extensions/extensions_ui.h
@@ -23,6 +23,7 @@ class Extension;
 class ExtensionsService;
 class FilePath;
 class PrefService;
+class RenderProcessHost;
 class UserScript;
 class Value;
 
@@ -185,7 +186,8 @@ class ExtensionsDOMHandler
 
   // Helper that lists the current active html pages for an extension.
   std::vector<ExtensionPage> GetActivePagesForExtension(
-      const std::string& extension_id);
+      RenderProcessHost* process,
+      Extension* extension);
 
   // Returns the best icon to display in the UI for an extension, or an empty
   // ExtensionResource if no good icon exists.
@@ -233,6 +235,13 @@ class ExtensionsDOMHandler
   // notification.
   bool ignore_notifications_;
 
+  // The page may be refreshed in response to a RENDER_VIEW_HOST_DELETED,
+  // but the iteration over RenderViewHosts will include the host because the
+  // notification is sent when it is in the process of being deleted (and before
+  // it is removed from the process). Keep a pointer to it so we can exclude
+  // it from the active views.
+  RenderViewHost* deleting_rvh_;
+
   DISALLOW_COPY_AND_ASSIGN(ExtensionsDOMHandler);
 };