Add UMA stats for pen palette

ash/common/system/chromeos/palette/common_palette_tool.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ash/common/system/chromeos/palette/common_palette_tool.h"
 
 #include "ash/common/shelf/shelf_constants.h"
 #include "ash/common/system/chromeos/palette/palette_ids.h"
 #include "ash/common/system/chromeos/palette/palette_tool_manager.h"
 #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"
 #include "ui/gfx/color_palette.h"
 #include "ui/gfx/paint_vector_icon.h"
 #include "ui/views/border.h"
 
 namespace ash {
 namespace {
 
@@ skipping 21 matching lines @@
   return CreateDefaultView(
       base::ASCIIToUTF16("[TODO] " + PaletteToolIdToString(GetToolId())));
 }
 
 void CommonPaletteTool::OnViewDestroyed() {
   highlight_view_ = nullptr;
 }
 
 void CommonPaletteTool::OnEnable() {
   PaletteTool::OnEnable();
+  start_time_ = base::TimeTicks::Now();
 
   if (highlight_view_) {
     highlight_view_->SetHighlight(true);
     highlight_view_->SetRightIconVisible(true);
   }
 }
 
 void CommonPaletteTool::OnDisable() {
   PaletteTool::OnDisable();
 
+  if (GetToolId() == PaletteToolId::LASER_POINTER) {
+    UMA_HISTOGRAM_CUSTOM_TIMES("Ash.Shelf.Palette.InLaserPointerMode",
+                               base::TimeTicks::Now() - start_time_,
+                               base::TimeDelta::FromMilliseconds(100),
+                               base::TimeDelta::FromHours(1), 50);
+  } else if (GetToolId() == PaletteToolId::MAGNIFY) {
+    UMA_HISTOGRAM_CUSTOM_TIMES("Ash.Shelf.Palette.InMagnifyMode",
+                               base::TimeTicks::Now() - start_time_,
+                               base::TimeDelta::FromMilliseconds(100),
+                               base::TimeDelta::FromHours(1), 50);
+  }
+
   if (highlight_view_) {
     highlight_view_->SetHighlight(false);
     highlight_view_->SetRightIconVisible(false);
   }
 }
 
 void CommonPaletteTool::OnViewClicked(views::View* sender) {
-  if (enabled())
+  delegate()->RecordPaletteOptionsUsage(
+      PaletteToolIdToPaletteTrayOptions(GetToolId()));
+  if (enabled()) {
     delegate()->DisableTool(GetToolId());
-  else
+    if (GetToolId() == PaletteToolId::LASER_POINTER) {
+      delegate()->RecordPaletteModeCancellation(
+          PaletteModeCancelType::PALETTE_MODE_LASER_POINTER_CANCELLED);
+    } else if (GetToolId() == PaletteToolId::MAGNIFY) {
+      delegate()->RecordPaletteModeCancellation(
+          PaletteModeCancelType::PALETTE_MODE_MAGNIFY_CANCELLED);
+    }
+  } else
     delegate()->EnableTool(GetToolId());

[Ilya Sherman, 2016/09/06 20:45:32]

nit: Please add curly braces to the else stmt, since the if stmt has them.

[xiaoyinh(OOO Sep 11-29), 2016/09/06 22:31:35]

Done.

 }
 
 views::View* CommonPaletteTool::CreateDefaultView(const base::string16& name) {
   gfx::ImageSkia icon =
       CreateVectorIcon(GetPaletteIconId(), kIconSize, gfx::kChromeIconGrey);
   gfx::ImageSkia check = CreateVectorIcon(gfx::VectorIconId::CHECK_CIRCLE,
                                           kIconSize, gfx::kGoogleGreen700);
 
   highlight_view_ = new HoverHighlightView(this);
   highlight_view_->SetBorder(
       views::Border::CreateEmptyBorder(0, kExtraMarginFromLeftEdge, 0, 0));
   highlight_view_->AddIconAndLabelCustomSize(icon, name, false, kIconSize,
                                              kMarginFromEdges,
                                              kMarginBetweenIconAndText);
   highlight_view_->AddRightIcon(check, kIconSize);
 
   if (enabled())
     highlight_view_->SetHighlight(true);
   else
     highlight_view_->SetRightIconVisible(false);
 
   return highlight_view_;
 }
 
 }  // namespace ash