Views: Add new SetFocusBehavior method.

ash/system/date/date_view.cc

Index: ash/system/date/date_view.cc
diff --git a/ash/system/date/date_view.cc b/ash/system/date/date_view.cc
index 7a96d9c0c0469fe28d82326ab90f319fbdcecc14..bdde34aa52d21a371a1dd299c1f6dd08c3b8e1e0 100644
--- a/ash/system/date/date_view.cc
+++ b/ash/system/date/date_view.cc
@@ -138,7 +138,7 @@ DateView::DateView()
   date_label_->SetEnabledColor(kHeaderTextColorNormal);
   UpdateTextInternal(base::Time::Now());
   AddChildView(date_label_);
-  SetFocusable(false);
+  SetFocusBehavior(FocusBehavior::NEVER);

[tapted, 2016/04/20 04:16:28]

(wait for sky to comment on this, but I wonder if the ash::ActionableView
constructor should be overloaded to take a `focus_behaviour_` argument, so
subclasses don't have to immediately `undo` stuff that's done there)

[sky, 2016/04/20 19:26:02]

I don't feel strongly here. I'm ok either way.

[karandeepb, 2016/04/21 03:16:33]

I don't think we should do this since ActionableView has many subclasses and
there are just 3 instances (DateView, TimeView and VolumeView) of this "undo"
behavior. Have moved the calls for DateView and TimeView to their base class
BaseDateTimeView.

 }
 
 DateView::~DateView() {
@@ -154,7 +154,8 @@ void DateView::SetAction(TrayDate::DateAction action) {
     SchedulePaint();
   }
   action_ = action;
-  SetFocusable(action_ != TrayDate::NONE);
+  SetFocusBehavior(action_ != TrayDate::NONE ? FocusBehavior::ALWAYS
+                                             : FocusBehavior::NEVER);
 }
 
 void DateView::UpdateTimeFormat() {
@@ -226,7 +227,7 @@ TimeView::TimeView(TrayDate::ClockLayout clock_layout)
   SetupLabels();
   UpdateTextInternal(base::Time::Now());
   UpdateClockLayout(clock_layout);
-  SetFocusable(false);
+  SetFocusBehavior(FocusBehavior::NEVER);
 }
 
 TimeView::~TimeView() {