[Android] Extend smart clip to return html along with text

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 46 matching lines @@
   if (delta.height == 0)
     return;
   if (compositor_) {
     cc::TopControlsState current = delta.height < 0 ? cc::SHOWN : cc::HIDDEN;
     compositor_->UpdateTopControlsState(top_controls_constraints_,
                                         current,
                                         true);
   }
 }
 
 void RenderViewImpl::OnExtractSmartClipData(const gfx::Rect& rect) {

[Yaron, 2014/07/09 23:08:14]

whomever reviewed this original CL

[Changwan Ryu, 2014/07/10 01:20:30]

Added jam@

On 2014/07/09 23:08:14, Yaron wrote:

   blink::WebString clip_text;
   blink::WebRect clip_rect;
   webview()->getSmartClipData(rect, clip_text, clip_rect);
+  blink::WebString clip_html = webview()->extractSmartClipDataAsMarkup(rect);

[Yaron, 2014/07/09 23:08:14]

can this not be done in the previous call?

[Changwan Ryu, 2014/07/10 01:20:30]

Fixed by extending getSmartClipData. However, changing the API may break the
build at one point, as webkit and this function will see different APIs at that
point. Would it be ok?

   Send(new ViewHostMsg_SmartClipDataExtracted(
-      routing_id_, clip_text, clip_rect));
+      routing_id_, clip_text, clip_html, clip_rect));
 }
 
 }  // namespace content