Track all extension frames in ProcessManager, inspect extensionoptions

extensions/browser/process_manager_observer.h

Index: extensions/browser/process_manager_observer.h
diff --git a/extensions/browser/process_manager_observer.h b/extensions/browser/process_manager_observer.h
index 668b3df6cc2402c47188448a38a76d49a1fdd372..47883ef62e7096c91616af95437e56e67cf18180 100644
--- a/extensions/browser/process_manager_observer.h
+++ b/extensions/browser/process_manager_observer.h
@@ -30,13 +30,21 @@ class ProcessManagerObserver {
   // Called immediately after the extension background host is destroyed.
   virtual void OnBackgroundHostClose(const std::string& extension_id) {}
 
+  // Called when a RenderFrameHost has been registered in an extension process.
   virtual void OnExtensionFrameRegistered(
       const std::string& extension_id,
       content::RenderFrameHost* render_frame_host) {}
 
+  // Called when a RenderFrameHost is no longer part of an extension process.
   virtual void OnExtensionFrameUnregistered(
       const std::string& extension_id,
       content::RenderFrameHost* render_frame_host) {}
+
+  // Called when a RenderFrameHost was navigated to another page within the
+  // extension process.
+  virtual void OnExtensionFrameNavigated(

[Devlin, 2015/12/01 21:40:49]

Am I just missing it?  Where is this used?

[robwu, 2015/12/01 23:19:58]

This will be used by https://codereview.chromium.org/1413543005/
 in chrome/browser/extensions/api/messaging/extension_message_port.cc. I want to
do all port lifetime management in the browser process, to make sure that ports
get cleaned up as soon as possible, regardless of renderer crashes etc.

+      const std::string& extension_id,
+      content::RenderFrameHost* render_frame_host) {}
 };
 
 }  // namespace extensions