Change default ink drop center points to center of contents bounds

chrome/browser/ui/views/tabs/tab.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 "chrome/browser/ui/views/tabs/tab.h"
 
 #include <stddef.h>
 #include <limits>
 #include <utility>
 
@@ skipping 302 matching lines @@
 
  private:
   // views::MaskedTargeterDelegate:
   View* TargetForRect(View* root, const gfx::Rect& rect) override {
     CHECK_EQ(root, this);
 
     if (!views::UsePointBasedTargeting(rect))
       return ViewTargeterDelegate::TargetForRect(root, rect);
 
     // Ignore the padding set on the button.
-    gfx::Rect contents_bounds = GetContentsBounds();
-    contents_bounds.set_x(GetMirroredXForRect(contents_bounds));
+    gfx::Rect contents_bounds = GetMirroredRect(GetContentsBounds());
 
 #if defined(USE_AURA)
     // Include the padding in hit-test for touch events.
     // TODO(pkasting): It seems like touch events would generate rects rather
     // than points and thus use the TargetForRect() call above.  If this is
     // reached, it may be from someone calling GetEventHandlerForPoint() while a
     // touch happens to be occurring.  In such a case, maybe we don't want this
     // code to run?  It's possible this block should be removed, or maybe this
     // whole function deleted.  Note that in these cases, we should probably
     // also remove the padding on the close button bounds (see Tab::Layout()),
     // as it will be pointless.
     if (aura::Env::GetInstance()->is_touch_down())
       contents_bounds = GetLocalBounds();
 #endif
 
     return contents_bounds.Intersects(rect) ? this : parent();
   }
 
   // We need to define this so hit-testing won't include the border region.
   bool GetHitTestMask(gfx::Path* mask) const override {
-    gfx::Rect button_bounds(GetContentsBounds());
-    button_bounds.set_x(GetMirroredXForRect(button_bounds));
-    mask->addRect(gfx::RectToSkRect(button_bounds));
+    mask->addRect(gfx::RectToSkRect(GetMirroredRect(GetContentsBounds())));
     return true;
   }
 
   Tab* tab_;
 
   DISALLOW_COPY_AND_ASSIGN(TabCloseButton);
 };
 
 ////////////////////////////////////////////////////////////////////////////////
 // ThrobberView
@@ skipping 904 matching lines @@
   const int kIndicatorRadius = 3;
   const int indicator_x = GetMirroredXWithWidthInView(
       favicon_bounds_.right() - kIndicatorRadius, kIndicatorRadius * 2);
   const int indicator_y = favicon_bounds_.bottom() - kIndicatorRadius;
   canvas->DrawCircle(gfx::Point(indicator_x + kIndicatorRadius,
                                 indicator_y + kIndicatorRadius),
                      kIndicatorRadius, indicator_flags);
 }
 
 void Tab::PaintIcon(gfx::Canvas* canvas) {
-  gfx::Rect bounds = favicon_bounds_;
-  bounds.set_x(GetMirroredXForRect(bounds));
+  gfx::Rect bounds = GetMirroredRect(favicon_bounds_);
   bounds.Offset(0, favicon_hiding_offset_);
   bounds.Intersect(GetContentsBounds());
   if (bounds.IsEmpty())
     return;
 
   // Throbber will do its own painting.
   if (data().network_state != TabRendererData::NETWORK_STATE_NONE &&
       data().network_state != TabRendererData::NETWORK_STATE_ERROR) {
     return;
   }
@@ skipping 155 matching lines @@
 }
 
 void Tab::ScheduleIconPaint() {
   gfx::Rect bounds = favicon_bounds_;
   if (bounds.IsEmpty())
     return;
 
   // Extends the area to the bottom when the crash animation is in progress.
   if (crash_icon_animation_->is_animating())
     bounds.set_height(height() - bounds.y());
-  bounds.set_x(GetMirroredXForRect(bounds));
-  SchedulePaintInRect(bounds);
+  SchedulePaintInRect(GetMirroredRect(bounds));
 }
 
 Tab::BackgroundCache::BackgroundCache() = default;
 Tab::BackgroundCache::~BackgroundCache() = default;