Replace BrowserProces::AddRefModule/RemoveModule by ScopedKeepAlive

chrome/browser/lifetime/keep_alive_registry.h

 // Copyright 2016 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_LIFETIME_KEEP_ALIVE_REGISTRY_H_
 #define CHROME_BROWSER_LIFETIME_KEEP_ALIVE_REGISTRY_H_
 
 #include <map>
 
 #include "base/macros.h"
 #include "base/memory/singleton.h"
 #include "base/observer_list.h"
 
 enum class KeepAliveOrigin;
 enum class KeepAliveRestartOption;
 class KeepAliveStateObserver;
 
 class KeepAliveRegistry {
  public:
   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);
 
  private:
   friend struct base::DefaultSingletonTraits<KeepAliveRegistry>;
   // Friend to be able to use Register/Unregister
   friend class ScopedKeepAlive;
   friend std::ostream& operator<<(std::ostream& out,
                                   const KeepAliveRegistry& registry);
@@ skipping 22 matching lines @@
   base::ObserverList<KeepAliveStateObserver> observers_;
 
   DISALLOW_COPY_AND_ASSIGN(KeepAliveRegistry);
 };
 
 #ifndef NDEBUG
 std::ostream& operator<<(std::ostream& out, const KeepAliveRegistry& registry);
 #endif  // ndef NDEBUG
 
 #endif  // CHROME_BROWSER_LIFETIME_KEEP_ALIVE_REGISTRY_H_