Swap touch selection handles when they are interchanged

ui/touch_selection/touch_handle_drawable_aura.cc

 // Copyright 2015 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 "ui/touch_selection/touch_handle_drawable_aura.h"
 
 #include "ui/aura/window.h"
 #include "ui/aura/window_targeter.h"
 #include "ui/aura_extra/image_window_delegate.h"
 #include "ui/base/cursor/cursor.h"
@@ skipping 99 matching lines @@
   window_delegate_->SetImage(*image);
 
   // Calculate the relative bounds.
   gfx::Size image_size = image->Size();
   int window_width = image_size.width() + 2 * kSelectionHandlePadding;
   int window_height = image_size.height() + 2 * kSelectionHandlePadding;
   relative_bounds_ =
       gfx::RectF(-kSelectionHandlePadding,
                  kSelectionHandleVerticalVisualOffset - kSelectionHandlePadding,
                  window_width, window_height);
+  gfx::Rect paint_bounds(relative_bounds_.x(), relative_bounds_.y(),
+                         relative_bounds_.width(), relative_bounds_.height());

[mohsen, 2015/12/21 21:05:50]

I think you should've used gfx::ToEnclosingRect() function to convert gfx::RectF
to gfx::Rect.

+  window_->SchedulePaintInRect(paint_bounds);
   UpdateBounds();
 }
 
 void TouchHandleDrawableAura::SetOrigin(const gfx::PointF& position) {
   origin_position_ = position;
   UpdateBounds();
 }
 
 void TouchHandleDrawableAura::SetAlpha(float alpha) {
   if (alpha == alpha_)
@@ skipping 15 matching lines @@
                kSelectionHandlePadding);
   return bounds;
 }
 
 float TouchHandleDrawableAura::GetDrawableHorizontalPaddingRatio() const {
   // Aura does not have any transparent padding for its handle drawable.
   return 0.0f;
 }
 
 }  // namespace ui