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

Side by Side Diff: chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc

Issue 2335203003: Add metrics to keep track of the tab activate/deactivate cycle (Closed)
Patch Set: Removed a problematic dcheck Created 4 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
« no previous file with comments | « chrome/browser/BUILD.gn ('k') | chrome/browser/metrics/tab_reactivation_tracker.h » ('j') | 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) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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/metrics/chrome_browser_main_extra_parts_metrics.h" 5 #include "chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 14 matching lines...) Expand all
25 #include "content/public/browser/browser_thread.h" 25 #include "content/public/browser/browser_thread.h"
26 #include "ui/base/touch/touch_device.h" 26 #include "ui/base/touch/touch_device.h"
27 #include "ui/base/ui_base_switches.h" 27 #include "ui/base/ui_base_switches.h"
28 #include "ui/display/screen.h" 28 #include "ui/display/screen.h"
29 #include "ui/events/event_switches.h" 29 #include "ui/events/event_switches.h"
30 30
31 #if defined(OS_ANDROID) && defined(__arm__) 31 #if defined(OS_ANDROID) && defined(__arm__)
32 #include <cpu-features.h> 32 #include <cpu-features.h>
33 #endif // defined(OS_ANDROID) && defined(__arm__) 33 #endif // defined(OS_ANDROID) && defined(__arm__)
34 34
35 #if !defined(OS_ANDROID)
36 #include "chrome/browser/metrics/tab_usage_recorder.h"
37 #endif // !defined(OS_ANDROID)
38
35 #if defined(OS_LINUX) && !defined(OS_CHROMEOS) 39 #if defined(OS_LINUX) && !defined(OS_CHROMEOS)
36 #include <gnu/libc-version.h> 40 #include <gnu/libc-version.h>
37 41
38 #include "base/version.h" 42 #include "base/version.h"
39 #if defined(USE_X11) 43 #if defined(USE_X11)
40 #include "ui/base/x/x11_util.h" 44 #include "ui/base/x/x11_util.h"
41 #endif 45 #endif
42 #endif // defined(OS_LINUX) && !defined(OS_CHROMEOS) 46 #endif // defined(OS_LINUX) && !defined(OS_CHROMEOS)
43 47
44 #if defined(USE_OZONE) || defined(USE_X11) 48 #if defined(USE_OZONE) || defined(USE_X11)
(...skipping 310 matching lines...) Expand 10 before | Expand all | Expand 10 after
355 content::BrowserThread::PostDelayedTask( 359 content::BrowserThread::PostDelayedTask(
356 content::BrowserThread::IO, FROM_HERE, 360 content::BrowserThread::IO, FROM_HERE,
357 base::Bind(&shell_integration::win::RecordIsPinnedToTaskbarHistogram), 361 base::Bind(&shell_integration::win::RecordIsPinnedToTaskbarHistogram),
358 kStartupMetricsGatheringDelay); 362 kStartupMetricsGatheringDelay);
359 #endif // defined(OS_WIN) 363 #endif // defined(OS_WIN)
360 364
361 display_count_ = display::Screen::GetScreen()->GetNumDisplays(); 365 display_count_ = display::Screen::GetScreen()->GetNumDisplays();
362 UMA_HISTOGRAM_COUNTS_100("Hardware.Display.Count.OnStartup", display_count_); 366 UMA_HISTOGRAM_COUNTS_100("Hardware.Display.Count.OnStartup", display_count_);
363 display::Screen::GetScreen()->AddObserver(this); 367 display::Screen::GetScreen()->AddObserver(this);
364 is_screen_observer_ = true; 368 is_screen_observer_ = true;
369
370 #if !defined(OS_ANDROID)
371 metrics::TabUsageRecorder::Initialize();
372 #endif // !defined(OS_ANDROID)
365 } 373 }
366 374
367 void ChromeBrowserMainExtraPartsMetrics::OnDisplayAdded( 375 void ChromeBrowserMainExtraPartsMetrics::OnDisplayAdded(
368 const display::Display& new_display) { 376 const display::Display& new_display) {
369 EmitDisplaysChangedMetric(); 377 EmitDisplaysChangedMetric();
370 } 378 }
371 379
372 void ChromeBrowserMainExtraPartsMetrics::OnDisplayRemoved( 380 void ChromeBrowserMainExtraPartsMetrics::OnDisplayRemoved(
373 const display::Display& old_display) { 381 const display::Display& old_display) {
374 EmitDisplaysChangedMetric(); 382 EmitDisplaysChangedMetric();
(...skipping 11 matching lines...) Expand all
386 } 394 }
387 } 395 }
388 396
389 namespace chrome { 397 namespace chrome {
390 398
391 void AddMetricsExtraParts(ChromeBrowserMainParts* main_parts) { 399 void AddMetricsExtraParts(ChromeBrowserMainParts* main_parts) {
392 main_parts->AddParts(new ChromeBrowserMainExtraPartsMetrics()); 400 main_parts->AddParts(new ChromeBrowserMainExtraPartsMetrics());
393 } 401 }
394 402
395 } // namespace chrome 403 } // namespace chrome
OLDNEW
« no previous file with comments | « chrome/browser/BUILD.gn ('k') | chrome/browser/metrics/tab_reactivation_tracker.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698