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

Unified Diff: ash/common/system/chromeos/palette/common_palette_tool.cc

Issue 2308823002: Add UMA stats for pen palette (Closed)
Patch Set: rebase Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: ash/common/system/chromeos/palette/common_palette_tool.cc
diff --git a/ash/common/system/chromeos/palette/common_palette_tool.cc b/ash/common/system/chromeos/palette/common_palette_tool.cc
index 9f8faf4c71dc6f65c38b75c9cdbe360a63780529..fbad6311ffbdac17640b81e6be3a1d4dd702f717 100644
--- a/ash/common/system/chromeos/palette/common_palette_tool.cc
+++ b/ash/common/system/chromeos/palette/common_palette_tool.cc
@@ -10,6 +10,7 @@
#include "ash/common/system/tray/hover_highlight_view.h"
#include "ash/common/system/tray/view_click_listener.h"
#include "base/logging.h"
+#include "base/metrics/histogram_macros.h"
#include "base/strings/utf_string_conversions.h"
#include "grit/ash_resources.h"
#include "ui/base/resource/resource_bundle.h"
@@ -33,6 +34,18 @@ const int kExtraMarginFromLeftEdge = 4;
// Distance between the icon and the name of the tool in DP.
const int kMarginBetweenIconAndText = 18;
+void AddHistogramTimes(PaletteToolId id, base::TimeDelta duration) {
+ if (id == PaletteToolId::LASER_POINTER) {
+ UMA_HISTOGRAM_CUSTOM_TIMES("Ash.Shelf.Palette.InLaserPointerMode", duration,
+ base::TimeDelta::FromMilliseconds(100),
+ base::TimeDelta::FromHours(1), 50);
+ } else if (id == PaletteToolId::MAGNIFY) {
+ UMA_HISTOGRAM_CUSTOM_TIMES("Ash.Shelf.Palette.InMagnifyMode", duration,
+ base::TimeDelta::FromMilliseconds(100),
+ base::TimeDelta::FromHours(1), 50);
+ }
+}
+
} // namespace
CommonPaletteTool::CommonPaletteTool(Delegate* delegate)
@@ -52,6 +65,7 @@ void CommonPaletteTool::OnViewDestroyed() {
void CommonPaletteTool::OnEnable() {
PaletteTool::OnEnable();
+ start_time_ = base::TimeTicks::Now();
if (highlight_view_) {
highlight_view_->SetHighlight(true);
@@ -61,6 +75,7 @@ void CommonPaletteTool::OnEnable() {
void CommonPaletteTool::OnDisable() {
PaletteTool::OnDisable();
+ AddHistogramTimes(GetToolId(), base::TimeTicks::Now() - start_time_);
if (highlight_view_) {
highlight_view_->SetHighlight(false);
@@ -69,10 +84,16 @@ void CommonPaletteTool::OnDisable() {
}
void CommonPaletteTool::OnViewClicked(views::View* sender) {
- if (enabled())
+ delegate()->RecordPaletteOptionsUsage(
+ PaletteToolIdToPaletteTrayOptions(GetToolId()));
+ if (enabled()) {
delegate()->DisableTool(GetToolId());
- else
+ delegate()->RecordPaletteModeCancellation(
+ PaletteToolIdToPaletteModeCancelType(GetToolId(),
+ false /*is_switched*/));
+ } else {
delegate()->EnableTool(GetToolId());
+ }
}
views::View* CommonPaletteTool::CreateDefaultView(const base::string16& name) {

Powered by Google App Engine
This is Rietveld 408576698