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

Unified Diff: Source/core/testing/Internals.cpp

Issue 307353002: Use Position instead of VisiblePosition for SurroundingText. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
Index: Source/core/testing/Internals.cpp
diff --git a/Source/core/testing/Internals.cpp b/Source/core/testing/Internals.cpp
index eb48a0c2f7d3ad6bb64d350b5536b3d8d0067ee8..598b6fa9edbb27c4cf212e3ff209bf0dacf287f7 100644
--- a/Source/core/testing/Internals.cpp
+++ b/Source/core/testing/Internals.cpp
@@ -2339,7 +2339,7 @@ String Internals::textSurroundingNode(Node* node, int x, int y, unsigned long ma
if (!node)
return String();
blink::WebPoint point(x, y);
- SurroundingText surroundingText(VisiblePosition(node->renderer()->positionForPoint(static_cast<IntPoint>(point))), maxLength);
+ SurroundingText surroundingText(node->renderer()->positionForPoint(static_cast<IntPoint>(point)).position(), maxLength);
return surroundingText.content();
}

Powered by Google App Engine
This is Rietveld 408576698