Index: third_party/WebKit/Source/web/WebLocalFrameImpl.cpp |
diff --git a/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp b/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp |
index a2a1ce0a3a8a5648f6090052e0c8cbc42a8cb1f8..bfacc4a75516c3e1011567a31fdfae424b1c80bc 100644 |
--- a/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp |
+++ b/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp |
@@ -982,7 +982,7 @@ bool WebLocalFrameImpl::hasMarkedText() const |
WebRange WebLocalFrameImpl::markedRange() const |
{ |
- return frame()->inputMethodController().compositionRange(); |
+ return frame()->inputMethodController().compositionEphemeralRange(); |
} |
bool WebLocalFrameImpl::firstRectForCharacterRange(unsigned location, unsigned length, WebRect& rectInViewport) const |
@@ -1109,7 +1109,7 @@ bool WebLocalFrameImpl::hasSelection() const |
WebRange WebLocalFrameImpl::selectionRange() const |
{ |
- return createRange(frame()->selection().selection().toNormalizedEphemeralRange()); |
+ return frame()->selection().selection().toNormalizedEphemeralRange(); |
} |
WebString WebLocalFrameImpl::selectionAsText() const |
@@ -1165,10 +1165,7 @@ void WebLocalFrameImpl::selectRange(const WebRange& webRange) |
DocumentLifecycle::DisallowTransitionScope(frame()->document()->lifecycle()); |
- // TODO(dglazkov): Use EphemeralRange here. |
- // See http://crbug.com/636216 for more details. |
- if (Range* range = webRange.createRange(frame())) |
- frame()->selection().setSelectedRange(range, VP_DEFAULT_AFFINITY, SelectionDirectionalMode::NonDirectional, NotUserTriggered); |
+ frame()->selection().setSelectedRange(webRange.createEphemeralRange(frame()), VP_DEFAULT_AFFINITY, SelectionDirectionalMode::NonDirectional, NotUserTriggered); |
} |
WebString WebLocalFrameImpl::rangeAsText(const WebRange& webRange) |
@@ -1179,12 +1176,7 @@ WebString WebLocalFrameImpl::rangeAsText(const WebRange& webRange) |
DocumentLifecycle::DisallowTransitionScope(frame()->document()->lifecycle()); |
- // TODO(dglazkov): Use EphemeralRange here. |
- // See http://crbug.com/636216 for more details. |
- if (Range* range = webRange.createRange(frame())) |
- return range->text(); |
- |
- return WebString(); |
+ return plainText(webRange.createEphemeralRange(frame()), TextIteratorEmitsObjectReplacementCharacter); |
} |
void WebLocalFrameImpl::moveRangeSelectionExtent(const WebPoint& point) |