Add CompositionMarkerList in preparation for DocumentMarkerController refactor

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

 /*
  * This file is part of the DOM implementation for WebCore.
  *
  * Copyright (C) 2006 Apple Computer, Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 143 matching lines @@
   const String& description() const;
   bool activeMatch() const;
   Color underlineColor() const;
   bool thick() const;
   Color backgroundColor() const;
   DocumentMarkerDetails* details() const;
 
   void setActiveMatch(bool);
   void clearDetails() { m_details.clear(); }
 
+  struct ShiftMarkerResult {
+    // Other classes should not call following setters.

[Xiaocheng, 2017/03/24 03:57:47]

Irrelevant change?

[rlanday, 2017/03/24 17:45:48]

oops, rebase mistake I think

+    unsigned newStartOffset;
+    unsigned newEndOffset;
+    bool shouldRemoveMarker;
+  };
+
+  ShiftMarkerResult getShiftedMarkerPosition(unsigned offset,
+                                             unsigned oldLength,
+                                             unsigned newLength) const;
+
   // Offset modifications are done by DocumentMarkerController.

[Xiaocheng, 2017/03/24 03:57:47]

DocumentMarkerList also calls them.

[rlanday, 2017/03/24 17:45:48]

Yeah, I have the comment updated in my big patch for the whole refactor, I
wasn't sure if I should change it here or not if we're not actually using the
new DocumentMarkerLists yet

[Xiaocheng, 2017/03/24 19:43:49]

Please modify the comment in this patch. Otherwise, the use of these settings in
DML looks confusing.

   // Other classes should not call following setters.
   void setStartOffset(unsigned offset) { m_startOffset = offset; }
   void setEndOffset(unsigned offset) { m_endOffset = offset; }
   void shiftOffsets(int delta);
 
   bool operator==(const DocumentMarker& o) const {
     return type() == o.type() && startOffset() == o.startOffset() &&
            endOffset() == o.endOffset();
   }
 
@@ skipping 22 matching lines @@
   virtual bool isDescription() const { return false; }
   virtual bool isTextMatch() const { return false; }
   virtual bool isComposition() const { return false; }
 
   DEFINE_INLINE_VIRTUAL_TRACE() {}
 };
 
 }  // namespace blink
 
 #endif  // DocumentMarker_h