Date and Time dialog for when the clock isn't synced.

chrome/browser/ui/ash/system_tray_delegate_chromeos.cc

Index: chrome/browser/ui/ash/system_tray_delegate_chromeos.cc
diff --git a/chrome/browser/ui/ash/system_tray_delegate_chromeos.cc b/chrome/browser/ui/ash/system_tray_delegate_chromeos.cc
index 01e8e3784fd0ea90c963719bbd22c737f6b289d6..2ca835a7cbd93f23142a316841769fe4ca335afb 100644
--- a/chrome/browser/ui/ash/system_tray_delegate_chromeos.cc
+++ b/chrome/browser/ui/ash/system_tray_delegate_chromeos.cc
@@ -67,6 +67,7 @@
 #include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h"
 #include "chrome/browser/chromeos/policy/device_cloud_policy_manager_chromeos.h"
 #include "chrome/browser/chromeos/profiles/multiprofiles_intro_dialog.h"
+#include "chrome/browser/chromeos/set_time_dialog.h"
 #include "chrome/browser/chromeos/settings/cros_settings.h"
 #include "chrome/browser/chromeos/sim_dialog_delegate.h"
 #include "chrome/browser/chromeos/ui/choose_mobile_network_dialog.h"
@@ -474,10 +475,17 @@ bool SystemTrayDelegateChromeOS::ShouldShowSettings() {
 
 void SystemTrayDelegateChromeOS::ShowDateSettings() {
   content::RecordAction(base::UserMetricsAction("ShowDateOptions"));
+
+  // Everybody can change the time and time zone (even though they are
+  // device settings).
+  if (GetUserLoginStatus() == ash::user::LOGGED_IN_NONE) {
+    SetTimeDialog::ShowDialog(GetNativeWindow());

[Daniel Erat, 2014/04/24 19:49:59]

this method seems like a strange place to call this. this method is usually used
to display the time zone and 24-hour clock settings page when the user clicks on
the time within the system tray, right?

does the user get to the time-setting dialog through the same route in the UI,
or are there other ways to get to it? if there are other ways, it may be better
to add a new method to the delegate.

also note that you probably want to record separate metrics for the time-setting
path so you can see how frequently it's triggered. right now, i imagine that
those calls would be lumped in with the ShowDateOptions user action reported at
the beginning of this method, which is probably undesirable.

[michaelpg, 2014/04/24 20:37:12]

I've added a separate method and updated DateView to handle multiple action
types.

+    return;
+  }
+
   std::string sub_page =
       std::string(chrome::kSearchSubPage) + "#" +
       l10n_util::GetStringUTF8(IDS_OPTIONS_SETTINGS_SECTION_TITLE_DATETIME);
-  // Everybody can change the time zone (even though it is a device setting).
   ShowSettingsSubPageForActiveUser(sub_page);
 }