Remove EPM:all_hosts_ and use all_extension_views_ instead.

chrome/browser/automation/automation_provider_observers.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 "chrome/browser/automation/automation_provider_observers.h"
 
 #include <deque>
 #include <string>
 #include <vector>
 
@@ skipping 489 matching lines @@
   if (automation_) {
     AutomationMsg_WaitForTabCountToBecome::WriteReplyParams(
         reply_message_.get(), true);
     automation_->Send(reply_message_.release());
   }
 
   delete this;
 }
 
 bool DidExtensionHostsStopLoading(ExtensionProcessManager* manager) {
-  for (ExtensionProcessManager::const_iterator iter = manager->begin();
-       iter != manager->end(); ++iter) {
+  for (ExtensionProcessManager::const_iterator iter =
+           manager->background_hosts().begin();

[benwells, 2012/04/20 03:55:14]

This logic has changed, its now only background hosts. aa: is this intentional /
correct?

[Aaron Boodman, 2012/04/20 06:34:56]

It depends on what the scripts that call this automation code are using it for.

To be conservative, let's just wait until all the RVH in EPM stop.

[benwells, 2012/04/23 05:00:26]

Done.

+       iter != manager->background_hosts().end(); ++iter) {
     if (!(*iter)->did_stop_loading())
       return false;
   }
   return true;
 }
 
 ExtensionUninstallObserver::ExtensionUninstallObserver(
     AutomationProvider* automation,
     IPC::Message* reply_message,
     const std::string& id)
@@ skipping 2531 matching lines @@
   }
 
   ExtensionHost* host = content::Details<ExtensionHost>(details).ptr();
   if (host->extension_id() == extension_id_ &&
       host->extension_host_type() == chrome::VIEW_TYPE_EXTENSION_POPUP) {
     AutomationJSONReply(automation_, reply_message_.release())
         .SendSuccess(NULL);
     delete this;
   }
 }