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

Unified Diff: Source/core/editing/FormatBlockCommand.cpp

Issue 420103002: Use tighter typing in editing: FormatBlockCommand / FrameSelection (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Keep body change out Created 6 years, 5 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 | Source/core/editing/FrameSelection.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/editing/FormatBlockCommand.cpp
diff --git a/Source/core/editing/FormatBlockCommand.cpp b/Source/core/editing/FormatBlockCommand.cpp
index 1cd30e66c7fbf3194704a5474bcf78593e76382b..36b6b2989d98c4efe3a0ff44710ff0a5f7e36760 100644
--- a/Source/core/editing/FormatBlockCommand.cpp
+++ b/Source/core/editing/FormatBlockCommand.cpp
@@ -60,12 +60,12 @@ void FormatBlockCommand::formatSelection(const VisiblePosition& startOfSelection
m_didApply = true;
}
-void FormatBlockCommand::formatRange(const Position& start, const Position& end, const Position& endOfSelection, RefPtrWillBeRawPtr<HTMLElement>& blockNode)
+void FormatBlockCommand::formatRange(const Position& start, const Position& end, const Position& endOfSelection, RefPtrWillBeRawPtr<HTMLElement>& blockElement)
{
- Element* refNode = enclosingBlockFlowElement(VisiblePosition(end));
+ Element* refElement = enclosingBlockFlowElement(VisiblePosition(end));
Element* root = editableRootForPosition(start);
// Root is null for elements with contenteditable=false.
- if (!root || !refNode)
+ if (!root || !refElement)
return;
Node* nodeToSplitTo = enclosingBlockToSplitTreeTo(start.deprecatedNode());
@@ -73,30 +73,30 @@ void FormatBlockCommand::formatRange(const Position& start, const Position& end,
RefPtrWillBeRawPtr<Node> nodeAfterInsertionPosition = outerBlock;
RefPtrWillBeRawPtr<Range> range = Range::create(document(), start, endOfSelection);
- if (isElementForFormatBlock(refNode->tagQName()) && VisiblePosition(start) == startOfBlock(VisiblePosition(start))
- && (VisiblePosition(end) == endOfBlock(VisiblePosition(end)) || isNodeVisiblyContainedWithin(*refNode, *range))
- && refNode != root && !root->isDescendantOf(refNode)) {
+ if (isElementForFormatBlock(refElement->tagQName()) && VisiblePosition(start) == startOfBlock(VisiblePosition(start))
+ && (VisiblePosition(end) == endOfBlock(VisiblePosition(end)) || isNodeVisiblyContainedWithin(*refElement, *range))
+ && refElement != root && !root->isDescendantOf(refElement)) {
// Already in a block element that only contains the current paragraph
- if (refNode->hasTagName(tagName()))
+ if (refElement->hasTagName(tagName()))
return;
- nodeAfterInsertionPosition = refNode;
+ nodeAfterInsertionPosition = refElement;
}
- if (!blockNode) {
+ if (!blockElement) {
// Create a new blockquote and insert it as a child of the root editable element. We accomplish
// this by splitting all parents of the current paragraph up to that point.
- blockNode = createBlockElement();
- insertNodeBefore(blockNode, nodeAfterInsertionPosition);
+ blockElement = createBlockElement();
+ insertNodeBefore(blockElement, nodeAfterInsertionPosition);
}
- Position lastParagraphInBlockNode = blockNode->lastChild() ? positionAfterNode(blockNode->lastChild()) : Position();
+ Position lastParagraphInBlockNode = blockElement->lastChild() ? positionAfterNode(blockElement->lastChild()) : Position();
bool wasEndOfParagraph = isEndOfParagraph(VisiblePosition(lastParagraphInBlockNode));
- moveParagraphWithClones(VisiblePosition(start), VisiblePosition(end), blockNode.get(), outerBlock.get());
+ moveParagraphWithClones(VisiblePosition(start), VisiblePosition(end), blockElement.get(), outerBlock.get());
// Copy the inline style of the original block element to the newly created block-style element.
if (outerBlock.get() != nodeAfterInsertionPosition.get() && toHTMLElement(nodeAfterInsertionPosition.get())->hasAttribute(styleAttr))
- blockNode->setAttribute(styleAttr, toHTMLElement(nodeAfterInsertionPosition.get())->getAttribute(styleAttr));
+ blockElement->setAttribute(styleAttr, toHTMLElement(nodeAfterInsertionPosition.get())->getAttribute(styleAttr));
if (wasEndOfParagraph && !isEndOfParagraph(VisiblePosition(lastParagraphInBlockNode)) && !isStartOfParagraph(VisiblePosition(lastParagraphInBlockNode)))
insertBlockPlaceholder(lastParagraphInBlockNode);
« no previous file with comments | « no previous file | Source/core/editing/FrameSelection.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698