Chromium Code Reviews| Index: chrome/browser/ui/toolbar/media_router_action.cc |
| diff --git a/chrome/browser/ui/toolbar/media_router_action.cc b/chrome/browser/ui/toolbar/media_router_action.cc |
| index 34877266f464fa542c6b46dc2c964bd0a42ed25f..6cda815fa45610a1189cbe8a3ff1fbc0b3d668fc 100644 |
| --- a/chrome/browser/ui/toolbar/media_router_action.cc |
| +++ b/chrome/browser/ui/toolbar/media_router_action.cc |
| @@ -4,11 +4,13 @@ |
| #include "chrome/browser/ui/toolbar/media_router_action.h" |
| +#include "base/metrics/user_metrics.h" |
| #include "base/strings/utf_string_conversions.h" |
| #include "chrome/browser/media/router/issue.h" |
| #include "chrome/browser/media/router/media_route.h" |
| #include "chrome/browser/media/router/media_router.h" |
| #include "chrome/browser/media/router/media_router_factory.h" |
| +#include "chrome/browser/media/router/media_router_metrics.h" |
| #include "chrome/browser/media/router/media_router_mojo_impl.h" |
| #include "chrome/browser/profiles/profile.h" |
| #include "chrome/browser/ui/browser.h" |
| @@ -130,9 +132,18 @@ ui::MenuModel* MediaRouterAction::GetContextMenu() { |
| } |
| bool MediaRouterAction::ExecuteAction(bool by_user) { |
| + base::RecordAction(base::UserMetricsAction("MediaRouter_Icon_Click")); |
| + |
| GetMediaRouterDialogController()->ShowMediaRouterDialog(); |
| - if (GetPlatformDelegate()) |
| - GetPlatformDelegate()->CloseOverflowMenuIfOpen(); |
| + if (GetPlatformDelegate()) { |
| + if (GetPlatformDelegate()->CloseOverflowMenuIfOpen()) { |
| + media_router::MediaRouterMetrics::RecordMediaRouterDialogOrigin( |
| + media_router::OVERFLOW_MENU); |
| + } else { |
| + media_router::MediaRouterMetrics::RecordMediaRouterDialogOrigin( |
| + media_router::TOOLBAR); |
| + } |
|
Peter Kasting
2015/10/23 23:48:51
Nit: Shorter:
media_router::MediaRouterMetric
apacible
2015/10/24 22:06:28
Done.
|
| + } |
| return true; |
| } |