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

Unified Diff: apps/load_and_launch_browsertest.cc

Issue 2386313003: Re-enable a load-and-launch browser test for win & linux (Closed)
Patch Set: Created 4 years, 2 months 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: apps/load_and_launch_browsertest.cc
diff --git a/apps/load_and_launch_browsertest.cc b/apps/load_and_launch_browsertest.cc
index 330a4f442ae19c97b54aeaaa9071a365922e8467..7ebe7ca5645e31aa453312eb87a9de51e0b1c10e 100644
--- a/apps/load_and_launch_browsertest.cc
+++ b/apps/load_and_launch_browsertest.cc
@@ -171,20 +171,9 @@ class LoadAndLaunchExtensionBrowserTest : public PlatformAppBrowserTest {
} // namespace
-
-// TODO(jackhou): Make this test not flaky on Vista or Linux Aura. See
-// http://crbug.com/176897
msw 2016/10/03 21:16:52 nit: mention BUG=176897 (and close that bug if thi
proberge 2016/10/04 13:31:54 Done.
-#if defined(OS_WIN) || (defined(OS_LINUX) && defined(USE_AURA))
-#define MAYBE_LoadAndLaunchAppChromeNotRunning \
- DISABLED_LoadAndLaunchAppChromeNotRunning
-#else
-#define MAYBE_LoadAndLaunchAppChromeNotRunning \
- LoadAndLaunchAppChromeNotRunning
-#endif
-
// Case where Chrome is not running.
IN_PROC_BROWSER_TEST_F(LoadAndLaunchPlatformAppBrowserTest,
- MAYBE_LoadAndLaunchAppChromeNotRunning) {
+ LoadAndLaunchAppChromeNotRunning) {
LoadAndLaunchApp();
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698