Don't show touch handles when selection hidden

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

Index: third_party/WebKit/Source/core/frame/FrameView.cpp
diff --git a/third_party/WebKit/Source/core/frame/FrameView.cpp b/third_party/WebKit/Source/core/frame/FrameView.cpp
index 6f3db2e977c3e9f30d4fc23ea5b43527f05c67d4..e97b0c3745f0bd2dc110a5c7467c9b042d5292d4 100644
--- a/third_party/WebKit/Source/core/frame/FrameView.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameView.cpp
@@ -2063,7 +2063,8 @@ bool FrameView::ComputeCompositedSelection(LocalFrame& frame,
 
   const VisibleSelection& visible_selection =
       frame.Selection().ComputeVisibleSelectionInDOMTree();
-  if (visible_selection.IsNone() || !frame.Selection().IsHandleVisible())
+  if (visible_selection.IsNone() || !frame.Selection().IsHandleVisible() ||
+      frame.Selection().LayoutSelectionForceHide())

[hugoh_UTC2, 2017/04/27 08:53:50]

Thanks a lot! I am rewriting the "hide selection" code right now in
https://codereview.chromium.org/2841093002 . In PS2 I expose a "HasFocus()" that
I think you can use instead of introducing LayoutSelectionForceHide().

[yosin_UTC9, 2017/04/27 09:55:11]

Yes, let's use |FrameSelection::HasFocus()| hugoh@ is introducing.

|LayoutSelectionForceHide()| doesn't represent selection state (has focus or not
focus) correctly.
At this time, we show selection for contenteditable regardless focus.

[hugoh_UTC2, 2017/05/15 15:33:58]

FrameSelection::SelectionHasFocus() and FrameSelection::IsHidden() have landed
and they can hopefully unblock this CL.

BTW, when using them, doesn't !visible_selection.IsNone() become redundant?

     return false;
 
   // Non-editable caret selections lack any kind of UI affordance, and