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

Side by Side Diff: ash/metrics/task_switch_metrics_recorder.cc

Issue 1132293004: Record time between switching the active window from Ash overview mode (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "ash/metrics/task_switch_metrics_recorder.h" 5 #include "ash/metrics/task_switch_metrics_recorder.h"
6 6
7 #include "ash/metrics/task_switch_time_tracker.h" 7 #include "ash/metrics/task_switch_time_tracker.h"
8 8
9 namespace ash { 9 namespace ash {
10 10
11 namespace { 11 namespace {
12 12
13 const char kShelfHistogramName[] = 13 const char kShelfHistogramName[] =
14 "Ash.Shelf.TimeBetweenNavigateToTaskSwitches"; 14 "Ash.Shelf.TimeBetweenNavigateToTaskSwitches";
15 15
16 const char kAcceleratorWindowCycleHistogramName[] = 16 const char kAcceleratorWindowCycleHistogramName[] =
17 "Ash.WindowCycleController.TimeBetweenTaskSwitches"; 17 "Ash.WindowCycleController.TimeBetweenTaskSwitches";
18 18
19 const char kOverviewModeHistogramName[] =
20 "Ash.WindowSelector.TimeBetweenActiveWindowChanges";
21
19 // Returns the histogram name for the given |task_switch_source|. 22 // Returns the histogram name for the given |task_switch_source|.
20 const char* GetHistogramName( 23 const char* GetHistogramName(
21 TaskSwitchMetricsRecorder::TaskSwitchSource task_switch_source) { 24 TaskSwitchMetricsRecorder::TaskSwitchSource task_switch_source) {
22 switch (task_switch_source) { 25 switch (task_switch_source) {
26 case TaskSwitchMetricsRecorder::kOverviewMode:
27 return kOverviewModeHistogramName;
23 case TaskSwitchMetricsRecorder::kShelf: 28 case TaskSwitchMetricsRecorder::kShelf:
24 return kShelfHistogramName; 29 return kShelfHistogramName;
25 case TaskSwitchMetricsRecorder::kWindowCycleController: 30 case TaskSwitchMetricsRecorder::kWindowCycleController:
26 return kAcceleratorWindowCycleHistogramName; 31 return kAcceleratorWindowCycleHistogramName;
27 } 32 }
28 NOTREACHED(); 33 NOTREACHED();
29 return nullptr; 34 return nullptr;
30 } 35 }
31 36
32 } // namespace 37 } // namespace
(...skipping 28 matching lines...) Expand all
61 66
62 const char* histogram_name = GetHistogramName(task_switch_source); 67 const char* histogram_name = GetHistogramName(task_switch_source);
63 DCHECK(histogram_name); 68 DCHECK(histogram_name);
64 69
65 histogram_map_.add( 70 histogram_map_.add(
66 task_switch_source, 71 task_switch_source,
67 make_scoped_ptr(new TaskSwitchTimeTracker(histogram_name))); 72 make_scoped_ptr(new TaskSwitchTimeTracker(histogram_name)));
68 } 73 }
69 74
70 } // namespace ash 75 } // namespace ash
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698