Split general DocumentMarkerController::AddMarker() method into spelling/grammar versions

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

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights
  * reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  * Copyright (C) Research In Motion Limited 2010. All rights reserved.
@@ skipping 105 matching lines @@
 DocumentMarkerController::DocumentMarkerController(Document& document)
     : possibly_existing_marker_types_(0), document_(&document) {
   SetContext(&document);
 }
 
 void DocumentMarkerController::Clear() {
   markers_.clear();
   possibly_existing_marker_types_ = 0;
 }
 
-void DocumentMarkerController::AddMarker(const Position& start,
-                                         const Position& end,
-                                         DocumentMarker::MarkerType type,
-                                         const String& description) {
+void DocumentMarkerController::AddSpellingMarker(const Position& start,
+                                                 const Position& end,
+                                                 const String& description) {
+  AddSpellCheckMarker(start, end, DocumentMarker::kSpelling, description);
+}
+
+void DocumentMarkerController::AddGrammarMarker(const Position& start,
+                                                const Position& end,
+                                                const String& description) {
+  AddSpellCheckMarker(start, end, DocumentMarker::kGrammar, description);
+}
+
+void DocumentMarkerController::AddSpellCheckMarker(
+    const Position& start,
+    const Position& end,
+    DocumentMarker::MarkerType type,
+    const String& description) {
+  DCHECK(type == DocumentMarker::kSpelling || type == DocumentMarker::kGrammar);

[yosin_UTC9, 2017/05/23 09:35:04]

nit: s/);/) << type;/

   // Use a TextIterator to visit the potentially multiple nodes the range
   // covers.
   for (TextIterator marked_text(start, end); !marked_text.AtEnd();
        marked_text.Advance()) {
     AddMarker(marked_text.CurrentContainer(),
               new DocumentMarker(
                   type, marked_text.StartOffsetInCurrentContainer(),
                   marked_text.EndOffsetInCurrentContainer(), description));
   }
 }
@@ skipping 592 matching lines @@
 }
 
 }  // namespace blink
 
 #ifndef NDEBUG
 void showDocumentMarkers(const blink::DocumentMarkerController* controller) {
   if (controller)
     controller->ShowMarkers();
 }
 #endif