Add DocumentMarker::createGrammarMarker() and createSpellingMarker()

third_party/WebKit/Source/core/editing/markers/DocumentMarker.h

Index: third_party/WebKit/Source/core/editing/markers/DocumentMarker.h
diff --git a/third_party/WebKit/Source/core/editing/markers/DocumentMarker.h b/third_party/WebKit/Source/core/editing/markers/DocumentMarker.h
index f09a4f3b36d02385ba9e69fc788a45ec18c810de..c81da8816b78ce285b764edf29d471604238b10c 100644
--- a/third_party/WebKit/Source/core/editing/markers/DocumentMarker.h
+++ b/third_party/WebKit/Source/core/editing/markers/DocumentMarker.h
@@ -139,15 +139,18 @@ class CORE_EXPORT DocumentMarker : public GarbageCollected<DocumentMarker> {
                                                  Color underlineColor,
                                                  bool thick,
                                                  Color backgroundColor);
+  static DocumentMarker* createGrammarMarker(

[rlanday, 2017/03/22 21:07:00]

I think we should pick whether we want to order these methods alphabetically or
in order of the MarkerType enum, and use the same ordering here and in
DocumentMarkerController for the add*Marker() methods

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

My personal preference is alphabetical order, since we can do without context.
But, there are many opinions.

Please do reordering if you want in another CL.

+      unsigned startOffset,
+      unsigned endOffset,
+      const String& description = emptyString);
+  static DocumentMarker* createSpellingMarker(
+      unsigned startOffset,
+      unsigned endOffset,
+      const String& description = emptyString);
   static DocumentMarker* createTextMatchMarker(unsigned startOffset,
                                                unsigned endOffset,
                                                bool activeMatch);
 
-  DocumentMarker(MarkerType,
-                 unsigned startOffset,
-                 unsigned endOffset,
-                 const String& description);
-
   DocumentMarker(const DocumentMarker&);
 
   MarkerType type() const { return m_type; }