[DMC #4] Add CompositionMarkerListImpl

third_party/WebKit/Source/core/editing/markers/CompositionMarkerListImpl.cpp

Index: third_party/WebKit/Source/core/editing/markers/CompositionMarkerListImpl.cpp
diff --git a/third_party/WebKit/Source/core/editing/markers/CompositionMarkerListImpl.cpp b/third_party/WebKit/Source/core/editing/markers/CompositionMarkerListImpl.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..2db6f1dbca2ba63c167f183bff6651c66e548738
--- /dev/null
+++ b/third_party/WebKit/Source/core/editing/markers/CompositionMarkerListImpl.cpp
@@ -0,0 +1,60 @@
+// 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.
+
+#include "core/editing/markers/CompositionMarkerListImpl.h"
+
+#include "core/editing/markers/DocumentMarkerListEditor.h"
+#include "core/editing/markers/RenderedDocumentMarker.h"
+
+namespace blink {
+
+bool CompositionMarkerListImpl::IsEmpty() const {
+  return markers_.IsEmpty();
+}
+
+void CompositionMarkerListImpl::Add(DocumentMarker* marker) {
+  DocumentMarkerListEditor::AddMarkerWithoutMergingOverlapping(&markers_,
+                                                               marker);
+}
+
+void CompositionMarkerListImpl::Clear() {
+  markers_.clear();
+}
+
+const HeapVector<Member<RenderedDocumentMarker>>&
+CompositionMarkerListImpl::GetMarkers() const {
+  return markers_;
+}
+
+bool CompositionMarkerListImpl::MoveMarkers(int length,
+                                            DocumentMarkerList* dst_markers_) {
+  return DocumentMarkerListEditor::MoveMarkers(&markers_, length, dst_markers_);
+}
+
+bool CompositionMarkerListImpl::RemoveMarkers(unsigned start_offset,
+                                              int length) {
+  return DocumentMarkerListEditor::RemoveMarkers(&markers_, start_offset,
+                                                 length);
+}
+
+bool CompositionMarkerListImpl::RemoveMarkersUnderWords(
+    const String& node_text,
+    const Vector<String>& words) {
+  return DocumentMarkerListEditor::RemoveMarkersUnderWords(&markers_, node_text,

[rlanday, 2017/04/26 20:54:07]

I added a call to DocumentMarkerListEditor::RemoveMarkersUnderWords() here since
we
have it rather than adding an empty implementation to the interface; I think it
doesn't matter too much one way or the other since we're going to move this
method
to SpellCheckMarkerListImpl in the next CL.

[Xiaocheng, 2017/04/26 22:20:37]

Now that DMC::RemoveMarkersUnderWords() only removes spelling and grammar
markers, I think NOTREACHED() is the best function body here.

+                                                           words);
+}
+
+bool CompositionMarkerListImpl::ShiftMarkers(unsigned offset,
+                                             unsigned old_length,
+                                             unsigned new_length) {
+  return DocumentMarkerListEditor::ShiftMarkers(&markers_, offset, old_length,
+                                                new_length);
+}
+
+DEFINE_TRACE(CompositionMarkerListImpl) {
+  visitor->Trace(markers_);
+  DocumentMarkerList::Trace(visitor);
+}
+
+}  // namespace blink