[DMC #24] Add SpellingMarker and GrammarMarker (subclasses of DocumentMarker)

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

+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef SpellCheckMarker_h
+#define SpellCheckMarker_h
+
+#include "core/editing/markers/DocumentMarker.h"
+
+namespace blink {
+
+class CORE_EXPORT SpellCheckMarker : public DocumentMarker {

[yosin_UTC9, 2017/06/01 03:29:12]

Please add a class comment.

+ public:
+  SpellCheckMarker(DocumentMarker::MarkerType,

[Xiaocheng, 2017/05/31 22:17:16]

The ctor should be protected.

+                   unsigned start_offset,
+                   unsigned end_offset,
+                   const String& description);
+
+  const String& Description() const;

[yosin_UTC9, 2017/06/01 03:29:12]

nit: Since this simple getter, you can implement here.

+
+ private:
+  const String description_;
+};

[yosin_UTC9, 2017/06/01 03:29:12]

nit: Could you add DISALLOW_COPY_AND_ASSING()?

+
+DEFINE_TYPE_CASTS(SpellCheckMarker,
+                  DocumentMarker,
+                  marker,
+                  marker->GetType() == DocumentMarker::kSpelling ||
+                      marker->GetType() == DocumentMarker::kGrammar,
+                  marker.GetType() == DocumentMarker::kSpelling ||
+                      marker.GetType() == DocumentMarker::kGrammar);
+
+}  // namespace blink
+
+#endif