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

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

Issue 7550004: WebUI TaskManager: make it default on Chrome OS (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move the initializing of observer before the loading. Created 9 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "base/message_loop.h"
6 #include "chrome/browser/browser_process.h"
7 #include "chrome/browser/profiles/profile.h"
8 #include "chrome/browser/task_manager/task_manager.h"
9 #include "chrome/browser/task_manager/task_manager_browsertest_util.h"
10 #include "chrome/browser/ui/browser.h"
11 #include "chrome/browser/ui/browser_window.h"
12 #include "chrome/common/chrome_notification_types.h"
13 #include "chrome/test/base/ui_test_utils.h"
14 #include "content/common/notification_source.h"
15
16 namespace {
17
18 class ResourceChangeObserver : public TaskManagerModelObserver {
19 public:
20 ResourceChangeObserver(const TaskManagerModel* model,
21 int target_resource_count)
22 : model_(model),
23 target_resource_count_(target_resource_count) {
24 }
25
26 virtual void OnModelChanged() OVERRIDE {
27 OnResourceChange();
28 }
29
30 virtual void OnItemsChanged(int start, int length) OVERRIDE {
31 OnResourceChange();
32 }
33
34 virtual void OnItemsAdded(int start, int length) OVERRIDE {
35 OnResourceChange();
36 }
37
38 virtual void OnItemsRemoved(int start, int length) OVERRIDE {
39 OnResourceChange();
40 }
41
42 private:
43 void OnResourceChange() {
44 if (model_->ResourceCount() == target_resource_count_)
45 MessageLoopForUI::current()->Quit();
46 }
47
48 const TaskManagerModel* model_;
49 const int target_resource_count_;
50 };
51
52 } // namespace
53
54 // static
55 void TaskManagerBrowserTestUtil::WaitForResourceChange(int target_count) {
56 TaskManagerModel* model = TaskManager::GetInstance()->model();
57
58 if (model->ResourceCount() == target_count)
59 return;
60 ResourceChangeObserver observer(model, target_count);
61 model->AddObserver(&observer);
62 ui_test_utils::RunMessageLoop();
63 model->RemoveObserver(&observer);
64 }
65
66 // static
67 void TaskManagerBrowserTestUtil::ShowTaskManagerAndWaitForReady(
68 Browser* browser) {
69 #if defined(WEBUI_TASK_MANAGER)
70 ui_test_utils::WindowedNotificationObserver observer(
71 chrome::NOTIFICATION_TASK_MANAGER_WINDOW_READY,
72 Source<TaskManagerModel>(TaskManager::GetInstance()->model()));
73 browser->window()->ShowTaskManager();
74 observer.Wait();
75 #else
76 browser->window()->ShowTaskManager();
77 #endif // defined(WEBUI_TASK_MANAGER)
78 }
79
OLDNEW
« no previous file with comments | « chrome/browser/task_manager/task_manager_browsertest_util.h ('k') | chrome/browser/ui/views/frame/browser_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698