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

Unified Diff: chrome/browser/ui/app_list/app_list_controller_browsertest.cc

Issue 2131463002: Purge the App Launcher code from Mac (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Zap mac-specific icon assets Created 4 years, 5 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
Index: chrome/browser/ui/app_list/app_list_controller_browsertest.cc
diff --git a/chrome/browser/ui/app_list/app_list_controller_browsertest.cc b/chrome/browser/ui/app_list/app_list_controller_browsertest.cc
index bb0eb859fee2891b3023211be6e301b525902b33..a604ba86c49dffaebd005db22d45ade843635eff 100644
--- a/chrome/browser/ui/app_list/app_list_controller_browsertest.cc
+++ b/chrome/browser/ui/app_list/app_list_controller_browsertest.cc
@@ -125,16 +125,9 @@ class AppListControllerSearchResultsBrowserTest
DISALLOW_COPY_AND_ASSIGN(AppListControllerSearchResultsBrowserTest);
};
-
-// Flaky on Mac. https://crbug.com/415264
-#if defined(OS_MACOSX)
-#define MAYBE_UninstallSearchResult DISABLED_UninstallSearchResult
-#else
-#define MAYBE_UninstallSearchResult UninstallSearchResult
-#endif
// Test showing search results, and uninstalling one of them while displayed.
IN_PROC_BROWSER_TEST_F(AppListControllerSearchResultsBrowserTest,
- MAYBE_UninstallSearchResult) {
+ UninstallSearchResult) {
base::FilePath test_extension_path;
ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &test_extension_path));
test_extension_path = test_extension_path.AppendASCII("extensions")

Powered by Google App Engine
This is Rietveld 408576698