Plumbing for browser process to access text surrounding selection.

content/browser/renderer_host/render_widget_host_view_base.h

Index: content/browser/renderer_host/render_widget_host_view_base.h
diff --git a/content/browser/renderer_host/render_widget_host_view_base.h b/content/browser/renderer_host/render_widget_host_view_base.h
index ad85021c85a0d6c210f740879d34ce588b2ca95b..0290311586800448e0442dfcaa258bd78ec7aaa7 100644
--- a/content/browser/renderer_host/render_widget_host_view_base.h
+++ b/content/browser/renderer_host/render_widget_host_view_base.h
@@ -316,6 +316,10 @@ class CONTENT_EXPORT RenderWidgetHostViewBase : public RenderWidgetHostView,
   virtual void SetScrollOffsetPinning(
       bool is_pinned_to_left, bool is_pinned_to_right) = 0;
 
+  virtual void OnTextSurroundingSelectionResponse(const base::string16& content,
+                                                  unsigned start_offset,
+                                                  unsigned end_offset) {};

[dcheng, 2014/05/21 21:05:38]

Is there a reason this isn't pure virtual like most of the other methods?
Consistency would be nice.

[mlamouri (slow - plz ping), 2014/05/21 21:19:40]

Making it pure virtual would require to have empty definitions in every single
implementation. That sounds unfortunate. I could do it if needed. Note that some
methods have empty bodies in this class.

+
 #if defined(OS_ANDROID)
   virtual void ShowDisambiguationPopup(const gfx::Rect& target_rect,
                                        const SkBitmap& zoomed_bitmap) = 0;