Stop dismissing selection handles when selection is kept

third_party/WebKit/Source/core/editing/VisibleSelection.h

 /*
  * Copyright (C) 2004 Apple Computer, Inc.  All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 104 matching lines @@
   bool isNonOrphanedCaretOrRange() const {
     return !isNone() && !start().isOrphan() && !end().isOrphan();
   }
 
   bool isBaseFirst() const { return m_baseIsFirst; }
   bool isDirectional() const { return m_isDirectional; }
   void setIsDirectional(bool isDirectional) { m_isDirectional = isDirectional; }
 
   void appendTrailingWhitespace();
 
+  bool isHandleVisible() const { return m_isHandleVisible; }
+
   // TODO(yosin) Most callers probably don't want these functions, but
   // are using them for historical reasons. |toNormalizedEphemeralRange()|
   // contracts the range around text, and moves the caret most backward
   // visually equivalent position before returning the range/positions.
   EphemeralRangeTemplate<Strategy> toNormalizedEphemeralRange() const;
 
   Element* rootEditableElement() const;
   bool isContentEditable() const;
   bool hasEditableStyle() const;
   bool isContentRichlyEditable() const;
@@ skipping 51 matching lines @@
   // Non-directional ignores m_baseIsFirst and selection always extends on shift
   // + arrow key.
   bool m_isDirectional : 1;
 
   TextGranularity m_granularity;
   // |updateIfNeeded()| uses |m_hasTrailingWhitespace| for word granularity.
   // |m_hasTrailingWhitespace| is set by |appendTrailingWhitespace()|.
   // TODO(yosin): Once we unify start/end and base/extent, we should get rid
   // of |m_hasTrailingWhitespace|.
   bool m_hasTrailingWhitespace : 1;
+
+  bool m_isHandleVisible : 1;  // True if handle should be visible.

[yosin_UTC9, 2017/01/25 03:49:08]

Please don't add |m_isHandleVisible| to |VisibleSelection|.
We hold |m_isHandleVisibile| in |FrameSelection|.

[Changwan Ryu, 2017/02/08 00:40:33]

Done.

 };
 
 extern template class CORE_EXTERN_TEMPLATE_EXPORT
     VisibleSelectionTemplate<EditingStrategy>;
 extern template class CORE_EXTERN_TEMPLATE_EXPORT
     VisibleSelectionTemplate<EditingInFlatTreeStrategy>;
 
 using VisibleSelection = VisibleSelectionTemplate<EditingStrategy>;
 using VisibleSelectionInFlatTree =
     VisibleSelectionTemplate<EditingInFlatTreeStrategy>;
@@ skipping 17 matching lines @@
 
 #ifndef NDEBUG
 // Outside the WebCore namespace for ease of invocation from gdb.
 void showTree(const blink::VisibleSelection&);
 void showTree(const blink::VisibleSelection*);
 void showTree(const blink::VisibleSelectionInFlatTree&);
 void showTree(const blink::VisibleSelectionInFlatTree*);
 #endif
 
 #endif  // VisibleSelection_h