Adds some debugging code in hopes of tracking down a

crash. Specifically:

. Adds a CHECK to IDMap that evaluates to true if adding NULL. The
  default isn't to do this check.
. browser_shutdown CHECKs if the # RPHs changes while iterating
  through the RPHs.

BUG=15615
TEST=none

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

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

1 files changed, 4 insertions, 0 deletions

diff --git a/chrome/browser/browser_shutdown.cc b/chrome/browser/browser_shutdown.cc
index 8ef0dbe..b67542d 100644
--- a/chrome/browser/browser_shutdown.cc
+++ b/chrome/browser/browser_shutdown.cc
@@ -65,6 +65,7 @@ void OnShutdownStarting(ShutdownType type) {
     // a no-op in some cases, so we still need to go through the normal
     // shutdown path for the ones that didn't exit here.
     shutdown_num_processes_slow_ = 0;
+    size_t start_rph_size = RenderProcessHost::size();
     for (RenderProcessHost::iterator hosts = RenderProcessHost::begin();
          hosts != RenderProcessHost::end();
          ++hosts) {
@@ -74,6 +75,9 @@ void OnShutdownStarting(ShutdownType type) {
         // higher up, it's not possible to get here. Confirm this and change
         // FastShutdownIfPossible to just be FastShutdown.
         shutdown_num_processes_slow_++;
+      // The number of RPHs should not have changed as the result of invoking
+      // FastShutdownIfPossible.
+      CHECK(start_rph_size == RenderProcessHost::size());
     }
   }
 }