| Index: chrome/browser/extensions/platform_app_browsertest.cc
|
| diff --git a/chrome/browser/extensions/platform_app_browsertest.cc b/chrome/browser/extensions/platform_app_browsertest.cc
|
| index 748090eeaa1d4010c4eda7276173c880269da896..fbb99a44ca0be5b8b99035652bfc7b2f9e8c9112 100644
|
| --- a/chrome/browser/extensions/platform_app_browsertest.cc
|
| +++ b/chrome/browser/extensions/platform_app_browsertest.cc
|
| @@ -150,7 +150,7 @@ IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, ExtensionWindowingApis) {
|
|
|
| // 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 =
|
| @@ -164,18 +164,17 @@ IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, ExtensionWindowingApis) {
|
| 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.
|
|
|