Make LoadMonitoringExtensionHostQueue remove itself as an ExtensionHost observer at the correct tim…

chrome/browser/devtools/devtools_sanity_browsertest.cc

 // Copyright (c) 2012 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.
 
 #include "base/bind.h"
 #include "base/cancelable_callback.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "base/path_service.h"
@@ skipping 332 matching lines @@
     return WaitForExtensionViewsToLoad();
   }
 
   bool WaitForExtensionViewsToLoad() {
     // Wait for all the extension render views that exist to finish loading.
     // NOTE: This assumes that the extension views list is not changing while
     // this method is running.
 
     content::NotificationRegistrar registrar;
     registrar.Add(this,
-                  extensions::NOTIFICATION_EXTENSION_HOST_DID_STOP_LOADING,
+                  extensions::NOTIFICATION_EXTENSION_HOST_DID_STOP_FIRST_LOAD,
                   content::NotificationService::AllSources());
     base::CancelableClosure timeout(
         base::Bind(&TimeoutCallback, "Extension host load timed out."));
     base::MessageLoop::current()->PostDelayedTask(
         FROM_HERE, timeout.callback(), TestTimeouts::action_timeout());
 
     extensions::ProcessManager* manager =
         extensions::ProcessManager::Get(browser()->profile());
     extensions::ProcessManager::ViewSet all_views = manager->GetAllViews();
     for (extensions::ProcessManager::ViewSet::const_iterator iter =
              all_views.begin();
          iter != all_views.end();) {
       if (!(*iter)->IsLoading())
         ++iter;
       else
         content::RunMessageLoop();
     }
 
     timeout.Cancel();
     return true;
   }
 
   void Observe(int type,
                const content::NotificationSource& source,
                const content::NotificationDetails& details) override {
     switch (type) {
       case extensions::NOTIFICATION_EXTENSION_LOADED_DEPRECATED:
-      case extensions::NOTIFICATION_EXTENSION_HOST_DID_STOP_LOADING:
+      case extensions::NOTIFICATION_EXTENSION_HOST_DID_STOP_FIRST_LOAD:
         base::MessageLoopForUI::current()->Quit();
         break;
       default:
         NOTREACHED();
         break;
     }
   }
 
   base::FilePath test_extensions_dir_;
 };
@@ skipping 569 matching lines @@
   browser()->profile()->GetPrefs()->SetBoolean(prefs::kDevToolsDisabled, true);
   ui_test_utils::NavigateToURL(browser(), GURL("about:blank"));
   content::WebContents* web_contents =
       browser()->tab_strip_model()->GetWebContentsAt(0);
   scoped_refptr<content::DevToolsAgentHost> agent(
       content::DevToolsAgentHost::GetOrCreateFor(web_contents));
   DevToolsWindow::OpenDevToolsWindow(web_contents);
   DevToolsWindow* window = DevToolsWindow::FindDevToolsWindow(agent.get());
   ASSERT_FALSE(window);
 }