Replace BrowserProces::AddRefModule/RemoveModule by ScopedKeepAlive

chrome/browser/lifetime/keep_alive_registry.h

Index: chrome/browser/lifetime/keep_alive_registry.h
diff --git a/chrome/browser/lifetime/keep_alive_registry.h b/chrome/browser/lifetime/keep_alive_registry.h
index cd0d4128b71adb002eaae3d026523131524b328e..e894af93fe79195b528034abe102c7f83520342c 100644
--- a/chrome/browser/lifetime/keep_alive_registry.h
+++ b/chrome/browser/lifetime/keep_alive_registry.h
@@ -20,11 +20,9 @@ class KeepAliveRegistry {
   static KeepAliveRegistry* GetInstance();
 
   // Methods to query the state of the registry.
-  // TODO(dgn): This currently does not give a complete picture. It has no
-  // information about the many places that rely on AddRefModule to keep the
-  // browser alive. Tracked by https://crbug.com/587926
   bool IsKeepingAlive() const;
   bool IsRestartAllowed() const;
+  bool IsOriginRegistered(KeepAliveOrigin origin) const;

[dgn, 2016/03/16 15:10:50]

Should I rename it xxForTesting? I suppose it could have some legitimate use in
non test code.

[sky, 2016/03/17 00:02:24]

I wouldn't bother.

 
   void AddObserver(KeepAliveStateObserver* observer);
   void RemoveObserver(KeepAliveStateObserver* observer);