Clean up RTL methods....

chrome/browser/ui/views/bookmark_bar_view.cc

 // Copyright (c) 2011 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/ui/views/bookmark_bar_view.h"
 
 #include <algorithm>
 #include <limits>
 #include <set>
 #include <vector>
@@ skipping 489 matching lines @@
       y = GetBookmarkButton(0)->y();
       h = GetBookmarkButton(0)->height();
     }
 
     // Since the drop indicator is painted directly onto the canvas, we must
     // make sure it is painted in the right location if the locale is RTL.
     gfx::Rect indicator_bounds(x - kDropIndicatorWidth / 2,
                                y,
                                kDropIndicatorWidth,
                                h);
-    indicator_bounds.set_x(MirroredLeftPointForRect(indicator_bounds));
+    indicator_bounds.set_x(GetMirroredXForRect(indicator_bounds));
 
     // TODO(sky/glen): make me pretty!
     canvas->FillRectInt(kDropIndicatorColor, indicator_bounds.x(),
                         indicator_bounds.y(), indicator_bounds.width(),
                         indicator_bounds.height());
   }
 }
 
 bool BookmarkBarView::GetDropFormats(
       int* formats,
@@ skipping 249 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());
+  gfx::Point adjusted_loc(GetMirroredXInView(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(adjusted_loc))
       return model_->GetBookmarkBarNode()->GetChild(i);
   }
 
@@ skipping 580 matching lines @@
                                             bool* is_over_other) {
   DCHECK(model_);
   DCHECK(model_->IsLoaded());
   DCHECK(data.is_valid());
 
   // The drop event uses the screen coordinates while the child Views are
   // always laid out from left to right (even though they are rendered from
   // right-to-left on RTL locales). Thus, in order to make sure the drop
   // coordinates calculation works, we mirror the event's X coordinate if the
   // locale is RTL.
-  int mirrored_x = MirroredXCoordinateInsideView(event.x());
+  int mirrored_x = GetMirroredXInView(event.x());
 
   *index = -1;
   *drop_on = false;
   *is_over_other = *is_over_overflow = false;
 
   if (event.y() < other_bookmarked_button_->y() ||
       event.y() >= other_bookmarked_button_->y() +
                       other_bookmarked_button_->height()) {
     // Mouse isn't over a button.
     return ui::DragDropTypes::DRAG_NONE;
@@ skipping 347 matching lines @@
   // The tooltip is the only way we have to display text explaining the error
   // to the user.
   sync_error_button->SetTooltipText(
       UTF16ToWide(l10n_util::GetStringUTF16(IDS_SYNC_BOOKMARK_BAR_ERROR_DESC)));
   sync_error_button->SetAccessibleName(
       l10n_util::GetStringUTF16(IDS_ACCNAME_SYNC_ERROR_BUTTON));
   sync_error_button->SetIcon(
       *ResourceBundle::GetSharedInstance().GetBitmapNamed(IDR_WARNING));
   return sync_error_button;
 }