Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6345)

Unified Diff: chrome/browser/extensions/api/app_window/app_window_apitest.cc

Issue 11117011: Keep browser process alive while there are platform apps with background pages running. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: And another test Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/app_window/app_window_apitest.cc
diff --git a/chrome/browser/extensions/api/app_window/app_window_apitest.cc b/chrome/browser/extensions/api/app_window/app_window_apitest.cc
index 5307888e5a18ad7ee38169d85a1205315a5e293c..d514e4dd9ae973b850d6ec39926432bec2be7ef7 100644
--- a/chrome/browser/extensions/api/app_window/app_window_apitest.cc
+++ b/chrome/browser/extensions/api/app_window/app_window_apitest.cc
@@ -35,6 +35,8 @@ IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, WindowsApiBounds) {
ready_listener.Reply(base::IntToString(slop));
ASSERT_TRUE(success_listener.WaitUntilSatisfied());
+
+ CloseShellWindowsAndWaitForAppToExit();
}
// TODO(asargent) - Fix onMinimzed event on OSX (crbug.com/162793) and figure
@@ -44,7 +46,7 @@ IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, WindowsApiBounds) {
IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, WindowsApiProperties) {
EXPECT_TRUE(
- RunExtensionTest("platform_apps/windows_api_properties")) << message_;
+ RunPlatformAppTest("platform_apps/windows_api_properties")) << message_;
}
#endif // defined(TOOLKIT_VIEWS)

Powered by Google App Engine
This is Rietveld 408576698