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

Side by Side Diff: third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp

Issue 2387263004: Manually remove many instances of a comma quirk arising from the reformat. (Closed)
Patch Set: merge with master; thakis nit Created 4 years, 2 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 /* 1 /*
2 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All r ights reserved. 2 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All r ights reserved.
3 * Copyright (C) 2005 Alexey Proskuryakov. 3 * Copyright (C) 2005 Alexey Proskuryakov.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions 6 * modification, are permitted provided that the following conditions
7 * are met: 7 * are met:
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 144 matching lines...) Expand 10 before | Expand all | Expand 10 after
155 m_needsAnotherNewline(false), 155 m_needsAnotherNewline(false),
156 m_textBox(nullptr), 156 m_textBox(nullptr),
157 m_remainingTextBox(nullptr), 157 m_remainingTextBox(nullptr),
158 m_firstLetterText(nullptr), 158 m_firstLetterText(nullptr),
159 m_lastTextNode(nullptr), 159 m_lastTextNode(nullptr),
160 m_lastTextNodeEndedWithCollapsedSpace(false), 160 m_lastTextNodeEndedWithCollapsedSpace(false),
161 m_sortedTextBoxesPosition(0), 161 m_sortedTextBoxesPosition(0),
162 m_behavior(adjustBehaviorFlags<Strategy>(behavior)), 162 m_behavior(adjustBehaviorFlags<Strategy>(behavior)),
163 m_handledFirstLetter(false), 163 m_handledFirstLetter(false),
164 m_shouldStop(false), 164 m_shouldStop(false),
165 m_handleShadowRoot(false) 165 m_handleShadowRoot(false),
166 // The call to emitsOriginalText() must occur after m_behavior is initiali zed. 166 // The call to emitsOriginalText() must occur after m_behavior is initiali zed.
167 ,
168 m_textState(emitsOriginalText()) { 167 m_textState(emitsOriginalText()) {
169 DCHECK(start.isNotNull()); 168 DCHECK(start.isNotNull());
170 DCHECK(end.isNotNull()); 169 DCHECK(end.isNotNull());
171 170
172 // TODO(dglazkov): TextIterator should not be created for documents that don't have a frame, 171 // TODO(dglazkov): TextIterator should not be created for documents that don't have a frame,
173 // but it currently still happens in some cases. See http://crbug.com/591877 f or details. 172 // but it currently still happens in some cases. See http://crbug.com/591877 f or details.
174 DCHECK(!start.document()->view() || !start.document()->view()->needsLayout()); 173 DCHECK(!start.document()->view() || !start.document()->view()->needsLayout());
175 DCHECK(!start.document()->needsLayoutTreeUpdate()); 174 DCHECK(!start.document()->needsLayoutTreeUpdate());
176 175
177 if (start.compareTo(end) > 0) { 176 if (start.compareTo(end) > 0) {
(...skipping 1146 matching lines...) Expand 10 before | Expand all | Expand 10 after
1324 String plainText(const EphemeralRangeInFlatTree& range, 1323 String plainText(const EphemeralRangeInFlatTree& range,
1325 TextIteratorBehaviorFlags behavior) { 1324 TextIteratorBehaviorFlags behavior) {
1326 return createPlainText<EditingInFlatTreeStrategy>(range, behavior); 1325 return createPlainText<EditingInFlatTreeStrategy>(range, behavior);
1327 } 1326 }
1328 1327
1329 template class CORE_TEMPLATE_EXPORT TextIteratorAlgorithm<EditingStrategy>; 1328 template class CORE_TEMPLATE_EXPORT TextIteratorAlgorithm<EditingStrategy>;
1330 template class CORE_TEMPLATE_EXPORT 1329 template class CORE_TEMPLATE_EXPORT
1331 TextIteratorAlgorithm<EditingInFlatTreeStrategy>; 1330 TextIteratorAlgorithm<EditingInFlatTreeStrategy>;
1332 1331
1333 } // namespace blink 1332 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698