Remove EPM:all_hosts_ and use all_extension_views_ instead.

chrome/browser/automation/automation_provider_observers.cc

Index: chrome/browser/automation/automation_provider_observers.cc
diff --git a/chrome/browser/automation/automation_provider_observers.cc b/chrome/browser/automation/automation_provider_observers.cc
index b49a726257fab96218aea318b5e7554409f45644..9d5564be30d412a744f324097c2a48270f87d8ca 100644
--- a/chrome/browser/automation/automation_provider_observers.cc
+++ b/chrome/browser/automation/automation_provider_observers.cc
@@ -507,8 +507,9 @@ void TabCountChangeObserver::CheckTabCount() {
 }
 
 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;
   }