Minor fix to touch selection logic.

views/touchui/touch_selection_controller_impl.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 "views/touchui/touch_selection_controller_impl.h"
 
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/canvas_skia.h"
 #include "ui/gfx/path.h"
 #include "ui/gfx/rect.h"
@@ skipping 143 matching lines @@
 
 void TouchSelectionControllerImpl::SelectionChanged(const gfx::Point& p1,
                                                     const gfx::Point& p2) {
   gfx::Point screen_pos_1(p1);
   View::ConvertPointToScreen(client_view_, &screen_pos_1);
   gfx::Point screen_pos_2(p2);
   View::ConvertPointToScreen(client_view_, &screen_pos_2);
 
   if (dragging_handle_) {
     // We need to reposition only the selection handle that is being dragged.
-    // The other handle stays the same.
-    SelectionHandleView* fixed_handle = selection_handle_1_.get();
-    if (fixed_handle == dragging_handle_)
-      fixed_handle = selection_handle_2_.get();
-
-    gfx::Point fixed_handle_pos = fixed_handle->GetScreenPosition();
-    fixed_handle_pos.Offset(kSelectionHandleRadius, 0);
-
-    if (fixed_handle_pos == screen_pos_1)
-      dragging_handle_->SetScreenPosition(screen_pos_2);
-    else
-      dragging_handle_->SetScreenPosition(screen_pos_1);
+    // The other handle stays the same. Also, the selection handle being dragged
+    // will always be at the end of selection, while the other handle will be at
+    // the start.
+    dragging_handle_->SetScreenPosition(screen_pos_2);
   } else {
     // Check if there is any selection at all.
     if (screen_pos_1 == screen_pos_2) {
       selection_handle_1_->SetVisible(false);
       selection_handle_2_->SetVisible(false);
       return;
     }
 
     if (client_view_->bounds().Contains(p1)) {
       selection_handle_1_->SetScreenPosition(screen_pos_1);
@@ skipping 62 matching lines @@
 bool TouchSelectionControllerImpl::IsSelectionHandle2Visible() {
   return selection_handle_2_->IsVisible();
 }
 
 TouchSelectionController* TouchSelectionController::create(
     TouchSelectionClientView* client_view) {
   return new TouchSelectionControllerImpl(client_view);
 }
 
 }  // namespace views