Fixes regression in showing bookmark menus where the wrong menu would...

chrome/browser/views/bookmark_bar_view.cc

 // Copyright (c) 2009 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 "chrome/browser/views/bookmark_bar_view.h"
 
 #include <algorithm>
 #include <limits>
 #include <set>
 #include <vector>
@@ skipping 782 matching lines @@
   return bookmark_drop_menu_ ? bookmark_drop_menu_->menu() : NULL;
 }
 
 const BookmarkNode* BookmarkBarView::GetNodeForButtonAt(const gfx::Point& loc,
                                                         int* start_index) {
   *start_index = 0;
 
   if (loc.x() < 0 || loc.x() >= width() || loc.y() < 0 || loc.y() >= height())
     return NULL;
 
+  gfx::Point adjusted_loc(MirroredXCoordinateInsideView(loc.x()), loc.y());
+
   // Check the buttons first.
   for (int i = 0; i < GetBookmarkButtonCount(); ++i) {
     views::View* child = GetChildViewAt(i);
     if (!child->IsVisible())
       break;
-    if (child->bounds().Contains(loc))
+    if (child->bounds().Contains(adjusted_loc))
       return model_->GetBookmarkBarNode()->GetChild(i);
   }
 
   // Then the overflow button.
   if (overflow_button_->IsVisible() &&
-      overflow_button_->bounds().Contains(loc)) {
+      overflow_button_->bounds().Contains(adjusted_loc)) {
     *start_index = GetFirstHiddenNodeIndex();
     return model_->GetBookmarkBarNode();
   }
 
   // And finally the other folder.
-  if (other_bookmarked_button_->bounds().Contains(loc))
+  if (other_bookmarked_button_->bounds().Contains(adjusted_loc))
     return model_->other_node();
 
   return NULL;
 }
 
 views::MenuButton* BookmarkBarView::GetMenuButtonForNode(
     const BookmarkNode* node) {
   if (node == model_->other_node())
     return other_bookmarked_button_;
   if (node == model_->GetBookmarkBarNode())
@@ skipping 873 matching lines @@
   // The tooltip is the only way we have to display text explaining the error
   // to the user.
   sync_error_button->SetTooltipText(
       l10n_util::GetString(IDS_SYNC_BOOKMARK_BAR_ERROR_DESC));
   sync_error_button->SetAccessibleName(
       l10n_util::GetString(IDS_ACCNAME_SYNC_ERROR_BUTTON));
   sync_error_button->SetIcon(
       *ResourceBundle::GetSharedInstance().GetBitmapNamed(IDR_WARNING));
   return sync_error_button;
 }