WebRange-ify WebWidget::applyReplacementRange.

third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp

Index: third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp b/third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp
index 86c93cfd5d6b182744b968eb9c2cdc3f249a90ae..530b89b36aebc95ba6c1f4fbc1a12c53eee95068 100644
--- a/third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp
+++ b/third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp
@@ -879,12 +879,12 @@ bool WebFrameWidgetImpl::getCompositionCharacterBounds(WebVector<WebRect>& bound
     return true;
 }
 
-void WebFrameWidgetImpl::applyReplacementRange(int start, int length)
+void WebFrameWidgetImpl::applyReplacementRange(const WebRange& range)

[esprehn, 2016/09/02 21:25:22]

fwiw WebRange is just a wrapper around some ints, so passing it by value is
probably fine, and might be faster than the reference I think?

Either is okay though, this code is not performance sensitive

 {
     if (LocalFrame* frame = focusedLocalFrameInWidget()) {
         // TODO(dglazkov): Going from LocalFrame to WebLocalFrameImpl seems
         // silly. What is going on here?
-        WebLocalFrameImpl::fromFrame(frame)->selectRange(WebRange(start, length));
+        WebLocalFrameImpl::fromFrame(frame)->selectRange(range);
     }
 }