ash: Add vertical spacing to multi-line text in tray.

ash/system/date/date_view.h

 // 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.
 
 #ifndef ASH_SYSTEM_DATE_DATE_VIEW_H_
 #define ASH_SYSTEM_DATE_DATE_VIEW_H_
 #pragma once
 
 #include "ash/system/tray/tray_views.h"
 #include "base/i18n/time_formatting.h"
 #include "base/timer.h"
 #include "ui/views/view.h"
 
 namespace views {
 class Label;
 }
 
 namespace ash {
 namespace internal {
 namespace tray {
 
-// This view is used for both the TrayDate tray icon and the TrayPower popup.
-class DateView : public ActionableView {
+// Abstract base class containing common updating and layout code for the
+// DateView popup and the TimeView tray icon.
+class BaseDateTimeView : public ActionableView {

[Daniel Erat, 2012/04/27 23:22:06]

Let me know if you think that this is ugly and would prefer composition (e.g.
some kind of updater class) to inheritance here.

[sadrul, 2012/04/30 15:05:52]

This looks good.

  public:
-  enum TimeType {
-    TIME,
-    DATE
-  };
+  virtual ~BaseDateTimeView();
 
-  explicit DateView(TimeType type);
+  // Updates the displayed text for the current time.
+  void UpdateText();
+
+ protected:
+  BaseDateTimeView();
+
+ private:
+  // Updates labels to display the current time.
+  virtual void UpdateTextInternal(const base::Time& now) = 0;
+
+  // Overridden from views::View.
+  virtual void OnLocaleChanged() OVERRIDE;
+
+  base::OneShotTimer<BaseDateTimeView> timer_;
+
+  DISALLOW_COPY_AND_ASSIGN(BaseDateTimeView);
+};
+
+// Popup view used to display the date and day of week.
+class DateView : public BaseDateTimeView {
+ public:
+  DateView();
   virtual ~DateView();
-  void UpdateTimeFormat();
-  views::Label* label() const { return label_; }
 
   // Sets whether the view is actionable. An actionable date view gives visual
   // feedback on hover, can be focused by keyboard, and clicking/pressing space
   // or enter on the view shows date-related settings.
   void SetActionable(bool actionable);
 
-  void UpdateText();
+ private:
+  // Overridden from BaseDateTimeView.
+  virtual void UpdateTextInternal(const base::Time& now) OVERRIDE;
 
- private:
   // Overridden from ActionableView.
   virtual bool PerformAction(const views::Event& event) OVERRIDE;
 
-  // Overridden from views::View.
-  virtual void OnLocaleChanged() OVERRIDE;
+  views::Label* date_label_;
+  views::Label* day_of_week_label_;
 
-  base::OneShotTimer<DateView> timer_;
-  base::HourClockType hour_type_;
-  TimeType type_;
   bool actionable_;
+
+  DISALLOW_COPY_AND_ASSIGN(DateView);
+};
+
+// Tray view used to display the current time.
+class TimeView : public BaseDateTimeView {
+ public:
+  TimeView();
+  virtual ~TimeView();
+
+  views::Label* label() const { return label_; }
+
+  // Updates the format of the displayed time.
+  void UpdateTimeFormat();
+
+ private:
+  // Overridden from BaseDateTimeView.
+  virtual void UpdateTextInternal(const base::Time& now) OVERRIDE;
+
+  // Overridden from ActionableView.
+  virtual bool PerformAction(const views::Event& event) OVERRIDE;
+
   views::Label* label_;
 
-  DISALLOW_COPY_AND_ASSIGN(DateView);
+  base::HourClockType hour_type_;
+
+  DISALLOW_COPY_AND_ASSIGN(TimeView);
 };
 
 }  // namespace tray
 }  // namespace internal
 }  // namespace ash
 
 #endif  // ASH_SYSTEM_DATE_DATE_VIEW_H_