touch ui: Do not hide text selection handles when screen is rotated.

ui/views/touchui/touch_selection_controller_impl.cc

 // Copyright (c) 2013 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/views/touchui/touch_selection_controller_impl.h"
 
 #include "base/macros.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/time/time.h"
 #include "ui/aura/client/cursor_client.h"
@@ skipping 625 matching lines @@
   client_view_->ExecuteCommand(command_id, event_flags);
 }
 
 void TouchSelectionControllerImpl::RunContextMenu() {
   // Context menu should appear centered on top of the selected region.
   const gfx::Rect rect = GetQuickMenuAnchorRect();
   const gfx::Point anchor(rect.CenterPoint().x(), rect.y());
   client_view_->OpenContextMenu(anchor);
 }
 
-void TouchSelectionControllerImpl::OnAncestorWindowTransformed(
-    aura::Window* window,
-    aura::Window* ancestor) {
-  client_view_->DestroyTouchSelection();
-}
-

[sammiequon, 2017/06/06 19:01:27]

Would removing this have other undesired effects?

[mohsen, 2017/06/07 18:20:38]

This was added for Project Athena where we had textfields that could move around
(crbug/407659). That project was cancelled and I'm not aware of any existing
code that applies such transforms on a focused textfield.

[sammiequon, 2017/06/07 18:53:49]

Acknowledged. Do you think we should revert
https://codereview.chromium.org/558203002 as well, since this is the only
instance of OnAncestorWindowTransformed?

 void TouchSelectionControllerImpl::OnWidgetClosing(Widget* widget) {
   DCHECK_EQ(client_widget_, widget);
   client_widget_->RemoveObserver(this);
   client_widget_ = nullptr;
 }
 
 void TouchSelectionControllerImpl::OnWidgetBoundsChanged(
     Widget* widget,
     const gfx::Rect& new_bounds) {
   DCHECK_EQ(client_widget_, widget);
@@ skipping 126 matching lines @@
 
 views::WidgetDelegateView* TouchSelectionControllerImpl::GetHandle1View() {
   return selection_handle_1_.get();
 }
 
 views::WidgetDelegateView* TouchSelectionControllerImpl::GetHandle2View() {
   return selection_handle_2_.get();
 }
 
 }  // namespace views