Fixes bug where clock button wasn't updating in response to a timezone

chrome/browser/chromeos/status/status_area_view_chromeos.h

Index: chrome/browser/chromeos/status/status_area_view_chromeos.h
diff --git a/chrome/browser/chromeos/status/status_area_view_chromeos.h b/chrome/browser/chromeos/status/status_area_view_chromeos.h
index df88d4143cb8c6adf1756dcbdcdf9071ce6772d5..5d6dda7872e13607a5f55e79f0b65acf9439051a 100644
--- a/chrome/browser/chromeos/status/status_area_view_chromeos.h
+++ b/chrome/browser/chromeos/status/status_area_view_chromeos.h
@@ -12,6 +12,8 @@
 #include "chrome/browser/chromeos/system/timezone_settings.h"
 #include "chrome/browser/chromeos/view_ids.h"
 
+class ClockMenuButton;
+
 namespace chromeos {
 
 class StatusAreaViewChromeos : public StatusAreaView,
@@ -41,9 +43,12 @@ class StatusAreaViewChromeos : public StatusAreaView,
   void SetDefaultUse24HourClock(bool use_24hour_clock);
 
   // Convenience function to add buttons to a status area for ChromeOS.
+  // |clock_button| is set to the ClockMenuButton that is created by this
+  // method.
   static void AddChromeosButtons(StatusAreaView* status_area,
                                  StatusAreaButton::Delegate* delegate,
-                                 ScreenMode screen_mode);
+                                 ScreenMode screen_mode,
+                                 ClockMenuButton** clock_button);

[stevenjb, 2011/11/18 21:18:29]

This seems a bit awkward to me. What do you think of adding GetButtonById() to
StatusAreaView instead?

[sky, 2011/11/18 21:23:24]

I went this route because of type safety. GetViewById throws that out the
window.

[stevenjb, 2011/11/18 23:04:48]

That's fair. Part of me wonders whether we should just duplicate the button
adding code in the two places we use it instead, but this seems fine for now.

 
  private:
   void UpdateClockText();