Refactor DocumentMarkerController

third_party/WebKit/Source/core/editing/markers/EditingMarkerList.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 EditingMarkerList_h
+#define EditingMarkerList_h
+
+#include "core/editing/markers/DocumentMarkerList.h"
+
+namespace blink {
+
+class DocumentMarker;
+
+class EditingMarkerList : public DocumentMarkerList {
+  WTF_MAKE_NONCOPYABLE(EditingMarkerList);
+
+ public:
+  explicit EditingMarkerList(DocumentMarkerController*);
+
+  // DocumentMarkerList implementations
+  void clear() final;
+
+  bool copyMarkers(unsigned startOffset,
+                   int length,
+                   Node* dstNode,
+                   int delta) const final;
+  void removeMarkers(unsigned startOffset,
+                     int length,
+                     bool shouldRemovePartiallyOverlappingMarkers,
+                     bool* didRemoveMarker) final;
+  bool shiftMarkers(unsigned offset,
+                    unsigned oldLength,
+                    unsigned newLength) final;
+
+  // EditingMarkerList-specific
+  virtual void insert(DocumentMarker*);
+
+  DECLARE_VIRTUAL_TRACE();
+
+ protected:
+  void sortMarkerList();
+
+  bool m_markersAreSorted;
+
+ private:
+  static bool compareByStart(const Member<DocumentMarker>& lhv,

[Xiaocheng, 2017/03/21 03:59:44]

This function should be hidden in an anonymous namespace in
EditingMarkerList.cpp

+                             const Member<DocumentMarker>& rhv);
+};
+
+}  // namespace blink
+
+#endif  // EditingMarkerList_h