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

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

Issue 1062413002: Remove unnecessary instrumentation for several jank bugs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Compile fixes Created 5 years, 8 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 224 matching lines...) Expand 10 before | Expand all | Expand 10 after
235 //////////////////////////////////////////////////////////////////////////////// 235 ////////////////////////////////////////////////////////////////////////////////
236 // TaskManagerModel class 236 // TaskManagerModel class
237 //////////////////////////////////////////////////////////////////////////////// 237 ////////////////////////////////////////////////////////////////////////////////
238 238
239 TaskManagerModel::TaskManagerModel(TaskManager* task_manager) 239 TaskManagerModel::TaskManagerModel(TaskManager* task_manager)
240 : pending_video_memory_usage_stats_update_(false), 240 : pending_video_memory_usage_stats_update_(false),
241 update_requests_(0), 241 update_requests_(0),
242 listen_requests_(0), 242 listen_requests_(0),
243 update_state_(IDLE), 243 update_state_(IDLE),
244 is_updating_byte_count_(false) { 244 is_updating_byte_count_(false) {
245 // TODO(vadimt): Remove ScopedTracker below once crbug.com/437890 is fixed.
246 tracked_objects::ScopedTracker tracking_profile1(
247 FROM_HERE_WITH_EXPLICIT_FUNCTION(
248 "437890 TaskManagerModel::TaskManagerModel1"));
249
250 AddResourceProvider( 245 AddResourceProvider(
251 new task_manager::BrowserProcessResourceProvider(task_manager)); 246 new task_manager::BrowserProcessResourceProvider(task_manager));
252
253 // TODO(vadimt): Remove ScopedTracker below once crbug.com/437890 is fixed.
254 tracked_objects::ScopedTracker tracking_profile2(
255 FROM_HERE_WITH_EXPLICIT_FUNCTION(
256 "437890 TaskManagerModel::TaskManagerModel2"));
257
258 AddResourceProvider(new task_manager::WebContentsResourceProvider( 247 AddResourceProvider(new task_manager::WebContentsResourceProvider(
259 task_manager, 248 task_manager,
260 scoped_ptr<WebContentsInformation>( 249 scoped_ptr<WebContentsInformation>(
261 new task_manager::BackgroundInformation()))); 250 new task_manager::BackgroundInformation())));
262
263 // TODO(vadimt): Remove ScopedTracker below once crbug.com/437890 is fixed.
264 tracked_objects::ScopedTracker tracking_profile3(
265 FROM_HERE_WITH_EXPLICIT_FUNCTION(
266 "437890 TaskManagerModel::TaskManagerModel3"));
267
268 AddResourceProvider(new task_manager::WebContentsResourceProvider( 251 AddResourceProvider(new task_manager::WebContentsResourceProvider(
269 task_manager, 252 task_manager,
270 scoped_ptr<WebContentsInformation>( 253 scoped_ptr<WebContentsInformation>(
271 new task_manager::TabContentsInformation()))); 254 new task_manager::TabContentsInformation())));
272 #if defined(ENABLE_PRINT_PREVIEW) 255 #if defined(ENABLE_PRINT_PREVIEW)
273
274 // TODO(vadimt): Remove ScopedTracker below once crbug.com/437890 is fixed.
275 tracked_objects::ScopedTracker tracking_profile4(
276 FROM_HERE_WITH_EXPLICIT_FUNCTION(
277 "437890 TaskManagerModel::TaskManagerModel4"));
278
279 AddResourceProvider(new task_manager::WebContentsResourceProvider( 256 AddResourceProvider(new task_manager::WebContentsResourceProvider(
280 task_manager, 257 task_manager,
281 scoped_ptr<WebContentsInformation>( 258 scoped_ptr<WebContentsInformation>(
282 new task_manager::PrintingInformation()))); 259 new task_manager::PrintingInformation())));
283 #endif // ENABLE_PRINT_PREVIEW 260 #endif // ENABLE_PRINT_PREVIEW
284
285 // TODO(vadimt): Remove ScopedTracker below once crbug.com/437890 is fixed.
286 tracked_objects::ScopedTracker tracking_profile5(
287 FROM_HERE_WITH_EXPLICIT_FUNCTION(
288 "437890 TaskManagerModel::TaskManagerModel5"));
289
290 AddResourceProvider(new task_manager::WebContentsResourceProvider( 261 AddResourceProvider(new task_manager::WebContentsResourceProvider(
291 task_manager, 262 task_manager,
292 scoped_ptr<WebContentsInformation>( 263 scoped_ptr<WebContentsInformation>(
293 new task_manager::PanelInformation()))); 264 new task_manager::PanelInformation())));
294
295 // TODO(vadimt): Remove ScopedTracker below once crbug.com/437890 is fixed.
296 tracked_objects::ScopedTracker tracking_profile6(
297 FROM_HERE_WITH_EXPLICIT_FUNCTION(
298 "437890 TaskManagerModel::TaskManagerModel6"));
299
300 AddResourceProvider( 265 AddResourceProvider(
301 new task_manager::ChildProcessResourceProvider(task_manager)); 266 new task_manager::ChildProcessResourceProvider(task_manager));
302
303 // TODO(vadimt): Remove ScopedTracker below once crbug.com/437890 is fixed.
304 tracked_objects::ScopedTracker tracking_profile7(
305 FROM_HERE_WITH_EXPLICIT_FUNCTION(
306 "437890 TaskManagerModel::TaskManagerModel7"));
307
308 AddResourceProvider(new task_manager::WebContentsResourceProvider( 267 AddResourceProvider(new task_manager::WebContentsResourceProvider(
309 task_manager, 268 task_manager,
310 scoped_ptr<WebContentsInformation>( 269 scoped_ptr<WebContentsInformation>(
311 new task_manager::ExtensionInformation()))); 270 new task_manager::ExtensionInformation())));
312
313 // TODO(vadimt): Remove ScopedTracker below once crbug.com/437890 is fixed.
314 tracked_objects::ScopedTracker tracking_profile8(
315 FROM_HERE_WITH_EXPLICIT_FUNCTION(
316 "437890 TaskManagerModel::TaskManagerModel8"));
317
318 AddResourceProvider(new task_manager::WebContentsResourceProvider( 271 AddResourceProvider(new task_manager::WebContentsResourceProvider(
319 task_manager, 272 task_manager,
320 scoped_ptr<WebContentsInformation>( 273 scoped_ptr<WebContentsInformation>(
321 new task_manager::GuestInformation()))); 274 new task_manager::GuestInformation())));
322 } 275 }
323 276
324 void TaskManagerModel::AddObserver(TaskManagerModelObserver* observer) { 277 void TaskManagerModel::AddObserver(TaskManagerModelObserver* observer) {
325 observer_list_.AddObserver(observer); 278 observer_list_.AddObserver(observer);
326 } 279 }
327 280
(...skipping 1277 matching lines...) Expand 10 before | Expand all | Expand 10 after
1605 params.host_desktop_type = desktop_type; 1558 params.host_desktop_type = desktop_type;
1606 chrome::Navigate(&params); 1559 chrome::Navigate(&params);
1607 } 1560 }
1608 1561
1609 TaskManager::TaskManager() 1562 TaskManager::TaskManager()
1610 : model_(new TaskManagerModel(this)) { 1563 : model_(new TaskManagerModel(this)) {
1611 } 1564 }
1612 1565
1613 TaskManager::~TaskManager() { 1566 TaskManager::~TaskManager() {
1614 } 1567 }
OLDNEW
« no previous file with comments | « chrome/browser/task_manager/browser_process_resource_provider.cc ('k') | components/domain_reliability/monitor.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698