Select All should show handles if they were already present

third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp

 // Copyright 2014 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 "core/editing/FrameSelection.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "core/dom/Document.h"
 #include "core/dom/Element.h"
 #include "core/dom/Text.h"
@@ skipping 290 matching lines @@
 
 TEST_F(FrameSelectionTest, SelectAllWithUnselectableRoot) {
   Element* select = document().createElement("select");
   document().replaceChild(select, document().documentElement());
   selection().selectAll();
   EXPECT_TRUE(selection().isNone()) << "Nothing should be selected if the "
                                        "content of the documentElement is not "
                                        "selctable.";
 }
 
+TEST_F(FrameSelectionTest, SelectAllKeepsHandle) {
+  setBodyContent("<div id=sample>abc</div>");
+  Element* sample = document().getElementById("sample");
+  const Position endOfText(sample->firstChild(), 3);
+  FrameSelection::SetSelectionOptions options =
+      FrameSelection::CloseTyping | FrameSelection::ClearTypingStyle |
+      FrameSelection::HandleVisible;
+  selection().setSelection(
+      SelectionInDOMTree::Builder().collapse(endOfText).build(), options);
+  EXPECT_TRUE(selection().isHandleVisible());
+  selection().selectAll();
+  EXPECT_TRUE(selection().isHandleVisible())
+      << "If handles were present before"
+         "selectAll. Then they should be present"
+         "after it.";
+}
+
 TEST_F(FrameSelectionTest, updateIfNeededAndFrameCaret) {
   setBodyContent("<style id=sample></style>");
   document().setDesignMode("on");
   updateAllLifecyclePhases();
   Element* sample = document().getElementById("sample");
   selection().setSelection(
       SelectionInDOMTree::Builder().collapse(Position(sample, 0)).build());
   EXPECT_EQ(Position(document().body(), 0), selection().start());
   EXPECT_EQ(selection().start(), caretPosition().position());
   document().body()->remove();
   // TODO(yosin): Once lazy canonicalization implemented, selection.start
   // should be Position(HTML, 0).
   EXPECT_EQ(Position(document().documentElement(), 1), selection().start());
   EXPECT_EQ(selection().start(), caretPosition().position());
   document().updateStyleAndLayout();
   selection().updateIfNeeded();
 
   // TODO(yosin): Once lazy canonicalization implemented, selection.start
   // should be Position(HTML, 0).
   EXPECT_EQ(Position(), selection().start())
       << "updateIfNeeded() makes selection to null.";
   EXPECT_EQ(selection().start(), caretPosition().position());
 }
 
 }  // namespace blink