[DMC #5] Add SpellCheckMarkerListImpl

third_party/WebKit/Source/core/editing/markers/SpellCheckMarkerListImpl.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 GenericDocumentMarkerListImpl_h
-#define GenericDocumentMarkerListImpl_h
+#ifndef SpellCheckMarkerListImpl_h
+#define SpellCheckMarkerListImpl_h
 
 #include "core/editing/markers/DocumentMarkerList.h"
 
 namespace blink {
 
-class RenderedDocumentMarker;
-
-// Temporary implementation of DocumentMarkerList that can handle
-// DocumentMarkers of all MarkerTypes. This will be removed once we have
-// specialized implementations for every MarkerType.
-class GenericDocumentMarkerListImpl final : public DocumentMarkerList {
+// Implementation of DocumentMarkerList for Spelling/Grammar markers.
+// Markers with touching endpoints are merged on insert. Markers are kept sorted
+// by start offset in order to be able to do this efficiently.
+class CORE_EXPORT SpellCheckMarkerListImpl : public DocumentMarkerList {
  public:
-  GenericDocumentMarkerListImpl() = default;
+  SpellCheckMarkerListImpl() = default;
 
   // DocumentMarkerList implementations
   bool IsEmpty() const final;
 
   void Add(DocumentMarker*) final;
   void Clear() final;
 
   const HeapVector<Member<RenderedDocumentMarker>>& GetMarkers() const final;
 
   bool MoveMarkers(int length, DocumentMarkerList* dst_list) final;
   bool RemoveMarkers(unsigned start_offset, int length) final;
-  bool RemoveMarkersUnderWords(const String& node_text,
-                               const Vector<String>& words) final;
   bool ShiftMarkers(unsigned offset,
                     unsigned old_length,
                     unsigned new_length) final;
+  DECLARE_VIRTUAL_TRACE();

[Xiaocheng, 2017/04/27 22:04:38]

Please make sure all other impls use DECLARE_VIRTUAL_TRACE

 
-  DECLARE_TRACE();
+  // SpellCheckMarkerListImpl-specific
+  // Returns true if a marker was removed, false otherwise.
+  bool RemoveMarkersUnderWords(const String& node_text,
+                               const Vector<String>& words);
 
  private:
   HeapVector<Member<RenderedDocumentMarker>> markers_;
 
-  DISALLOW_COPY_AND_ASSIGN(GenericDocumentMarkerListImpl);
+  DISALLOW_COPY_AND_ASSIGN(SpellCheckMarkerListImpl);
 };
 
 }  // namespace blink
 
-#endif  // GenericDocumentMarkerListImpl_h
+#endif  // SpellCheckMarkerListImpl_h