Remove EPM:all_hosts_ and use all_extension_views_ instead.

chrome/browser/extensions/platform_app_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/command_line.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/extensions/extension_browsertest.h"
 #include "chrome/browser/extensions/extension_host.h"
@@ skipping 68 matching lines @@
     // Now we have a new platform app running.
     EXPECT_EQ(platform_app_count + 1, GetPlatformAppCount());
   }
 
   // Gets the number of platform apps that are running.
   size_t GetPlatformAppCount() {
     int count = 0;
     ExtensionProcessManager* process_manager =
         browser()->profile()->GetExtensionProcessManager();
     ExtensionProcessManager::const_iterator iter;
-    for (iter = process_manager->begin(); iter != process_manager->end();
+    ExtensionProcessManager::ExtensionHostSet platform_app_hosts =
+        process_manager->platform_app_hosts();
+    for (iter = platform_app_hosts.begin(); iter != platform_app_hosts.end();
          ++iter) {
-      ExtensionHost* host = *iter;
-      if (host->extension() && host->extension()->is_platform_app())
+      if ((*iter)->extension())
         count++;
     }
 
     return count;
   }
 
   // Gets the WebContents associated with the ExtensionHost of the first
   // platform app that is found (most tests only deal with one platform
   // app, so this is good enough).
   WebContents* GetFirstPlatformAppWebContents() {
     ExtensionProcessManager* process_manager =
         browser()->profile()->GetExtensionProcessManager();
     ExtensionProcessManager::const_iterator iter;
-    for (iter = process_manager->begin(); iter != process_manager->end();
+    ExtensionProcessManager::ExtensionHostSet platform_app_hosts =
+        process_manager->platform_app_hosts();
+    for (iter = platform_app_hosts.begin(); iter != platform_app_hosts.end();
          ++iter) {
-      ExtensionHost* host = *iter;
-      if (host->extension() && host->extension()->is_platform_app())
-        return host->host_contents();
+      return (*iter)->host_contents();
     }
 
     return NULL;
   }
 };
 
 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, OpenAppInShellContainer) {
   ASSERT_EQ(0u, GetPlatformAppCount());
   LoadAndLaunchPlatformApp("empty");
   ASSERT_EQ(1u, GetPlatformAppCount());
@@ skipping 68 matching lines @@
 #if defined(USE_AURA)
 // On Aura, this currently fails because the window width is returned as 256
 // instead of 250. See http://crbug.com/119410.
 #define MAYBE_WindowsApi FAILS_WindowsApi
 #else
 #define MAYBE_WindowsApi WindowsApi
 #endif
 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, MAYBE_WindowsApi) {
   ASSERT_TRUE(RunPlatformAppTest("platform_apps/windows_api")) << message_;
 }