[MD] some tweaks to download shelf

chrome/browser/ui/views/toolbar/app_menu_button.cc

Index: chrome/browser/ui/views/toolbar/app_menu_button.cc
diff --git a/chrome/browser/ui/views/toolbar/app_menu_button.cc b/chrome/browser/ui/views/toolbar/app_menu_button.cc
index 12c0e38dfdc7ac277baa9d7eda618c0c97d26bf7..a9aa56e3f9f5f596dee7e734da0744602257406f 100644
--- a/chrome/browser/ui/views/toolbar/app_menu_button.cc
+++ b/chrome/browser/ui/views/toolbar/app_menu_button.cc
@@ -190,13 +190,8 @@ void AppMenuButton::SetTrailingMargin(int margin) {
 
 gfx::Point AppMenuButton::GetInkDropCenter() const {
   // ToolbarView extends the bounds of the app button to the right in maximized
-  // mode. So instead of using the center point of local bounds, we use the
-  // center point (adjusted for RTL layouts) of the preferred size, which
-  // doesn't change in maximized mode.
-  const int visible_width = GetPreferredSize().width();
-  return gfx::Point(
-      (GetMirroredXWithWidthInView(0, visible_width) + visible_width) / 2,
-      height() / 2);
+  // mode.
+  return image()->bounds().CenterPoint();

[varkha, 2016/03/03 04:39:10]

This seems wrong. I am running with --lang=he in command line and a maximized
browser displays hover / activated ripple in a wrong spot.
Interestingly enough the positioning is wrong before this change but it was
closer to the visible center of the icon meaning there was a slight regression
even before that.

 }
 
 const char* AppMenuButton::GetClassName() const {