Index: content/browser/android/content_view_core_impl.cc |
diff --git a/content/browser/android/content_view_core_impl.cc b/content/browser/android/content_view_core_impl.cc |
index 69449fb5db4c006a4e69bc9a574e69eeb8c9a6d1..94a3f948ff2dbd792b15c3b446628e80f7c1afbf 100644 |
--- a/content/browser/android/content_view_core_impl.cc |
+++ b/content/browser/android/content_view_core_impl.cc |
@@ -777,17 +777,24 @@ void ContentViewCoreImpl::RemoveLayer(scoped_refptr<cc::Layer> layer) { |
root_layer_->SetIsDrawable(true); |
} |
-void ContentViewCoreImpl::SelectBetweenCoordinates(const gfx::PointF& start, |
- const gfx::PointF& end) { |
+void ContentViewCoreImpl::MoveRangeSelectionExtent(const gfx::PointF& extent) { |
if (!web_contents_) |
return; |
- gfx::Point start_point = gfx::Point(start.x(), start.y()); |
- gfx::Point end_point = gfx::Point(end.x(), end.y()); |
- if (start_point == end_point) |
+ web_contents_->MoveRangeSelectionExtent(gfx::Point(extent.x(), extent.y())); |
jdduke (slow)
2014/11/03 17:58:48
I wonder, will the renderer guard against the sele
christiank
2014/11/04 09:44:28
Yes it should. WebLocalFrameImpl::moveRangeSelecti
|
+} |
+ |
+void ContentViewCoreImpl::SelectBetweenCoordinates(const gfx::PointF& base, |
+ const gfx::PointF& extent) { |
+ if (!web_contents_) |
+ return; |
+ |
+ gfx::Point base_point = gfx::Point(base.x(), base.y()); |
+ gfx::Point extent_point = gfx::Point(extent.x(), extent.y()); |
+ if (base_point == extent_point) |
return; |
- web_contents_->SelectRange(start_point, end_point); |
+ web_contents_->SelectRange(base_point, extent_point); |
} |
ui::ViewAndroid* ContentViewCoreImpl::GetViewAndroid() const { |