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

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

Issue 663023009: Standardize usage of virtual/override/final in chrome/browser/task_manager/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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
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.h" 5 #include "chrome/browser/task_manager/task_manager.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/i18n/number_formatting.h" 8 #include "base/i18n/number_formatting.h"
9 #include "base/i18n/rtl.h" 9 #include "base/i18n/rtl.h"
10 #include "base/prefs/pref_registry_simple.h" 10 #include "base/prefs/pref_registry_simple.h"
(...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after
158 158
159 } // namespace 159 } // namespace
160 160
161 class TaskManagerModelGpuDataManagerObserver 161 class TaskManagerModelGpuDataManagerObserver
162 : public content::GpuDataManagerObserver { 162 : public content::GpuDataManagerObserver {
163 public: 163 public:
164 TaskManagerModelGpuDataManagerObserver() { 164 TaskManagerModelGpuDataManagerObserver() {
165 content::GpuDataManager::GetInstance()->AddObserver(this); 165 content::GpuDataManager::GetInstance()->AddObserver(this);
166 } 166 }
167 167
168 virtual ~TaskManagerModelGpuDataManagerObserver() { 168 ~TaskManagerModelGpuDataManagerObserver() override {
169 content::GpuDataManager::GetInstance()->RemoveObserver(this); 169 content::GpuDataManager::GetInstance()->RemoveObserver(this);
170 } 170 }
171 171
172 static void NotifyVideoMemoryUsageStats( 172 static void NotifyVideoMemoryUsageStats(
173 const content::GPUVideoMemoryUsageStats& video_memory_usage_stats) { 173 const content::GPUVideoMemoryUsageStats& video_memory_usage_stats) {
174 TaskManager::GetInstance()->model()->NotifyVideoMemoryUsageStats( 174 TaskManager::GetInstance()->model()->NotifyVideoMemoryUsageStats(
175 video_memory_usage_stats); 175 video_memory_usage_stats);
176 } 176 }
177 177
178 virtual void OnVideoMemoryUsageStatsUpdate( 178 void OnVideoMemoryUsageStatsUpdate(const content::GPUVideoMemoryUsageStats&
179 const content::GPUVideoMemoryUsageStats& video_memory_usage_stats) 179 video_memory_usage_stats) override {
180 override {
181 if (BrowserThread::CurrentlyOn(BrowserThread::UI)) { 180 if (BrowserThread::CurrentlyOn(BrowserThread::UI)) {
182 NotifyVideoMemoryUsageStats(video_memory_usage_stats); 181 NotifyVideoMemoryUsageStats(video_memory_usage_stats);
183 } else { 182 } else {
184 BrowserThread::PostTask( 183 BrowserThread::PostTask(
185 BrowserThread::UI, FROM_HERE, base::Bind( 184 BrowserThread::UI, FROM_HERE, base::Bind(
186 &TaskManagerModelGpuDataManagerObserver:: 185 &TaskManagerModelGpuDataManagerObserver::
187 NotifyVideoMemoryUsageStats, 186 NotifyVideoMemoryUsageStats,
188 video_memory_usage_stats)); 187 video_memory_usage_stats));
189 } 188 }
190 } 189 }
(...skipping 1344 matching lines...) Expand 10 before | Expand all | Expand 10 after
1535 params.host_desktop_type = desktop_type; 1534 params.host_desktop_type = desktop_type;
1536 chrome::Navigate(&params); 1535 chrome::Navigate(&params);
1537 } 1536 }
1538 1537
1539 TaskManager::TaskManager() 1538 TaskManager::TaskManager()
1540 : model_(new TaskManagerModel(this)) { 1539 : model_(new TaskManagerModel(this)) {
1541 } 1540 }
1542 1541
1543 TaskManager::~TaskManager() { 1542 TaskManager::~TaskManager() {
1544 } 1543 }
OLDNEW
« no previous file with comments | « chrome/browser/task_manager/tab_contents_information.cc ('k') | chrome/browser/task_manager/task_manager_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698