Implemented system tray UI for new account management.

ash/system/tray/hover_highlight_view.cc

Index: ash/system/tray/hover_highlight_view.cc
diff --git a/ash/system/tray/hover_highlight_view.cc b/ash/system/tray/hover_highlight_view.cc
index bc8f017ab7aa3b98c92129696ed0f113014f92cb..04b0c33528cbb2fb5246ecab6ec77beb722fedae 100644
--- a/ash/system/tray/hover_highlight_view.cc
+++ b/ash/system/tray/hover_highlight_view.cc
@@ -65,20 +65,21 @@ void HoverHighlightView::AddIconAndLabel(const gfx::ImageSkia& image,
 }
 
 views::Label* HoverHighlightView::AddLabel(const base::string16& text,
+                                           gfx::HorizontalAlignment alignment,
                                            gfx::Font::FontStyle style) {
   SetLayoutManager(new views::FillLayout());
   text_label_ = new views::Label(text);
-  int margin = kTrayPopupPaddingHorizontal +
-      kTrayPopupDetailsLabelExtraLeftMargin;
-  int left_margin = 0;
-  int right_margin = 0;
-  if (base::i18n::IsRTL())
-    right_margin = margin;
-  else
-    left_margin = margin;
+  int left_margin = kTrayPopupPaddingHorizontal;
+  int right_margin = kTrayPopupPaddingHorizontal;
+  if (alignment != gfx::ALIGN_CENTER) {

[Mr4D (OOO till 08-26), 2014/03/25 23:54:13]

Looking at this I think you imply which alignments are allowed. As such you
should document that in the header definition.

[dzhioev (left Google), 2014/03/26 13:29:22]

Actually any alignment is allowed now. There was problems with ALIGN_CENTER
before my changes were made. Margin were asymmetric, and if you set alignment of
text_label_ to ALIGN_CENTER it was shifted to the right. I made margin symmetric
for ALIGN_CENTER case.

[Mr4D (OOO till 08-26), 2014/03/26 16:49:53]

In that case: Shouldn't the two cases below then do different things for
ALIGN_LEFT / ALIGN_RIGHT?

[dzhioev (left Google), 2014/03/26 23:48:52]

For ALIGN_LEFT we add kTrayPopupDetailsLabelExtraLeftMargin to the left margin.
I don't know which behavior user will expect if he pass ALIGN_RIGHT here though
(nobody is using ALIGN_RIGHT now, and probably never will). I think we should
add extra padding to the left even for ALIGN_RIGHT.

+    if (base::i18n::IsRTL())
+      right_margin += kTrayPopupDetailsLabelExtraLeftMargin;
+    else
+      left_margin += kTrayPopupDetailsLabelExtraLeftMargin;
+  }
   text_label_->SetBorder(
       views::Border::CreateEmptyBorder(5, left_margin, 5, right_margin));
-  text_label_->SetHorizontalAlignment(gfx::ALIGN_LEFT);
+  text_label_->SetHorizontalAlignment(alignment);
   text_label_->SetFontList(text_label_->font_list().DeriveWithStyle(style));
   // Do not set alpha value in disable color. It will have issue with elide
   // blending filter in disabled state for rendering label text color.
@@ -120,7 +121,7 @@ views::Label* HoverHighlightView::AddCheckableLabel(const base::string16& text,
     SetAccessibleName(text);
     return text_label_;
   }
-  return AddLabel(text, style);
+  return AddLabel(text, gfx::ALIGN_LEFT, style);
 }
 
 void HoverHighlightView::SetExpandable(bool expandable) {