Add DocumentMarker::createGrammarMarker() and createSpellingMarker()

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

Index: third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
diff --git a/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp b/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
index 3a823ce66b060d9b7b3b2e3ce722e44bb6d9cd0b..074b12ecb743fd4247b9af2c46e2697da8380c91 100644
--- a/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
+++ b/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
@@ -64,14 +64,22 @@ void DocumentMarkerController::addMarker(const Position& start,
                                          const Position& end,
                                          DocumentMarker::MarkerType type,
                                          const String& description) {
+  DCHECK(type == DocumentMarker::Grammar || type == DocumentMarker::Spelling);
   // Use a TextIterator to visit the potentially multiple nodes the range
   // covers.
   for (TextIterator markedText(start, end); !markedText.atEnd();
        markedText.advance()) {
-    addMarker(markedText.currentContainer(),
-              new DocumentMarker(
-                  type, markedText.startOffsetInCurrentContainer(),
-                  markedText.endOffsetInCurrentContainer(), description));
+    DocumentMarker* marker;
+    if (type == DocumentMarker::Grammar) {

[yosin_UTC9, 2017/03/23 04:01:27]

Could you add TODO for introducing add{Grammaer,Spelling}Marker()?
We want to expose DocumentMarker::MarkerType.

+      marker = DocumentMarker::createGrammarMarker(
+          markedText.startOffsetInCurrentContainer(),
+          markedText.endOffsetInCurrentContainer(), description);
+    } else {
+      marker = DocumentMarker::createSpellingMarker(
+          markedText.startOffsetInCurrentContainer(),
+          markedText.endOffsetInCurrentContainer(), description);
+    }
+    addMarker(markedText.currentContainer(), marker);
   }
 }