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

Unified Diff: chrome/browser/task_manager/task_manager_browsertest_util.h

Issue 1293813005: Change the new-task-manager flag so that the new implementation is enabled by default for all users. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixing old TM browser tests Created 5 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_manager/task_manager_browsertest_util.h
diff --git a/chrome/browser/task_manager/task_manager_browsertest_util.h b/chrome/browser/task_manager/task_manager_browsertest_util.h
index 334fa268596dc801bc2cc7b5c8de67f8a4a97e12..2050932eb300a7c505d7785bf074a1101ddb8b69 100644
--- a/chrome/browser/task_manager/task_manager_browsertest_util.h
+++ b/chrome/browser/task_manager/task_manager_browsertest_util.h
@@ -10,6 +10,10 @@
namespace task_manager {
namespace browsertest_util {
+// For the old task manager browser tests, they must call this to disable the
+// use of the old implementation and revert back to the old one.
Lei Zhang 2015/08/14 22:44:24 the first "old" is suppose to be new, right?
afakhry 2015/08/14 23:04:40 Actually it's the second "old". Done.
Lei Zhang 2015/08/14 23:22:58 Eh? Surely you also meant the first one since that
afakhry 2015/08/14 23:25:51 There used to be three "old"s in the comment! :D I
+void EnableOldTaskManager();
+
// Runs the message loop, observing the task manager, until there are exactly
// |resource_count| many resources whose titles match the pattern
// |title_pattern|. The match is done via string_util's base::MatchPattern, so
« no previous file with comments | « chrome/browser/task_manager/task_manager_browsertest.cc ('k') | chrome/browser/task_manager/task_manager_browsertest_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698