Don't show touch handles when selection hidden

third_party/WebKit/Source/core/frame/FrameView.cpp

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Dirk Mueller <mueller@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  *           (C) 2006 Graham Dennis (graham.dennis@gmail.com)
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
@@ skipping 2051 matching lines @@
   }
 }
 
 bool FrameView::ComputeCompositedSelection(LocalFrame& frame,
                                            CompositedSelection& selection) {
   if (!frame.View() || frame.View()->ShouldThrottleRendering())
     return false;
 
   const VisibleSelection& visible_selection =
       frame.Selection().ComputeVisibleSelectionInDOMTree();
-  if (visible_selection.IsNone() || !frame.Selection().IsHandleVisible())
+  if (!frame.Selection().IsHandleVisible() || frame.Selection().IsHidden())

[yosin_UTC9, 2017/05/16 01:13:09]

Thanks for changing order. |FS::IsHidden()| is not cheap function.

     return false;
 
   // Non-editable caret selections lack any kind of UI affordance, and
   // needn't be tracked by the client.
   if (visible_selection.IsCaret() && !visible_selection.IsContentEditable())
     return false;
 
   VisiblePosition visible_start(visible_selection.VisibleStart());
   RenderedPosition rendered_start(visible_start);
   rendered_start.PositionInGraphicsLayerBacking(selection.start, true);
@@ skipping 3316 matching lines @@
 void FrameView::SetAnimationHost(
     std::unique_ptr<CompositorAnimationHost> host) {
   animation_host_ = std::move(host);
 }
 
 LayoutUnit FrameView::CaretWidth() const {
   return LayoutUnit(GetChromeClient()->WindowToViewportScalar(1));
 }
 
 }  // namespace blink