Make PreviousRootInlineBoxCandidatePosition() to ignore nodes without layout object

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

 // Copyright 2015 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/VisibleUnits.h"
 
 #include <ostream>  // NOLINT
 #include "bindings/core/v8/V8BindingForTesting.h"
 #include "core/dom/Text.h"
 #include "core/editing/EditingTestBase.h"
@@ skipping 1891 matching lines @@
 TEST_F(VisibleUnitsTest,
        canonicalizationWithCollapsedSpaceAndIsolatedCombiningCharacter) {
   SetBodyContent("<p>  &#x20E3;</p>");  // Leading space is necessary
 
   Node* paragraph = GetDocument().QuerySelector("p");
   Node* text = paragraph->firstChild();
   Position start = CanonicalPositionOf(Position::BeforeNode(paragraph));
   EXPECT_EQ(Position(text, 2), start);
 }
 
+TEST_F(VisibleUnitsTest,
+       PreviousRootInlineBoxCandidatePositionWithDisplayNone) {
+  SetBodyContent(
+      "<div contenteditable>"
+      "<div id=one>one abc</div>"
+      "<div id=two>two <b id=none style=display:none>def</b> ghi</div>"
+      "</div>");
+  Element* const one = GetDocument().getElementById("one");
+  Element* const two = GetDocument().getElementById("two");
+  const VisiblePosition& visible_position =
+      CreateVisiblePosition(Position::LastPositionInNode(two));
+  EXPECT_EQ(Position(one->firstChild(), 7),
+            PreviousRootInlineBoxCandidatePosition(
+                two->lastChild(), visible_position, kContentIsEditable));
+}
+
 }  // namespace blink