Make spacing between uber tray items consistent with UI spec.

ash/system/ime/tray_ime.cc

Index: ash/system/ime/tray_ime.cc
diff --git a/ash/system/ime/tray_ime.cc b/ash/system/ime/tray_ime.cc
index 03bf41034db91de69e4f304559cd765cdef26062..c8fce5c3ec4fa739d392d0431ce2f4b49387430f 100644
--- a/ash/system/ime/tray_ime.cc
+++ b/ash/system/ime/tray_ime.cc
@@ -240,6 +240,9 @@ void TrayIME::UpdateTrayLabel(const IMEInfo& current, size_t count) {
       tray_label_->label()->SetText(current.short_name);
     }
     tray_label_->SetVisible(count > 1);
+    SetTrayLabelItemBorder(tray_label_,
+        ash::Shell::GetInstance()->system_tray()->shelf_alignment());
+    tray_label_->Layout();
   }
 }
 
@@ -248,8 +251,8 @@ views::View* TrayIME::CreateTrayView(user::LoginStatus status) {
   tray_label_ = new TrayItemView;
   tray_label_->CreateLabel();
   SetupLabelForTray(tray_label_->label());
-  tray_label_->label()->set_border(
-      views::Border::CreateEmptyBorder(0, 2, 0, 2));
+  UpdateAfterShelfAlignmentChange(
+      ash::Shell::GetInstance()->system_tray()->shelf_alignment());

[sadrul, 2012/07/19 16:46:28]

Looks like this is necessary in all CreateTrayView. I would suggest either
sending the alignment as a param, or calling UpdateAfterShelfAlignmentChange
after the call to CreateTrayView in SystemTray.

[jennyz, 2012/07/19 18:33:33]

Done.

   return tray_label_;
 }
 
@@ -298,6 +301,10 @@ void TrayIME::DestroyNotificationView() {
 void TrayIME::UpdateAfterLoginStatusChange(user::LoginStatus status) {
 }
 
+void TrayIME::UpdateAfterShelfAlignmentChange(ShelfAlignment alignment) {
+  SetTrayLabelItemBorder(tray_label_, alignment);

[sadrul, 2012/07/19 16:46:28]

This also seems to be necessary for all items. So I would rather just call
SetTrayLabelItemBorder from the default implementation, instead of overriding
this for every item.

[jennyz, 2012/07/19 18:33:33]

This may require some more refactoring, since SystemTrayItem does not own the
tray_view from CreateTrayView, and each kind of tray may create different kind
of view object. TrayImageItem does cover a lot of tray items, so that we only
need to handle UpdateAfterShelfAlignment in that class.   But others like
TrayUser, TrayNetwork, TrayPower, TrayIME, TrayDate, etc, each create its own
tray_view, and knows what kind of  view it created by themselves, it could be
either an image or label. So it is hard to handle in the default handler in
parent class. One possible refactoring may be to add some base class like the
way we did for TrayImageItem for label tray item, and handle the default
UpdateAfterShelfAlignment there, any suggestion? And do we want to do this for
this cl?

+}
+
 void TrayIME::OnIMERefresh(bool show_message) {
   SystemTrayDelegate* delegate = Shell::GetInstance()->tray_delegate();
   IMEInfoList list;