Don't ever split DocumentMarkers on remove

third_party/WebKit/Source/core/editing/markers/DocumentMarkerControllerTest.cpp

Index: third_party/WebKit/Source/core/editing/markers/DocumentMarkerControllerTest.cpp
diff --git a/third_party/WebKit/Source/core/editing/markers/DocumentMarkerControllerTest.cpp b/third_party/WebKit/Source/core/editing/markers/DocumentMarkerControllerTest.cpp
index 786a88c4fbfd804b7e5df7face8f779b681836dc..5ca0e06654c5ed1ffd009f39633b2e2784ff4fae 100644
--- a/third_party/WebKit/Source/core/editing/markers/DocumentMarkerControllerTest.cpp
+++ b/third_party/WebKit/Source/core/editing/markers/DocumentMarkerControllerTest.cpp
@@ -265,8 +265,7 @@ TEST_F(DocumentMarkerControllerTest, SetMarkerActiveTest) {
   EXPECT_TRUE(MarkerController().SetMarkersActive(range, true));
 }
 
-TEST_F(DocumentMarkerControllerTest,
-       RemoveStartOfMarkerDoRemovePartiallyOverlapping) {
+TEST_F(DocumentMarkerControllerTest, RemoveStartOfMarker) {
   SetBodyInnerHTML("<b>abc</b>");
   GetDocument().UpdateStyleAndLayout();
   Element* b_element = ToElement(GetDocument().body()->FirstChild());
@@ -278,38 +277,13 @@ TEST_F(DocumentMarkerControllerTest,
 
   // Remove markers that overlap "a"
   marker_range = PlainTextRange(0, 1).CreateRange(*b_element);
-  GetDocument().Markers().RemoveMarkers(
-      marker_range, DocumentMarker::AllMarkers(),
-      DocumentMarkerController::kRemovePartiallyOverlappingMarker);
+  GetDocument().Markers().RemoveMarkers(marker_range,
+                                        DocumentMarker::AllMarkers());
 
   EXPECT_EQ(0u, MarkerController().Markers().size());
 }
 
-TEST_F(DocumentMarkerControllerTest,
-       RemoveStartOfMarkerDontRemovePartiallyOverlapping) {
-  SetBodyInnerHTML("<b>abc</b>");
-  GetDocument().UpdateStyleAndLayout();
-  Element* b_element = ToElement(GetDocument().body()->FirstChild());
-
-  // Add marker under "abc"
-  EphemeralRange marker_range = PlainTextRange(0, 3).CreateRange(*b_element);
-  GetDocument().Markers().AddTextMatchMarker(
-      marker_range, DocumentMarker::MatchStatus::kInactive);
-
-  // Remove portion of marker that overlaps "a"
-  marker_range = PlainTextRange(0, 1).CreateRange(*b_element);
-  GetDocument().Markers().RemoveMarkers(
-      marker_range, DocumentMarker::AllMarkers(),
-      DocumentMarkerController::kDoNotRemovePartiallyOverlappingMarker);
-
-  EXPECT_EQ(1u, MarkerController().Markers().size());
-
-  EXPECT_EQ(1u, MarkerController().Markers()[0]->StartOffset());
-  EXPECT_EQ(3u, MarkerController().Markers()[0]->EndOffset());
-}
-
-TEST_F(DocumentMarkerControllerTest,
-       RemoveMiddleOfMarkerDoRemovePartiallyOverlapping) {
+TEST_F(DocumentMarkerControllerTest, RemoveMiddleOfMarker) {
   SetBodyInnerHTML("<b>abc</b>");
   GetDocument().UpdateStyleAndLayout();
   Element* b_element = ToElement(GetDocument().body()->FirstChild());
@@ -321,41 +295,13 @@ TEST_F(DocumentMarkerControllerTest,
 
   // Remove markers that overlap "b"
   marker_range = PlainTextRange(1, 2).CreateRange(*b_element);
-  GetDocument().Markers().RemoveMarkers(
-      marker_range, DocumentMarker::AllMarkers(),
-      DocumentMarkerController::kRemovePartiallyOverlappingMarker);
+  GetDocument().Markers().RemoveMarkers(marker_range,
+                                        DocumentMarker::AllMarkers());
 
   EXPECT_EQ(0u, MarkerController().Markers().size());
 }
 
-TEST_F(DocumentMarkerControllerTest,
-       RemoveMiddleOfMarkerDontRemovePartiallyOverlapping) {
-  SetBodyInnerHTML("<b>abc</b>");
-  GetDocument().UpdateStyleAndLayout();
-  Element* b_element = ToElement(GetDocument().body()->FirstChild());
-
-  // Add marker under "abc"
-  EphemeralRange marker_range = PlainTextRange(0, 3).CreateRange(*b_element);
-  GetDocument().Markers().AddTextMatchMarker(
-      marker_range, DocumentMarker::MatchStatus::kInactive);
-
-  // Remove portion of marker that overlaps "b"
-  marker_range = PlainTextRange(1, 2).CreateRange(*b_element);
-  GetDocument().Markers().RemoveMarkers(
-      marker_range, DocumentMarker::AllMarkers(),
-      DocumentMarkerController::kDoNotRemovePartiallyOverlappingMarker);
-
-  EXPECT_EQ(2u, MarkerController().Markers().size());
-
-  EXPECT_EQ(0u, MarkerController().Markers()[0]->StartOffset());
-  EXPECT_EQ(1u, MarkerController().Markers()[0]->EndOffset());
-
-  EXPECT_EQ(2u, MarkerController().Markers()[1]->StartOffset());
-  EXPECT_EQ(3u, MarkerController().Markers()[1]->EndOffset());
-}
-
-TEST_F(DocumentMarkerControllerTest,
-       RemoveEndOfMarkerDoRemovePartiallyOverlapping) {
+TEST_F(DocumentMarkerControllerTest, RemoveEndOfMarker) {
   SetBodyInnerHTML("<b>abc</b>");
   GetDocument().UpdateStyleAndLayout();
   Element* b_element = ToElement(GetDocument().body()->FirstChild());
@@ -367,34 +313,10 @@ TEST_F(DocumentMarkerControllerTest,
 
   // Remove markers that overlap "c"
   marker_range = PlainTextRange(2, 3).CreateRange(*b_element);
-  GetDocument().Markers().RemoveMarkers(
-      marker_range, DocumentMarker::AllMarkers(),
-      DocumentMarkerController::kRemovePartiallyOverlappingMarker);
+  GetDocument().Markers().RemoveMarkers(marker_range,
+                                        DocumentMarker::AllMarkers());
 
   EXPECT_EQ(0u, MarkerController().Markers().size());
 }
 
-TEST_F(DocumentMarkerControllerTest,
-       RemoveEndOfMarkerDontRemovePartiallyOverlapping) {
-  SetBodyInnerHTML("<b>abc</b>");
-  GetDocument().UpdateStyleAndLayout();
-  Element* b_element = ToElement(GetDocument().body()->FirstChild());
-
-  // Add marker under "abc"
-  EphemeralRange marker_range = PlainTextRange(0, 3).CreateRange(*b_element);
-  GetDocument().Markers().AddTextMatchMarker(
-      marker_range, DocumentMarker::MatchStatus::kInactive);
-
-  // Remove portion of marker that overlaps "c"
-  marker_range = PlainTextRange(2, 3).CreateRange(*b_element);
-  GetDocument().Markers().RemoveMarkers(
-      marker_range, DocumentMarker::AllMarkers(),
-      DocumentMarkerController::kDoNotRemovePartiallyOverlappingMarker);
-
-  EXPECT_EQ(1u, MarkerController().Markers().size());
-
-  EXPECT_EQ(0u, MarkerController().Markers()[0]->StartOffset());
-  EXPECT_EQ(2u, MarkerController().Markers()[0]->EndOffset());
-}
-
 }  // namespace blink