Remove chrome.windows.* support for platform apps.

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 "chrome/browser/automation/automation_util.h"
 #include "chrome/browser/tab_contents/render_view_context_menu.h"
 #include "chrome/browser/extensions/extension_test_message_listener.h"
 #include "chrome/browser/extensions/platform_app_browsertest_util.h"
 #include "chrome/browser/extensions/shell_window_registry.h"
 #include "chrome/browser/ui/browser.h"
@@ skipping 132 matching lines @@
   // extensions API.
   const Extension* extension = LoadExtension(
       test_data_dir_.AppendASCII("common/background_page"));
   ASSERT_EQ(1U, RunGetWindowsFunctionForExtension(extension));
 
   // And no shell windows.
   ASSERT_EQ(0U, GetShellWindowCount());
 
   // Launch a platform app that shows a window.
   ExtensionTestMessageListener launched_listener("Launched", false);
-  const Extension* platform_app = LoadAndLaunchPlatformApp("minimal");
+  LoadAndLaunchPlatformApp("minimal");
   ASSERT_TRUE(launched_listener.WaitUntilSatisfied());
   ASSERT_EQ(1U, GetShellWindowCount());
   ShellWindowRegistry::ShellWindowSet shell_windows =
       ShellWindowRegistry::Get(browser()->profile())->shell_windows();
   int shell_window_id = (*shell_windows.begin())->session_id().id();
 
   // But it's not visible to the extensions API, it still thinks there's just
   // one browser window.
   ASSERT_EQ(1U, RunGetWindowsFunctionForExtension(extension));
   // It can't look it up by ID either
   ASSERT_FALSE(RunGetWindowFunctionForExtension(shell_window_id, extension));
 
   // The app can also only see one window (its own).
-  ASSERT_EQ(1U, RunGetWindowsFunctionForExtension(platform_app));
-  ASSERT_TRUE(RunGetWindowFunctionForExtension(shell_window_id, platform_app));
+  // TODO(jeremya): add an extension function to get a shell window by ID, and
+  // to get a list of all the shell windows, so we can test this.
 
   // Launch another platform app that also shows a window.
   ExtensionTestMessageListener launched_listener2("Launched", false);
-  const Extension* platform_app2 = LoadAndLaunchPlatformApp("context_menu");
+  LoadAndLaunchPlatformApp("context_menu");
   ASSERT_TRUE(launched_listener2.WaitUntilSatisfied());
 
   // There are two total shell windows, but each app can only see its own.
   ASSERT_EQ(2U, GetShellWindowCount());
-  ASSERT_EQ(1U, RunGetWindowsFunctionForExtension(platform_app));
-  ASSERT_EQ(1U, RunGetWindowsFunctionForExtension(platform_app2));
+  // TODO(jeremya): as above, this requires more extension functions.
 }
 
 // TODO(benwells): fix these tests for ChromeOS.
 #if !defined(OS_CHROMEOS)
 // Tests that command line parameters get passed through to platform apps
 // via launchData correctly when launching with a file.
 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, LaunchWithFile) {
   SetCommandLineArg( "platform_apps/launch_files/test.txt");
   ASSERT_TRUE(RunPlatformAppTest("platform_apps/launch_file"))
       << message_;
@@ skipping 46 matching lines @@
 }
 
 // Tests that no launch data is sent through if there are no arguments passed
 // on the command line
 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, LaunchWithNothing) {
   ClearCommandLineArgs();
   ASSERT_TRUE(RunPlatformAppTest("platform_apps/launch_nothing"))
       << message_;
 }
 #endif  // defined(OS_CHROMEOS)