Added UMA metrics for display settings UI in the status area.

ash/system/chromeos/tray_display.cc

Index: ash/system/chromeos/tray_display.cc
diff --git a/ash/system/chromeos/tray_display.cc b/ash/system/chromeos/tray_display.cc
index 8bf2f602d99be57c9115f5c8d59962eff7ade0ca..657987c24fab081b51a8dc1f807858a052b24003 100644
--- a/ash/system/chromeos/tray_display.cc
+++ b/ash/system/chromeos/tray_display.cc
@@ -8,6 +8,7 @@
 #include <utility>
 #include <vector>
 
+#include "ash/common/metrics/user_metrics_action.h"
 #include "ash/common/system/chromeos/devicetype_utils.h"
 #include "ash/common/system/system_notifier.h"
 #include "ash/common/system/tray/actionable_view.h"
@@ -112,13 +113,14 @@ base::string16 GetAllDisplayInfo() {
   return base::JoinString(lines, base::ASCIIToUTF16("\n"));
 }
 
-void OpenSettings() {
+// Attempts to open the display settings, returns true if successful.
+bool OpenSettings() {
   // switch is intentionally introduced without default, to cause an error when
   // a new type of login status is introduced.
   switch (WmShell::Get()->system_tray_delegate()->GetUserLoginStatus()) {
     case LoginStatus::NOT_LOGGED_IN:
     case LoginStatus::LOCKED:
-      return;
+      return false;
 
     case LoginStatus::USER:
     case LoginStatus::OWNER:
@@ -127,8 +129,22 @@ void OpenSettings() {
     case LoginStatus::SUPERVISED:
     case LoginStatus::KIOSK_APP:
       SystemTrayDelegate* delegate = WmShell::Get()->system_tray_delegate();
-      if (delegate->ShouldShowSettings())
+      if (delegate->ShouldShowSettings()) {
         delegate->ShowDisplaySettings();
+        return true;
+      }
+  }
+
+  return false;
+}
+
+// Callback to handle a user selecting the notification view.
+void OpenSettingsFromNotification() {
+  WmShell::Get()->RecordUserMetricsAction(
+      UMA_STATUS_AREA_DISPLAY_NOTIFICATION_SELECTED);
+  if (OpenSettings()) {
+    WmShell::Get()->RecordUserMetricsAction(
+        UMA_STATUS_AREA_DISPLAY_NOTIFICATION_SHOW_SETTINGS);
   }
 }
 
@@ -280,7 +296,12 @@ class DisplayView : public ActionableView {
 
   // Overridden from ActionableView.
   bool PerformAction(const ui::Event& event) override {
-    OpenSettings();
+    WmShell::Get()->RecordUserMetricsAction(
+        UMA_STATUS_AREA_DISPLAY_DEFAULT_SELECTED);
+    if (OpenSettings()) {
+      WmShell::Get()->RecordUserMetricsAction(
+          UMA_STATUS_AREA_DISPLAY_DEFAULT_SHOW_SETTINGS);
+    }
     return true;
   }
 
@@ -405,8 +426,10 @@ void TrayDisplay::CreateOrUpdateNotification(
                                  system_notifier::kNotifierDisplay),
       message_center::RichNotificationData(),
       new message_center::HandleNotificationClickedDelegate(
-          base::Bind(&OpenSettings))));
+          base::Bind(&OpenSettingsFromNotification))));
 
+  WmShell::Get()->RecordUserMetricsAction(
+      UMA_STATUS_AREA_DISPLAY_NOTIFICATION_CREATED);
   message_center::MessageCenter::Get()->AddNotification(
       std::move(notification));
 }