Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(285)

Side by Side Diff: third_party/WebKit/Source/core/editing/markers/EditingMarkerList.cpp

Issue 2723663002: Refactor DocumentMarkerController (Closed)
Patch Set: Use correct base commit Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "core/editing/markers/EditingMarkerList.h"
6
7 #include <algorithm>
8 #include "core/editing/markers/DocumentMarkerController.h"
9
10 namespace blink {
11
12 EditingMarkerList::EditingMarkerList(
13 DocumentMarkerController* documentMarkerController)
14 : DocumentMarkerList(documentMarkerController), m_markersAreSorted(false) {}
15
16 void EditingMarkerList::clear() {
17 m_markers.clear();
18 m_markersAreSorted = true;
19 }
20
21 bool EditingMarkerList::copyMarkers(unsigned startOffset,
22 int length,
23 Node* dstNode,
24 int delta) const {
25 bool docDirty = false;
26 unsigned endOffset = startOffset + length - 1;
27 for (Member<DocumentMarker> marker : m_markers) {
28 // pin the marker to the specified range and apply the shift delta
29 if (marker->endOffset() >= startOffset &&
30 marker->startOffset() <= endOffset) {
31 docDirty = true;
32 if (marker->startOffset() < startOffset)
33 marker->setStartOffset(startOffset);
34 if (marker->endOffset() > endOffset)
35 marker->setEndOffset(endOffset);
36 marker->shiftOffsets(delta);
37
38 m_documentMarkerController->addMarker(dstNode, marker);
39 }
40 }
41
42 return docDirty;
43 }
44
45 void EditingMarkerList::removeMarkers(
46 unsigned startOffset,
47 int length,
48 bool shouldRemovePartiallyOverlappingMarkers,
49 bool* didRemoveMarker) {
50 unsigned endOffset = startOffset + length;
51 size_t markerIndex = 0;
52 if (m_markersAreSorted) {
53 markerIndex =
54 getPosOfFirstMarkerNotEndingBefore(markerIndex) - m_markers.begin();
55 }
56
57 for (; markerIndex < m_markers.size(); ++markerIndex) {
58 DocumentMarker& marker = *m_markers.at(markerIndex);
59
60 if (!m_markersAreSorted) {
61 if (marker.endOffset() <= startOffset)
62 continue;
63 }
64
65 if (marker.startOffset() >= endOffset) {
66 if (m_markersAreSorted)
67 break;
68 continue;
69 }
70
71 // pitch the old marker
72 m_markers.remove(markerIndex);
73 *didRemoveMarker = true;
74
75 if (shouldRemovePartiallyOverlappingMarkers) {
76 // Stop here. Don't add resulting slices back.
77 continue;
78 }
79
80 // add either of the resulting slices that are left after removing target
81 if (startOffset > marker.startOffset()) {
82 DocumentMarker* newLeft = new DocumentMarker(marker);
83 newLeft->setEndOffset(startOffset);
84 m_markers.insert(markerIndex, *newLeft);
85 // Move to the marker after the inserted one.
86 ++markerIndex;
87 }
88 if (marker.endOffset() > endOffset) {
89 DocumentMarker* newRight = new DocumentMarker(marker);
90 newRight->setStartOffset(endOffset);
91 m_markers.insert(markerIndex, *newRight);
92 // Move to the marker after the inserted one.
93 ++markerIndex;
94 }
95 }
96 }
97
98 bool EditingMarkerList::shiftMarkers(unsigned offset,
99 unsigned oldLength,
100 unsigned newLength) {
101 bool didShift = false;
102 for (auto it = m_markers.begin(); it != m_markers.end(); ++it) {
103 DocumentMarker& marker = **it;
104
105 DocumentMarker::ShiftMarkerResult result =
106 marker.getShiftedMarkerPosition(offset, oldLength, newLength);
107 if (result.shouldRemoveMarker) {
108 m_markers.remove(it - m_markers.begin());
109 --it;
110
111 didShift = true;
112 } else if (result.newStartOffset != marker.startOffset() ||
113 result.newEndOffset != marker.endOffset()) {
114 marker.setStartOffset(result.newStartOffset);
115 marker.setEndOffset(result.newEndOffset);
116
117 didShift = true;
118 }
119 }
120
121 return didShift;
122 }
123
124 void EditingMarkerList::insert(DocumentMarker* marker) {
125 DCHECK(marker->type() == allowedMarkerType());
126 if (m_markersAreSorted && !m_markers.isEmpty() &&
127 m_markers.back()->endOffset() > marker->startOffset())
128 m_markersAreSorted = false;
129 m_markers.push_back(marker);
130 }
131
132 void EditingMarkerList::sortMarkerList() {
133 std::sort(m_markers.begin(), m_markers.end(), compareByStart);
134 }
135
136 bool EditingMarkerList::compareByStart(const Member<DocumentMarker>& lhv,
137 const Member<DocumentMarker>& rhv) {
138 return lhv->startOffset() < rhv->startOffset();
139 }
140
141 DEFINE_TRACE(EditingMarkerList) {
142 visitor->trace(m_markers);
143 DocumentMarkerList::trace(visitor);
144 }
145
146 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698