Added Ash.SystemMenu.DefaultView.VisibleItems histogram.

ash/common/system/chromeos/settings/tray_settings.cc

 // Copyright (c) 2012 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/settings/tray_settings.h"
 
 #include "ash/common/session/session_state_delegate.h"
 #include "ash/common/system/chromeos/power/power_status.h"
 #include "ash/common/system/chromeos/power/power_status_view.h"
 #include "ash/common/system/tray/actionable_view.h"
@@ skipping 108 matching lines @@
   LoginStatus login_status_;
   views::Label* label_;
   ash::PowerStatusView* power_status_view_;
 
   DISALLOW_COPY_AND_ASSIGN(SettingsDefaultView);
 };
 
 }  // namespace tray
 
 TraySettings::TraySettings(SystemTray* system_tray)
-    : SystemTrayItem(system_tray), default_view_(NULL) {}
+    : SystemTrayItem(system_tray, OS_SETTINGS), default_view_(NULL) {}

[tdanderson, 2016/07/19 21:58:39]

Just SETTINGS instead of OS_SETTINGS to match class name and remain consistent
with your naming of other enum values?

[bruthig, 2016/07/20 18:15:52]

Done.

 
 TraySettings::~TraySettings() {}
 
 views::View* TraySettings::CreateTrayView(LoginStatus status) {
   return NULL;
 }
 
 views::View* TraySettings::CreateDefaultView(LoginStatus status) {
   if ((status == LoginStatus::NOT_LOGGED_IN || status == LoginStatus::LOCKED) &&
       !PowerStatus::Get()->IsBatteryPresent())
@@ skipping 14 matching lines @@
 
 void TraySettings::DestroyDefaultView() {
   default_view_ = NULL;
 }
 
 void TraySettings::DestroyDetailedView() {}
 
 void TraySettings::UpdateAfterLoginStatusChange(LoginStatus status) {}
 
 }  // namespace ash