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

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

Issue 2829543002: [DMC #5] Add SpellCheckMarkerListImpl (Closed)
Patch Set: Remove comment about RemoveMarkersUnderWords() being SpellCheckMarkerListImpl-specific (it's not ye… Created 3 years, 7 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
1 // Copyright 2017 The Chromium Authors. All rights reserved. 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 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "core/editing/markers/CompositionMarkerListImpl.h" 5 #include "core/editing/markers/CompositionMarkerListImpl.h"
6 6
7 #include "core/editing/markers/DocumentMarkerListEditor.h" 7 #include "core/editing/markers/DocumentMarkerListEditor.h"
8 #include "core/editing/markers/RenderedDocumentMarker.h" 8 #include "core/editing/markers/RenderedDocumentMarker.h"
9 9
10 namespace blink { 10 namespace blink {
(...skipping 23 matching lines...) Expand all
34 34
35 bool CompositionMarkerListImpl::RemoveMarkers(unsigned start_offset, 35 bool CompositionMarkerListImpl::RemoveMarkers(unsigned start_offset,
36 int length) { 36 int length) {
37 return DocumentMarkerListEditor::RemoveMarkers(&markers_, start_offset, 37 return DocumentMarkerListEditor::RemoveMarkers(&markers_, start_offset,
38 length); 38 length);
39 } 39 }
40 40
41 bool CompositionMarkerListImpl::RemoveMarkersUnderWords( 41 bool CompositionMarkerListImpl::RemoveMarkersUnderWords(
42 const String& node_text, 42 const String& node_text,
43 const Vector<String>& words) { 43 const Vector<String>& words) {
44 // This method will be removed from the DocumentMarkerList interface once 44 // This method will be removed from the DocumentMarkerList interface in a
45 // SpellCheckMatchMarkerListImpl is added. DocumentMarkerController shouldn't 45 // later CL. SpellCheckMatchMarkerListImpl is added. DocumentMarkerController
46 // try to call it on a list storing Composition markers. 46 // shouldn't try to call it on a list storing Composition markers.
47 NOTREACHED(); 47 NOTREACHED();
48 return false; 48 return false;
49 } 49 }
50 50
51 bool CompositionMarkerListImpl::ShiftMarkers(unsigned offset, 51 bool CompositionMarkerListImpl::ShiftMarkers(unsigned offset,
52 unsigned old_length, 52 unsigned old_length,
53 unsigned new_length) { 53 unsigned new_length) {
54 return DocumentMarkerListEditor::ShiftMarkers(&markers_, offset, old_length, 54 return DocumentMarkerListEditor::ShiftMarkers(&markers_, offset, old_length,
55 new_length); 55 new_length);
56 } 56 }
57 57
58 DEFINE_TRACE(CompositionMarkerListImpl) { 58 DEFINE_TRACE(CompositionMarkerListImpl) {
59 visitor->Trace(markers_); 59 visitor->Trace(markers_);
60 DocumentMarkerList::Trace(visitor); 60 DocumentMarkerList::Trace(visitor);
61 } 61 }
62 62
63 } // namespace blink 63 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698