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

Unified Diff: chrome/browser/task_management/task_manager_browsertest.cc

Issue 2197483003: Move the Mac Task Manager to the new backend code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: mark Created 4 years, 4 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/task_management/task_manager_browsertest.cc
diff --git a/chrome/browser/task_management/task_manager_browsertest.cc b/chrome/browser/task_management/task_manager_browsertest.cc
index 1a130ea1cd5d73e112be30a0802ff675e25b3f79..00c731862c38a1b7949569854080b7fb3b19eea2 100644
--- a/chrome/browser/task_management/task_manager_browsertest.cc
+++ b/chrome/browser/task_management/task_manager_browsertest.cc
@@ -98,10 +98,6 @@ class TaskManagerBrowserTest : public ExtensionBrowserTest {
base::RunLoop().RunUntilIdle(); // OnWindowClosed happens asynchronously.
}
- void Refresh() {
- task_management::TaskManagerTester::MaybeRefreshLegacyInstance();
- }
-
GURL GetTestURL() {
return ui_test_utils::GetTestUrl(
base::FilePath(base::FilePath::kCurrentDirectory),
@@ -692,7 +688,6 @@ IN_PROC_BROWSER_TEST_F(TaskManagerBrowserTest, NoticeHostedAppTabChanges) {
ASSERT_TRUE(LoadExtension(
test_data_dir_.AppendASCII("api_test").AppendASCII("app_process")));
// Force the TaskManager to query the title.
- Refresh();
ASSERT_NO_FATAL_FAILURE(WaitForTaskManagerRows(2, MatchAnyTab()));
ASSERT_NO_FATAL_FAILURE(WaitForTaskManagerRows(1, MatchAboutBlankTab()));
ASSERT_NO_FATAL_FAILURE(WaitForTaskManagerRows(1, MatchTab("Unmodified")));
@@ -1186,12 +1181,6 @@ IN_PROC_BROWSER_TEST_P(TaskManagerOOPIFBrowserTest,
}
IN_PROC_BROWSER_TEST_P(TaskManagerOOPIFBrowserTest, OrderingOfDependentRows) {
-#if defined(OS_MACOSX)
- // The ordering under test here is not implemented in the legacy task manager.
- if (!task_management::TaskManagerInterface::IsNewTaskManagerEnabled())
- return;
-#endif
-
ShowTaskManager();
GURL a_with_frames(embedded_test_server()->GetURL(

Powered by Google App Engine
This is Rietveld 408576698