boundaryTextInserted/Removed() should not call markValid() for irrelavent changes

third_party/WebKit/Source/core/dom/RangeTest.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/dom/Range.h"
 
 #include "bindings/core/v8/ExceptionStatePlaceholder.h"
 #include "core/dom/Element.h"
 #include "core/dom/NodeList.h"
 #include "core/dom/Text.h"
+#include "core/editing/FrameSelection.h"

[yosin_UTC9, 2016/08/19 06:02:43]

nit: RangeTest.cpp should not include files in "include/" and "frame/"

[Xiaocheng, 2016/08/19 06:37:45]

Whoops, I forgot to remove them.

+#include "core/frame/LocalFrame.h"
 #include "core/html/HTMLBodyElement.h"
 #include "core/html/HTMLDocument.h"
 #include "core/html/HTMLElement.h"
 #include "core/html/HTMLHtmlElement.h"
 #include "platform/heap/Handle.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "wtf/Compiler.h"
 #include "wtf/RefPtr.h"
 #include "wtf/text/AtomicString.h"
 
@@ skipping 127 matching lines @@
 
     Document* anotherDocument = Document::create();
     anotherDocument->appendChild(foo);
 
     EXPECT_EQ(bar, range->startContainer());
     EXPECT_EQ(0, range->startOffset());
     EXPECT_EQ(foo, range->endContainer());
     EXPECT_EQ(1, range->endOffset());
 }
 
+// Regression test for crbug.com/639184
+TEST_F(RangeTest, NotMarkedValidByIrrelevantTextInsert)
+{
+    document().body()->setInnerHTML("<div><span id=span1>foo</span>bar<span id=span2>baz</span></div>", ASSERT_NO_EXCEPTION);
+
+    Element* div = document().querySelector("div");
+    Element* span1 = document().getElementById("span1");
+    Element* span2 = document().getElementById("span2");
+    Text* text = toText(div->childNodes()->item(1));
+
+    Range* range = Range::create(document(), span2, 0, div, 3);
+
+    div->removeChild(span1, ASSERT_NO_EXCEPTION);
+    text->insertData(0, "bar", ASSERT_NO_EXCEPTION);
+
+    EXPECT_TRUE(range->boundaryPointsValid());
+    EXPECT_EQ(span2, range->startContainer());
+    EXPECT_EQ(0, range->startOffset());
+    EXPECT_EQ(div, range->endContainer());
+    EXPECT_EQ(2, range->endOffset());
+}
+
+// Regression test for crbug.com/639184
+TEST_F(RangeTest, NotMarkedValidByIrrelevantTextRemove)
+{
+    document().body()->setInnerHTML("<div><span id=span1>foofoo</span>bar<span id=span2>baz</span></div>", ASSERT_NO_EXCEPTION);
+
+    Element* div = document().querySelector("div");
+    Element* span1 = document().getElementById("span1");
+    Element* span2 = document().getElementById("span2");
+    Text* text = toText(div->childNodes()->item(1));
+
+    Range* range = Range::create(document(), span2, 0, div, 3);
+
+    div->removeChild(span1, ASSERT_NO_EXCEPTION);
+    text->deleteData(0, 3, ASSERT_NO_EXCEPTION);
+
+    EXPECT_TRUE(range->boundaryPointsValid());
+    EXPECT_EQ(span2, range->startContainer());
+    EXPECT_EQ(0, range->startOffset());
+    EXPECT_EQ(div, range->endContainer());
+    EXPECT_EQ(2, range->endOffset());
+}
+
 } // namespace blink