Refactor Samsung SmartClip implementation.

content/renderer/render_view_impl_android.cc

Index: content/renderer/render_view_impl_android.cc
diff --git a/content/renderer/render_view_impl_android.cc b/content/renderer/render_view_impl_android.cc
index e13fea073487b12ac8bd62bd17c7e86aee345aae..aa3d3a1808cb88bf88df9e481b36a77161102d90 100644
--- a/content/renderer/render_view_impl_android.cc
+++ b/content/renderer/render_view_impl_android.cc
@@ -71,13 +71,4 @@ void RenderViewImpl::didScrollWithKeyboard(const blink::WebSize& delta) {
   UpdateBrowserControlsState(top_controls_constraints_, current, true);
 }
 
-void RenderViewImpl::OnExtractSmartClipData(const gfx::Rect& rect) {

[aelias_OOO_until_Jul13, 2017/02/07 20:48:13]

Yes, the bulk of the code on the Blink side was already cross-platform and
separating out this tiny amount of shim code doesn't achieve much (IMO having
Android-only code just harms testability and refactorability in general anyway).

-  blink::WebString clip_text;
-  blink::WebString clip_html;
-  blink::WebRect clip_rect;
-  webview()->extractSmartClipData(rect, clip_text, clip_html, clip_rect);
-  Send(new ViewHostMsg_SmartClipDataExtracted(routing_id_, clip_text.utf16(),
-                                              clip_html.utf16(), clip_rect));
-}
-
 }  // namespace content