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

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

Issue 1311853004: Rename visiblePositionOf() to createVisiblePosition() (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: 2015-09-01T14:43:58 Created 5 years, 4 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
Index: Source/core/editing/VisibleUnits.cpp
diff --git a/Source/core/editing/VisibleUnits.cpp b/Source/core/editing/VisibleUnits.cpp
index 5ea7f0761594e76895bff49edc5582112b48ebb1..28579fe46185daa92389591df43c9b3a471b0780 100644
--- a/Source/core/editing/VisibleUnits.cpp
+++ b/Source/core/editing/VisibleUnits.cpp
@@ -814,12 +814,12 @@ static PositionInComposedTreeWithAffinity startOfLine(const PositionInComposedTr
// FIXME: Rename this function to reflect the fact it ignores bidi levels.
VisiblePosition startOfLine(const VisiblePosition& currentPosition)
{
- return visiblePositionOf(startOfLine(currentPosition.toPositionWithAffinity(), UseInlineBoxOrdering));
+ return createVisiblePosition(startOfLine(currentPosition.toPositionWithAffinity(), UseInlineBoxOrdering));
}
VisiblePosition logicalStartOfLine(const VisiblePosition& currentPosition)
{
- return visiblePositionOf(startOfLine(currentPosition.toPositionWithAffinity(), UseLogicalOrdering));
+ return createVisiblePosition(startOfLine(currentPosition.toPositionWithAffinity(), UseLogicalOrdering));
}
static VisiblePosition endPositionForLine(const VisiblePosition& c, LineEndpointComputationMode mode)
@@ -1029,7 +1029,7 @@ VisiblePosition previousLinePosition(const VisiblePosition& visiblePosition, Lay
Node* node = layoutObject.node();
if (node && editingIgnoresContent(node))
return VisiblePosition(positionInParentBeforeNode(*node));
- return visiblePositionOf(layoutObject.positionForPoint(pointInLine));
+ return createVisiblePosition(layoutObject.positionForPoint(pointInLine));
}
// Could not find a previous line. This means we must already be on the first line.
@@ -1085,7 +1085,7 @@ VisiblePosition nextLinePosition(const VisiblePosition& visiblePosition, LayoutU
Node* node = layoutObject.node();
if (node && editingIgnoresContent(node))
return VisiblePosition(positionInParentBeforeNode(*node));
- return visiblePositionOf(layoutObject.positionForPoint(pointInLine));
+ return createVisiblePosition(layoutObject.positionForPoint(pointInLine));
}
// Could not find a next line. This means we must already be on the last line.
« no previous file with comments | « Source/core/editing/VisibleSelection.cpp ('k') | Source/core/editing/serializers/StyledMarkupSerializer.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698