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

Side by Side Diff: chrome/browser/task_manager/task_manager_browsertest_util.cc

Issue 1960703002: [Part 3 of 6] Switch ColumnSpecifier to enum class. This makes straddling namespaces easier. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@tm_file_split2
Patch Set: Rebase Created 4 years, 7 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 unified diff | Download patch
« no previous file with comments | « chrome/browser/task_manager/task_manager_browsertest_util.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/task_manager/task_manager_browsertest_util.h" 5 #include "chrome/browser/task_manager/task_manager_browsertest_util.h"
6 6
7 #include "base/bind_helpers.h" 7 #include "base/bind_helpers.h"
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/location.h" 9 #include "base/location.h"
10 #include "base/memory/ptr_util.h" 10 #include "base/memory/ptr_util.h"
(...skipping 102 matching lines...) Expand 10 before | Expand all | Expand 10 after
113 // TaskManagerTester: 113 // TaskManagerTester:
114 int GetRowCount() override { return model_->RowCount(); } 114 int GetRowCount() override { return model_->RowCount(); }
115 115
116 base::string16 GetRowTitle(int row) override { 116 base::string16 GetRowTitle(int row) override {
117 return model_->GetText(row, IDS_TASK_MANAGER_TASK_COLUMN); 117 return model_->GetText(row, IDS_TASK_MANAGER_TASK_COLUMN);
118 } 118 }
119 119
120 void ToggleColumnVisibility(ColumnSpecifier column) override { 120 void ToggleColumnVisibility(ColumnSpecifier column) override {
121 int column_id = 0; 121 int column_id = 0;
122 switch (column) { 122 switch (column) {
123 case COLUMN_NONE: 123 case ColumnSpecifier::COLUMN_NONE:
124 return; 124 return;
125 case SQLITE_MEMORY_USED: 125 case ColumnSpecifier::SQLITE_MEMORY_USED:
126 column_id = IDS_TASK_MANAGER_SQLITE_MEMORY_USED_COLUMN; 126 column_id = IDS_TASK_MANAGER_SQLITE_MEMORY_USED_COLUMN;
127 break; 127 break;
128 case V8_MEMORY_USED: 128 case ColumnSpecifier::V8_MEMORY_USED:
129 case V8_MEMORY: 129 case ColumnSpecifier::V8_MEMORY:
130 column_id = IDS_TASK_MANAGER_JAVASCRIPT_MEMORY_ALLOCATED_COLUMN; 130 column_id = IDS_TASK_MANAGER_JAVASCRIPT_MEMORY_ALLOCATED_COLUMN;
131 break; 131 break;
132 } 132 }
133 model_->ToggleColumnVisibility(column_id); 133 model_->ToggleColumnVisibility(column_id);
134 } 134 }
135 135
136 int64_t GetColumnValue(ColumnSpecifier column, int row) override { 136 int64_t GetColumnValue(ColumnSpecifier column, int row) override {
137 task_management::TaskId task_id = model_->tasks_[row]; 137 task_management::TaskId task_id = model_->tasks_[row];
138 int64_t value = 0; 138 int64_t value = 0;
139 int64_t ignored = 0; 139 int64_t ignored = 0;
140 bool success = false; 140 bool success = false;
141 141
142 switch (column) { 142 switch (column) {
143 case COLUMN_NONE: 143 case ColumnSpecifier::COLUMN_NONE:
144 break; 144 break;
145 case V8_MEMORY: 145 case ColumnSpecifier::V8_MEMORY:
146 success = task_manager()->GetV8Memory(task_id, &value, &ignored); 146 success = task_manager()->GetV8Memory(task_id, &value, &ignored);
147 break; 147 break;
148 case V8_MEMORY_USED: 148 case ColumnSpecifier::V8_MEMORY_USED:
149 success = task_manager()->GetV8Memory(task_id, &ignored, &value); 149 success = task_manager()->GetV8Memory(task_id, &ignored, &value);
150 break; 150 break;
151 case SQLITE_MEMORY_USED: 151 case ColumnSpecifier::SQLITE_MEMORY_USED:
152 value = task_manager()->GetSqliteMemoryUsed(task_id); 152 value = task_manager()->GetSqliteMemoryUsed(task_id);
153 success = true; 153 success = true;
154 break; 154 break;
155 } 155 }
156 if (!success) 156 if (!success)
157 return 0; 157 return 0;
158 return value; 158 return value;
159 } 159 }
160 160
161 int32_t GetTabId(int row) override { 161 int32_t GetTabId(int row) override {
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
204 int GetRowCount() override { return model_->ResourceCount(); } 204 int GetRowCount() override { return model_->ResourceCount(); }
205 205
206 base::string16 GetRowTitle(int row) override { 206 base::string16 GetRowTitle(int row) override {
207 return model_->GetResourceTitle(row); 207 return model_->GetResourceTitle(row);
208 } 208 }
209 209
210 int64_t GetColumnValue(ColumnSpecifier column, int row) override { 210 int64_t GetColumnValue(ColumnSpecifier column, int row) override {
211 size_t value = 0; 211 size_t value = 0;
212 bool success = false; 212 bool success = false;
213 switch (column) { 213 switch (column) {
214 case COLUMN_NONE: 214 case ColumnSpecifier::COLUMN_NONE:
215 break; 215 break;
216 case V8_MEMORY: 216 case ColumnSpecifier::V8_MEMORY:
217 success = model_->GetV8Memory(row, &value); 217 success = model_->GetV8Memory(row, &value);
218 break; 218 break;
219 case V8_MEMORY_USED: 219 case ColumnSpecifier::V8_MEMORY_USED:
220 success = model_->GetV8MemoryUsed(row, &value); 220 success = model_->GetV8MemoryUsed(row, &value);
221 break; 221 break;
222 case SQLITE_MEMORY_USED: 222 case ColumnSpecifier::SQLITE_MEMORY_USED:
223 success = model_->GetSqliteMemoryUsedBytes(row, &value); 223 success = model_->GetSqliteMemoryUsedBytes(row, &value);
224 break; 224 break;
225 } 225 }
226 if (!success) 226 if (!success)
227 return 0; 227 return 0;
228 return static_cast<int64_t>(value); 228 return static_cast<int64_t>(value);
229 } 229 }
230 230
231 void ToggleColumnVisibility(ColumnSpecifier column) override { 231 void ToggleColumnVisibility(ColumnSpecifier column) override {
232 // Doing nothing is okay here; the legacy TaskManager always collects all 232 // Doing nothing is okay here; the legacy TaskManager always collects all
(...skipping 97 matching lines...) Expand 10 before | Expand all | Expand 10 after
330 } 330 }
331 return match_count; 331 return match_count;
332 } 332 }
333 333
334 int64_t GetColumnValue(int index) { 334 int64_t GetColumnValue(int index) {
335 return task_manager_tester_->GetColumnValue(column_specifier_, index); 335 return task_manager_tester_->GetColumnValue(column_specifier_, index);
336 } 336 }
337 337
338 const char* GetColumnName() { 338 const char* GetColumnName() {
339 switch (column_specifier_) { 339 switch (column_specifier_) {
340 case COLUMN_NONE: 340 case ColumnSpecifier::COLUMN_NONE:
341 return "N/A"; 341 return "N/A";
342 case V8_MEMORY: 342 case ColumnSpecifier::V8_MEMORY:
343 return "V8 Memory"; 343 return "V8 Memory";
344 case V8_MEMORY_USED: 344 case ColumnSpecifier::V8_MEMORY_USED:
345 return "V8 Memory Used"; 345 return "V8 Memory Used";
346 case SQLITE_MEMORY_USED: 346 case ColumnSpecifier::SQLITE_MEMORY_USED:
347 return "SQLite Memory Used"; 347 return "SQLite Memory Used";
348 } 348 }
349 return "N/A"; 349 return "N/A";
350 } 350 }
351 351
352 void OnTimeout() { 352 void OnTimeout() {
353 base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE, 353 base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE,
354 run_loop_.QuitClosure()); 354 run_loop_.QuitClosure());
355 FAIL() << "Timed out.\n" << DumpTaskManagerModel(); 355 FAIL() << "Timed out.\n" << DumpTaskManagerModel();
356 } 356 }
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
393 std::unique_ptr<TaskManagerTester> GetTaskManagerTester() { 393 std::unique_ptr<TaskManagerTester> GetTaskManagerTester() {
394 if (IsNewTaskManagerViewEnabled()) 394 if (IsNewTaskManagerViewEnabled())
395 return base::WrapUnique(new TaskManagerTesterImpl(base::Closure())); 395 return base::WrapUnique(new TaskManagerTesterImpl(base::Closure()));
396 else 396 else
397 return base::WrapUnique(new LegacyTaskManagerTesterImpl(base::Closure())); 397 return base::WrapUnique(new LegacyTaskManagerTesterImpl(base::Closure()));
398 } 398 }
399 399
400 void WaitForTaskManagerRows(int required_count, 400 void WaitForTaskManagerRows(int required_count,
401 const base::string16& title_pattern) { 401 const base::string16& title_pattern) {
402 const int column_value_dont_care = 0; 402 const int column_value_dont_care = 0;
403 ResourceChangeObserver observer(required_count, title_pattern, COLUMN_NONE, 403 ResourceChangeObserver observer(required_count, title_pattern,
404 ColumnSpecifier::COLUMN_NONE,
404 column_value_dont_care); 405 column_value_dont_care);
405 observer.RunUntilSatisfied(); 406 observer.RunUntilSatisfied();
406 } 407 }
407 408
408 void WaitForTaskManagerStatToExceed(const base::string16& title_pattern, 409 void WaitForTaskManagerStatToExceed(const base::string16& title_pattern,
409 ColumnSpecifier column_getter, 410 ColumnSpecifier column_getter,
410 size_t min_column_value) { 411 size_t min_column_value) {
411 const int wait_for_one_match = 1; 412 const int wait_for_one_match = 1;
412 ResourceChangeObserver observer(wait_for_one_match, title_pattern, 413 ResourceChangeObserver observer(wait_for_one_match, title_pattern,
413 column_getter, min_column_value); 414 column_getter, min_column_value);
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
484 base::string16 MatchUtility(const base::string16& title) { 485 base::string16 MatchUtility(const base::string16& title) {
485 return l10n_util::GetStringFUTF16(IDS_TASK_MANAGER_UTILITY_PREFIX, title); 486 return l10n_util::GetStringFUTF16(IDS_TASK_MANAGER_UTILITY_PREFIX, title);
486 } 487 }
487 488
488 base::string16 MatchAnyUtility() { 489 base::string16 MatchAnyUtility() {
489 return MatchUtility(base::ASCIIToUTF16("*")); 490 return MatchUtility(base::ASCIIToUTF16("*"));
490 } 491 }
491 492
492 } // namespace browsertest_util 493 } // namespace browsertest_util
493 } // namespace task_manager 494 } // namespace task_manager
OLDNEW
« no previous file with comments | « chrome/browser/task_manager/task_manager_browsertest_util.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698