Refactor Samsung SmartClip implementation.

content/renderer/render_view_impl_android.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/render_view_impl.h"
 
 #include "base/command_line.h"
 #include "base/message_loop/message_loop.h"
 #include "cc/trees/layer_tree_host.h"
 #include "content/common/view_messages.h"
@@ skipping 53 matching lines @@
   if (delta.height == 0)
     return;
 
   BrowserControlsState current = delta.height < 0
                                      ? BROWSER_CONTROLS_STATE_SHOWN
                                      : BROWSER_CONTROLS_STATE_HIDDEN;
 
   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