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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: third_party/WebKit/Source/core/editing/commands/ReplaceSelectionCommand.cpp

Issue 2395393002: Prune deprecated functions from ReplaceSelectionCommand (Closed)
Patch Set: 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/editing/commands/ReplaceSelectionCommand.cpp
diff --git a/third_party/WebKit/Source/core/editing/commands/ReplaceSelectionCommand.cpp b/third_party/WebKit/Source/core/editing/commands/ReplaceSelectionCommand.cpp
index f8a7387d755aca338ef3caecc01af769375405bf..987600fc3f5103261d7ce00bfab464b8fdecdc58 100644
--- a/third_party/WebKit/Source/core/editing/commands/ReplaceSelectionCommand.cpp
+++ b/third_party/WebKit/Source/core/editing/commands/ReplaceSelectionCommand.cpp
@@ -154,8 +154,8 @@ static Position positionAvoidingPrecedingNodes(Position pos) {
if (nextPosition == pos ||
enclosingBlock(nextPosition.computeContainerNode()) !=
enclosingBlockElement ||
- createVisiblePositionDeprecated(pos).deepEquivalent() !=
- createVisiblePositionDeprecated(nextPosition).deepEquivalent())
+ createVisiblePosition(pos).deepEquivalent() !=
+ createVisiblePosition(nextPosition).deepEquivalent())
break;
}
return pos;
@@ -472,14 +472,14 @@ bool ReplaceSelectionCommand::shouldMergeStart(
// pasted into an unquoted position. If that unquoted position happens to be
// right after another blockquote, we don't want to merge and risk stripping a
// valid block (and newline) from the pasted content.
- if (isStartOfParagraphDeprecated(startOfInsertedContent) &&
+ if (isStartOfParagraph(startOfInsertedContent) &&
selectionStartWasInsideMailBlockquote &&
hasMatchingQuoteLevel(prev, positionAtEndOfInsertedContent()))
return true;
return !selectionStartWasStartOfParagraph &&
!fragmentHasInterchangeNewlineAtStart &&
- isStartOfParagraphDeprecated(startOfInsertedContent) &&
+ isStartOfParagraph(startOfInsertedContent) &&
!isHTMLBRElement(
*startOfInsertedContent.deepEquivalent().anchorNode()) &&
shouldMerge(startOfInsertedContent, prev);
@@ -494,7 +494,7 @@ bool ReplaceSelectionCommand::shouldMergeEnd(
return false;
return !selectionEndWasEndOfParagraph &&
- isEndOfParagraphDeprecated(endOfInsertedContent) &&
+ isEndOfParagraph(endOfInsertedContent) &&
!isHTMLBRElement(
*endOfInsertedContent.deepEquivalent().anchorNode()) &&
shouldMerge(endOfInsertedContent, next);
@@ -755,8 +755,9 @@ void ReplaceSelectionCommand::moveElementOutOfAncestor(
if (!hasEditableStyle(*ancestor->parentNode()))
return;
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
VisiblePosition positionAtEndOfNode =
- createVisiblePositionDeprecated(lastPositionInOrAfterNode(element));
+ createVisiblePosition(lastPositionInOrAfterNode(element));
VisiblePosition lastPositionInParagraph =
VisiblePosition::lastPositionInNode(ancestor);
if (positionAtEndOfNode.deepEquivalent() ==
@@ -817,24 +818,27 @@ void ReplaceSelectionCommand::removeUnrenderedTextNodesAtEnds(
VisiblePosition ReplaceSelectionCommand::positionAtEndOfInsertedContent()
const {
+ // TODO(xiaochengh): Hoist the call and change it into a DCHECK.
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
// TODO(yosin): We should set |m_endOfInsertedContent| not in SELECT
// element, since contents of SELECT elements, e.g. OPTION, OPTGROUP, are
// not editable, or SELECT element is an atomic on editing.
HTMLSelectElement* enclosingSelect = toHTMLSelectElement(
enclosingElementWithTag(m_endOfInsertedContent, selectTag));
if (enclosingSelect)
- return createVisiblePositionDeprecated(
- lastPositionInOrAfterNode(enclosingSelect));
+ return createVisiblePosition(lastPositionInOrAfterNode(enclosingSelect));
if (m_endOfInsertedContent.isOrphan())
return VisiblePosition();
- return createVisiblePositionDeprecated(m_endOfInsertedContent);
+ return createVisiblePosition(m_endOfInsertedContent);
}
VisiblePosition ReplaceSelectionCommand::positionAtStartOfInsertedContent()
const {
+ // TODO(xiaochengh): Hoist the call and change it into a DCHECK.
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
if (m_startOfInsertedContent.isOrphan())
return VisiblePosition();
- return createVisiblePositionDeprecated(m_startOfInsertedContent);
+ return createVisiblePosition(m_startOfInsertedContent);
}
static void removeHeadContents(ReplacementFragment& fragment) {
@@ -1005,20 +1009,21 @@ void ReplaceSelectionCommand::mergeEndIfNeeded(EditingState* editingState) {
// paragraph already in the document, unless the paragraph to move would
// include the what was the start of the selection that was pasted into, so
// that we preserve that paragraph's block styles.
- bool mergeForward = !(
- inSameParagraphDeprecated(startOfInsertedContent, endOfInsertedContent) &&
- !isStartOfParagraphDeprecated(startOfInsertedContent));
+ bool mergeForward =
+ !(inSameParagraph(startOfInsertedContent, endOfInsertedContent) &&
+ !isStartOfParagraph(startOfInsertedContent));
VisiblePosition destination = mergeForward
? nextPositionOf(endOfInsertedContent)
: endOfInsertedContent;
+ // TODO(xiaochengh): Stop storing VisiblePositions through mutations.
VisiblePosition startOfParagraphToMove =
- mergeForward ? startOfParagraphDeprecated(endOfInsertedContent)
+ mergeForward ? startOfParagraph(endOfInsertedContent)
: nextPositionOf(endOfInsertedContent);
// Merging forward could result in deleting the destination anchor node.
// To avoid this, we add a placeholder node before the start of the paragraph.
- if (endOfParagraphDeprecated(startOfParagraphToMove).deepEquivalent() ==
+ if (endOfParagraph(startOfParagraphToMove).deepEquivalent() ==
destination.deepEquivalent()) {
HTMLBRElement* placeholder = HTMLBRElement::create(document());
insertNodeBefore(placeholder,
@@ -1026,22 +1031,26 @@ void ReplaceSelectionCommand::mergeEndIfNeeded(EditingState* editingState) {
editingState);
if (editingState->isAborted())
return;
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
destination = VisiblePosition::beforeNode(placeholder);
+ startOfParagraphToMove =
+ createVisiblePosition(startOfParagraphToMove.toPositionWithAffinity());
}
- moveParagraph(startOfParagraphToMove,
- endOfParagraphDeprecated(startOfParagraphToMove), destination,
- editingState);
+ moveParagraph(startOfParagraphToMove, endOfParagraph(startOfParagraphToMove),
+ destination, editingState);
if (editingState->isAborted())
return;
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
+
// Merging forward will remove m_endOfInsertedContent from the document.
if (mergeForward) {
- if (m_startOfInsertedContent.isOrphan())
+ if (m_startOfInsertedContent.isOrphan()) {
m_startOfInsertedContent =
- endingSelection().visibleStartDeprecated().deepEquivalent();
- m_endOfInsertedContent =
- endingSelection().visibleEndDeprecated().deepEquivalent();
+ endingSelection().visibleStart().deepEquivalent();
+ }
+ m_endOfInsertedContent = endingSelection().visibleEnd().deepEquivalent();
// If we merged text nodes, m_endOfInsertedContent could be null. If
// this is the case, we use m_startOfInsertedContent.
if (m_endOfInsertedContent.isNull())
@@ -1124,6 +1133,8 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
if (trivialReplaceResult)
return;
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
+
// We can skip matching the style if the selection is plain text.
if ((selection.start().anchorNode()->layoutObject() &&
selection.start().anchorNode()->layoutObject()->style()->userModify() ==
@@ -1138,13 +1149,12 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
m_insertionStyle->mergeTypingStyle(&document());
}
- const VisiblePosition visibleStart = selection.visibleStartDeprecated();
- const VisiblePosition visibleEnd = selection.visibleEndDeprecated();
+ const VisiblePosition visibleStart = selection.visibleStart();
+ const VisiblePosition visibleEnd = selection.visibleEnd();
- const bool selectionEndWasEndOfParagraph =
- isEndOfParagraphDeprecated(visibleEnd);
+ const bool selectionEndWasEndOfParagraph = isEndOfParagraph(visibleEnd);
const bool selectionStartWasStartOfParagraph =
- isStartOfParagraphDeprecated(visibleStart);
+ isStartOfParagraph(visibleStart);
Element* enclosingBlockOfVisibleStart =
enclosingBlock(visibleStart.deepEquivalent().anchorNode());
@@ -1170,7 +1180,7 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
// Merge blocks if the start of the selection was in a Mail blockquote,
// since we handle that case specially to prevent nesting.
bool mergeBlocksAfterDelete = startIsInsideMailBlockquote ||
- isEndOfParagraphDeprecated(visibleEnd) ||
+ isEndOfParagraph(visibleEnd) ||
isStartOfBlock(visibleStart);
// FIXME: We should only expand to include fully selected special elements
// if we are copying a selection and pasting it on top of itself.
@@ -1178,10 +1188,10 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
if (editingState->isAborted())
return;
if (fragment.hasInterchangeNewlineAtStart()) {
- VisiblePosition startAfterDelete =
- endingSelection().visibleStartDeprecated();
- if (isEndOfParagraphDeprecated(startAfterDelete) &&
- !isStartOfParagraphDeprecated(startAfterDelete) &&
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
+ VisiblePosition startAfterDelete = endingSelection().visibleStart();
+ if (isEndOfParagraph(startAfterDelete) &&
+ !isStartOfParagraph(startAfterDelete) &&
!isEndOfEditableOrNonEditableContent(startAfterDelete))
setEndingSelection(nextPositionOf(startAfterDelete));
else
@@ -1194,13 +1204,15 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
if (fragment.hasInterchangeNewlineAtStart()) {
const VisiblePosition next =
nextPositionOf(visibleStart, CannotCrossEditingBoundary);
- if (isEndOfParagraphDeprecated(visibleStart) &&
- !isStartOfParagraphDeprecated(visibleStart) && next.isNotNull())
+ if (isEndOfParagraph(visibleStart) && !isStartOfParagraph(visibleStart) &&
+ next.isNotNull()) {
setEndingSelection(next);
- else
+ } else {
insertParagraphSeparator(editingState);
- if (editingState->isAborted())
- return;
+ if (editingState->isAborted())
+ return;
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
+ }
}
// We split the current paragraph in two to avoid nesting the blocks from
// the fragment inside the current block.
@@ -1217,15 +1229,13 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
// <div>xbar<div>bar</div><div>bazx</div></div>
// Don't do this if the selection started in a Mail blockquote.
if (m_preventNesting && !startIsInsideMailBlockquote &&
- !isEndOfParagraphDeprecated(
- endingSelection().visibleStartDeprecated()) &&
- !isStartOfParagraphDeprecated(
- endingSelection().visibleStartDeprecated())) {
+ !isEndOfParagraph(endingSelection().visibleStart()) &&
+ !isStartOfParagraph(endingSelection().visibleStart())) {
insertParagraphSeparator(editingState);
if (editingState->isAborted())
return;
- setEndingSelection(
- previousPositionOf(endingSelection().visibleStartDeprecated()));
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
+ setEndingSelection(previousPositionOf(endingSelection().visibleStart()));
}
}
@@ -1257,6 +1267,8 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
// <div>foo</div> into hello^ world.
prepareWhitespaceAtPositionForSplit(insertionPos);
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
+
// If the downstream node has been removed there's no point in continuing.
if (!mostForwardCaretPosition(insertionPos).anchorNode())
return;
@@ -1285,8 +1297,7 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
enclosingBlockOfInsertionPos != currentRoot &&
!isTableCell(enclosingBlockOfInsertionPos) &&
!startIsInsideMailBlockquote) {
- VisiblePosition visibleInsertionPos =
- createVisiblePositionDeprecated(insertionPos);
+ VisiblePosition visibleInsertionPos = createVisiblePosition(insertionPos);
if (isEndOfBlock(visibleInsertionPos) &&
!(isStartOfBlock(visibleInsertionPos) &&
fragment.hasInterchangeNewlineAtEnd()))
@@ -1310,6 +1321,8 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
removeHeadContents(fragment);
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
+
// We don't want the destination to end up inside nodes that weren't selected.
// To avoid that, we move the position forward without changing the visible
// position so we're still at the same visible location, but outside of
@@ -1430,6 +1443,8 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
return;
}
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
+
// Mutation events (bug 20161) may have already removed the inserted content
if (!insertedNodes.firstNodeInserted() ||
!insertedNodes.firstNodeInserted()->isConnected())
@@ -1442,7 +1457,7 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
!enclosingBlockOfInsertionPos->isConnected())
enclosingBlockOfInsertionPos = nullptr;
- VisiblePosition startOfInsertedContent = createVisiblePositionDeprecated(
+ VisiblePosition startOfInsertedContent = createVisiblePosition(
firstPositionInOrBeforeNode(insertedNodes.firstNodeInserted()));
// We inserted before the enclosingBlockOfInsertionPos to prevent nesting, and
@@ -1453,7 +1468,7 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
insertionPos.anchorNode() == enclosingBlockOfInsertionPos->parentNode() &&
(unsigned)insertionPos.computeEditingOffset() <
enclosingBlockOfInsertionPos->nodeIndex() &&
- !isStartOfParagraphDeprecated(startOfInsertedContent)) {
+ !isStartOfParagraph(startOfInsertedContent)) {
insertNodeAt(HTMLBRElement::create(document()),
startOfInsertedContent.deepEquivalent(), editingState);
if (editingState->isAborted())
@@ -1510,6 +1525,7 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
if (shouldMergeStart(selectionStartWasStartOfParagraph,
fragment.hasInterchangeNewlineAtStart(),
startIsInsideMailBlockquote)) {
+ // TODO(xiaochengh): Stop storing VisiblePositions through mutations.
VisiblePosition startOfParagraphToMove = positionAtStartOfInsertedContent();
VisiblePosition destination = previousPositionOf(startOfParagraphToMove);
// We need to handle the case where we need to merge the end
@@ -1536,7 +1552,7 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
// Insert a line break just after the inserted content to separate it from
// what comes after and prevent that from happening.
VisiblePosition endOfInsertedContent = positionAtEndOfInsertedContent();
- if (startOfParagraphDeprecated(endOfInsertedContent).deepEquivalent() ==
+ if (startOfParagraph(endOfInsertedContent).deepEquivalent() ==
startOfParagraphToMove.deepEquivalent()) {
insertNodeAt(HTMLBRElement::create(document()),
endOfInsertedContent.deepEquivalent(), editingState);
@@ -1548,19 +1564,25 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
return;
}
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
+
// FIXME: Maintain positions for the start and end of inserted content
// instead of keeping nodes. The nodes are only ever used to create
// positions where inserted content starts/ends.
moveParagraph(startOfParagraphToMove,
- endOfParagraphDeprecated(startOfParagraphToMove), destination,
- editingState);
+ endOfParagraph(createVisiblePosition(
+ startOfParagraphToMove.toPositionWithAffinity())),
+ destination, editingState);
if (editingState->isAborted())
return;
+
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
m_startOfInsertedContent = mostForwardCaretPosition(
- endingSelection().visibleStartDeprecated().deepEquivalent());
- if (m_endOfInsertedContent.isOrphan())
+ endingSelection().visibleStart().deepEquivalent());
+ if (m_endOfInsertedContent.isOrphan()) {
m_endOfInsertedContent = mostBackwardCaretPosition(
- endingSelection().visibleEndDeprecated().deepEquivalent());
+ endingSelection().visibleEnd().deepEquivalent());
+ }
}
Position lastPositionToSelect;
@@ -1570,7 +1592,7 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
nextPositionOf(endOfInsertedContent, CannotCrossEditingBoundary);
if (selectionEndWasEndOfParagraph ||
- !isEndOfParagraphDeprecated(endOfInsertedContent) || next.isNull()) {
+ !isEndOfParagraph(endOfInsertedContent) || next.isNull()) {
if (HTMLTextFormControlElement* textControl =
enclosingTextFormControl(currentRoot)) {
if (!insertedNodes.lastLeafInserted()->nextSibling()) {
@@ -1583,8 +1605,8 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
VisiblePosition::afterNode(insertedNodes.lastLeafInserted()));
// Select up to the paragraph separator that was added.
lastPositionToSelect =
- endingSelection().visibleStartDeprecated().deepEquivalent();
- } else if (!isStartOfParagraphDeprecated(endOfInsertedContent)) {
+ endingSelection().visibleStart().deepEquivalent();
+ } else if (!isStartOfParagraph(endOfInsertedContent)) {
setEndingSelection(endOfInsertedContent);
Element* enclosingBlockElement =
enclosingBlock(endOfInsertedContent.deepEquivalent().anchorNode());
@@ -1609,9 +1631,11 @@ void ReplaceSelectionCommand::doApply(EditingState* editingState) {
return;
}
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
+
// Select up to the paragraph separator that was added.
lastPositionToSelect =
- endingSelection().visibleStartDeprecated().deepEquivalent();
+ endingSelection().visibleStart().deepEquivalent();
updateNodesInserted(lastPositionToSelect.anchorNode());
}
} else {
@@ -1658,15 +1682,14 @@ bool ReplaceSelectionCommand::shouldRemoveEndBR(
// Remove the br if it is collapsed away and so is unnecessary.
if (!document().inNoQuirksMode() && isEndOfBlock(visiblePos) &&
- !isStartOfParagraphDeprecated(visiblePos))
+ !isStartOfParagraph(visiblePos))
return true;
// A br that was originally holding a line open should be displaced by
// inserted content or turned into a line break.
// A br that was originally acting as a line break should still be acting as a
// line break, not as a placeholder.
- return isStartOfParagraphDeprecated(visiblePos) &&
- isEndOfParagraphDeprecated(visiblePos);
+ return isStartOfParagraph(visiblePos) && isEndOfParagraph(visiblePos);
}
bool ReplaceSelectionCommand::shouldPerformSmartReplace() const {
@@ -1703,7 +1726,7 @@ void ReplaceSelectionCommand::addSpacesForSmartReplace(
}
bool needsTrailingSpace =
- !isEndOfParagraphDeprecated(endOfInsertedContent) &&
+ !isEndOfParagraph(endOfInsertedContent) &&
!isCharacterSmartReplaceExemptConsideringNonBreakingSpace(
characterAfter(endOfInsertedContent), false);
if (needsTrailingSpace && endNode) {
@@ -1742,7 +1765,7 @@ void ReplaceSelectionCommand::addSpacesForSmartReplace(
}
bool needsLeadingSpace =
- !isStartOfParagraphDeprecated(startOfInsertedContent) &&
+ !isStartOfParagraph(startOfInsertedContent) &&
!isCharacterSmartReplaceExemptConsideringNonBreakingSpace(
characterBefore(startOfInsertedContent), true);
if (needsLeadingSpace && startNode) {
@@ -1806,11 +1829,13 @@ void ReplaceSelectionCommand::completeHTMLReplacement(
m_startOfInsertedRange = start;
m_endOfInsertedRange = end;
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
+
if (m_selectReplacement)
- setEndingSelection(createVisibleSelectionDeprecated(
+ setEndingSelection(createVisibleSelection(
start, end, SelDefaultAffinity, endingSelection().isDirectional()));
else
- setEndingSelection(createVisibleSelectionDeprecated(
+ setEndingSelection(createVisibleSelection(
end, SelDefaultAffinity, endingSelection().isDirectional()));
}
@@ -1922,10 +1947,9 @@ Node* ReplaceSelectionCommand::insertAsListItems(HTMLElement* listElement,
isHTMLListElement(listElement->firstChild()))
listElement = toHTMLElement(listElement->firstChild());
- bool isStart =
- isStartOfParagraphDeprecated(createVisiblePositionDeprecated(insertPos));
- bool isEnd =
- isEndOfParagraphDeprecated(createVisiblePositionDeprecated(insertPos));
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
+ bool isStart = isStartOfParagraph(createVisiblePosition(insertPos));
+ bool isEnd = isEndOfParagraph(createVisiblePosition(insertPos));
bool isMiddle = !isStart && !isEnd;
Node* lastNode = insertionBlock;
@@ -2020,8 +2044,9 @@ bool ReplaceSelectionCommand::performTrivialReplace(
m_startOfInsertedRange = start;
m_endOfInsertedRange = end;
+ document().updateStyleAndLayoutIgnorePendingStylesheets();
VisibleSelection selectionAfterReplace =
- createVisibleSelectionDeprecated(m_selectReplacement ? start : end, end);
+ createVisibleSelection(m_selectReplacement ? start : end, end);
setEndingSelection(selectionAfterReplace);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698