DocumentMarkerList refactor as an interface

third_party/WebKit/Source/core/editing/markers/DocumentMarkerListUtils.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 DocumentMarkerListUtils_h
+#define DocumentMarkerListUtils_h
+
+#include "core/editing/markers/DocumentMarkerList.h"
+
+namespace blink {
+
+class DocumentMarkerListUtils {

[yosin_UTC9, 2017/04/07 03:16:22]

Please avoid to use abbrev name, it should call |Utilities|.
Please avoid to use |Utilities|, it doesn't provide enough information.
And, it says this class is a kitchen sink.

At this time, we can call DocumentMarkerListEditor or Manipulator.

+  using MarkerList = HeapVector<Member<DocumentMarker>>;

[Xiaocheng, 2017/04/07 00:06:33]

This is the same as DocumentMarkerVector.

+
+ public:
+  static DocumentMarkerList::iterator
+  getPosOfFirstMarkerNotEndingBeforeInSortedList(MarkerList*,
+                                                 size_t startOffset);
+
+  static void appendMarkersToInputList(const MarkerList*,
+                                       DocumentMarkerVector* inputList);
+
+  static DocumentMarkerList::DidCopyMarkerOrNot copyUnsortedMarkers(
+      MarkerList*,
+      unsigned startOffset,
+      int length,
+      DocumentMarkerList* dstList,
+      int delta);
+  static DocumentMarkerList::DidCopyMarkerOrNot copySortedMarkers(
+      MarkerList*,
+      unsigned startOffset,
+      int length,
+      DocumentMarkerList* dstList,
+      int delta);
+
+  static DocumentMarkerList::DidRemoveMarkerOrNot removeUnsortedMarkers(
+      MarkerList*,
+      unsigned startOffset,
+      int length,
+      bool shouldRemovePartiallyOverlappingMarkers);
+  static DocumentMarkerList::DidRemoveMarkerOrNot removeSortedMarkers(
+      MarkerList*,
+      unsigned startOffset,
+      int length,
+      bool shouldRemovePartiallyOverlappingMarkers);
+
+  static DocumentMarkerList::DidShiftMarkerOrNot shiftUnsortedMarkers(
+      MarkerList*,
+      unsigned offset,
+      unsigned oldLength,
+      unsigned newLength);
+  static DocumentMarkerList::DidShiftMarkerOrNot shiftSortedMarkers(
+      MarkerList*,
+      unsigned offset,
+      unsigned oldLength,
+      unsigned newLength);
+
+ private:
+  DocumentMarkerListUtils();
+};
+
+}  // namespace blink
+
+#endif  // DocumentMarkerListUtils_h