Use the correct launcher assets for shelf alignment.

ash/launcher/launcher_button.cc

 // 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.
 
 #include "ash/launcher/launcher_button.h"
 
 #include <algorithm>
 
 #include "ash/launcher/launcher_button_host.h"
+#include "ash/wm/shelf_layout_manager.h"
 #include "grit/ash_resources.h"
 #include "skia/ext/image_operations.h"
 #include "ui/base/accessibility/accessible_view_state.h"
 #include "ui/base/animation/animation_delegate.h"
 #include "ui/base/animation/throb_animation.h"
 #include "ui/base/events/event_constants.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/compositor/layer.h"
 #include "ui/compositor/scoped_layer_animation_settings.h"
 #include "ui/gfx/canvas.h"
@@ skipping 85 matching lines @@
 }
 
 bool LauncherButton::IconView::HitTestRect(const gfx::Rect& rect) const {
   // Return false so that LauncherButton gets all the mouse events.
   return false;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // LauncherButton
 
-LauncherButton* LauncherButton::Create(views::ButtonListener* listener,
-                                       LauncherButtonHost* host) {
-  LauncherButton* button = new LauncherButton(listener, host);
+LauncherButton* LauncherButton::Create(
+    views::ButtonListener* listener,
+    LauncherButtonHost* host,
+    ShelfLayoutManager* shelf_layout_manager) {
+  LauncherButton* button =
+      new LauncherButton(listener, host, shelf_layout_manager);
   button->Init();
   return button;
 }
 
 LauncherButton::LauncherButton(views::ButtonListener* listener,
-                               LauncherButtonHost* host)
+                               LauncherButtonHost* host,
+                               ShelfLayoutManager* shelf_layout_manager)
     : CustomButton(listener),
       host_(host),
       icon_view_(NULL),
       bar_(new BarView),
-      state_(STATE_NORMAL) {
+      state_(STATE_NORMAL),
+      shelf_layout_manager_(shelf_layout_manager) {
   set_accessibility_focusable(true);
 
   const gfx::ShadowValue kShadows[] = {
     gfx::ShadowValue(gfx::Point(0, 2), 0, SkColorSetARGB(0x1A, 0, 0, 0)),
     gfx::ShadowValue(gfx::Point(0, 3), 1, SkColorSetARGB(0x1A, 0, 0, 0)),
     gfx::ShadowValue(gfx::Point(0, 0), 1, SkColorSetARGB(0x54, 0, 0, 0)),
   };
   icon_shadows_.assign(kShadows, kShadows + arraysize(kShadows));
 
   AddChildView(bar_);
@@ skipping 123 matching lines @@
 void LauncherButton::GetAccessibleState(ui::AccessibleViewState* state) {
   state->role = ui::AccessibilityTypes::ROLE_PUSHBUTTON;
   state->name = host_->GetAccessibleName(this);
 }
 
 void LauncherButton::Layout() {
   const gfx::Rect button_bounds(GetContentsBounds());
   int x_offset = 0, y_offset = 0;
   gfx::Rect icon_bounds;
 
-  if (host_->GetShelfAlignment() == SHELF_ALIGNMENT_BOTTOM) {
+  if (shelf_layout_manager_->GetAlignment() == SHELF_ALIGNMENT_BOTTOM) {
     icon_bounds.SetRect(
         button_bounds.x(), button_bounds.y() + kIconPad,
         button_bounds.width(), kIconSize);
     if (ShouldHop(state_))
       y_offset -= kHopSpacing;
   } else {
     icon_bounds.SetRect(
         button_bounds.x() + kIconPad, button_bounds.y(),
         kIconSize, button_bounds.height());
     if (!ShouldHop(state_))
       x_offset += kHopSpacing;
   }
 
-  if (host_->GetShelfAlignment() == SHELF_ALIGNMENT_LEFT)
+  if (shelf_layout_manager_->GetAlignment() == SHELF_ALIGNMENT_LEFT)
     x_offset = -x_offset;
   icon_bounds.Offset(x_offset, y_offset);
   icon_view_->SetBoundsRect(icon_bounds);
   bar_->SetBoundsRect(GetContentsBounds());
 }
 
 void LauncherButton::ChildPreferredSizeChanged(views::View* child) {
   Layout();
 }
 
@@ skipping 43 matching lines @@
   icon_view_->SetVerticalAlignment(views::ImageView::LEADING);
 
   AddChildView(icon_view_);
 }
 
 LauncherButton::IconView* LauncherButton::CreateIconView() {
   return new IconView;
 }
 
 bool LauncherButton::IsShelfHorizontal() const {
-  return host_->GetShelfAlignment() == SHELF_ALIGNMENT_BOTTOM;
+  return shelf_layout_manager_->IsHorizontalAlignment();
 }
 
 void LauncherButton::UpdateState() {
   if (state_ == STATE_NORMAL) {
     bar_->SetVisible(false);
   } else {
     int bar_id;
-    if (IsShelfHorizontal()) {
-      if (state_ & STATE_ACTIVE)
-        bar_id = IDR_AURA_LAUNCHER_UNDERLINE_BOTTOM_ACTIVE;
-      else if (state_ & (STATE_HOVERED | STATE_FOCUSED | STATE_ATTENTION))
-        bar_id = IDR_AURA_LAUNCHER_UNDERLINE_BOTTOM_HOVER;
-      else
-        bar_id = IDR_AURA_LAUNCHER_UNDERLINE_BOTTOM_RUNNING;
+    if (state_ & STATE_ACTIVE) {
+      bar_id = shelf_layout_manager_->SelectValueForShelfAlignment(
+          IDR_AURA_LAUNCHER_UNDERLINE_BOTTOM_ACTIVE,
+          IDR_AURA_LAUNCHER_UNDERLINE_LEFT_ACTIVE,
+          IDR_AURA_LAUNCHER_UNDERLINE_RIGHT_ACTIVE);
+    } else if (state_ & (STATE_HOVERED | STATE_FOCUSED | STATE_ATTENTION)) {
+      bar_id = shelf_layout_manager_->SelectValueForShelfAlignment(
+          IDR_AURA_LAUNCHER_UNDERLINE_BOTTOM_HOVER,
+          IDR_AURA_LAUNCHER_UNDERLINE_LEFT_HOVER,
+          IDR_AURA_LAUNCHER_UNDERLINE_RIGHT_HOVER);
     } else {
-      if (state_ & STATE_ACTIVE)
-        bar_id = IDR_AURA_LAUNCHER_UNDERLINE_LEFT_ACTIVE;
-      else if (state_ & (STATE_HOVERED | STATE_FOCUSED | STATE_ATTENTION))
-        bar_id = IDR_AURA_LAUNCHER_UNDERLINE_LEFT_HOVER;
-      else
-        bar_id = IDR_AURA_LAUNCHER_UNDERLINE_LEFT_RUNNING;
+      bar_id = shelf_layout_manager_->SelectValueForShelfAlignment(
+          IDR_AURA_LAUNCHER_UNDERLINE_BOTTOM_RUNNING,
+          IDR_AURA_LAUNCHER_UNDERLINE_LEFT_RUNNING,
+          IDR_AURA_LAUNCHER_UNDERLINE_RIGHT_RUNNING);
     }
-
     ResourceBundle& rb = ResourceBundle::GetSharedInstance();
     bar_->SetImage(rb.GetImageNamed(bar_id).ToImageSkia());
     bar_->SetVisible(true);
   }
+  bool rtl = base::i18n::IsRTL();
+  bar_->SetHorizontalAlignment(
+      shelf_layout_manager_->SelectValueForShelfAlignment(
+          views::ImageView::CENTER,
+          rtl ? views::ImageView::TRAILING : views::ImageView::LEADING,
+          rtl ? views::ImageView::LEADING : views::ImageView::TRAILING));
+  bar_->SetVerticalAlignment(
+      shelf_layout_manager_->SelectValueForShelfAlignment(
+          views::ImageView::TRAILING,
+          views::ImageView::CENTER,
+          views::ImageView::CENTER));
 
-  switch (host_->GetShelfAlignment()) {
+  switch (shelf_layout_manager_->GetAlignment()) {
     case SHELF_ALIGNMENT_BOTTOM:
       bar_->SetHorizontalAlignment(views::ImageView::CENTER);
       bar_->SetVerticalAlignment(views::ImageView::TRAILING);
       break;
     case SHELF_ALIGNMENT_LEFT:
       bar_->SetHorizontalAlignment(
           base::i18n::IsRTL() ? views::ImageView::TRAILING :
                                 views::ImageView::LEADING);
       bar_->SetVerticalAlignment(views::ImageView::CENTER);
       break;
     case SHELF_ALIGNMENT_RIGHT:
       bar_->SetHorizontalAlignment(
           base::i18n::IsRTL() ? views::ImageView::LEADING :
                                 views::ImageView::TRAILING);
       bar_->SetVerticalAlignment(views::ImageView::CENTER);
       break;
   }
 
+  // Force bar to layout as alignment may have changed but not bounds.
+  bar_->Layout();
   Layout();
+  bar_->SchedulePaint();
   SchedulePaint();
 }
 
 }  // namespace internal
 }  // namespace ash