[DMC #3.5] Split up DocumentMarkerListEditor::AddMarker() into two methods

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

Index: third_party/WebKit/Source/core/editing/markers/GenericDocumentMarkerListImpl.cpp
diff --git a/third_party/WebKit/Source/core/editing/markers/GenericDocumentMarkerListImpl.cpp b/third_party/WebKit/Source/core/editing/markers/GenericDocumentMarkerListImpl.cpp
index 587e67e74f607f166383a26bd66370935c324c11..3c4ceb3289296a65bf76646083c1da8976def1b4 100644
--- a/third_party/WebKit/Source/core/editing/markers/GenericDocumentMarkerListImpl.cpp
+++ b/third_party/WebKit/Source/core/editing/markers/GenericDocumentMarkerListImpl.cpp
@@ -14,7 +14,20 @@ bool GenericDocumentMarkerListImpl::IsEmpty() const {
 }
 
 void GenericDocumentMarkerListImpl::Add(DocumentMarker* marker) {
-  DocumentMarkerListEditor::AddMarker(&markers_, marker);
+  switch (marker->GetType()) {
+    case DocumentMarker::kSpelling:
+    case DocumentMarker::kGrammar:
+      DocumentMarkerListEditor::AddMarkerAndMergeOverlapping(&markers_, marker);
+      return;
+    case DocumentMarker::kTextMatch:
+    case DocumentMarker::kComposition:
+      DocumentMarkerListEditor::AddMarkerWithoutMergingOverlapping(&markers_,
+                                                                   marker);
+      return;
+  }
+
+  NOTREACHED();

[yosin_UTC9, 2017/04/28 01:01:28]

Let's print marker->GetType() to add more info to crash log.

e.g. NOTREACHED() << "Unhanded marker type: " << marker->GetType();

+  return;
 }
 
 void GenericDocumentMarkerListImpl::Clear() {