Defect 248426: Speak rendered text when no selection is made on Mac

content/renderer/render_view_impl.h

Index: content/renderer/render_view_impl.h
diff --git a/content/renderer/render_view_impl.h b/content/renderer/render_view_impl.h
index 2cd5f8c93de4054d8b4423ee4ca7e30fd77e55c8..3184c25c3a4597fdbb843e4f89d6a7c206112869 100644
--- a/content/renderer/render_view_impl.h
+++ b/content/renderer/render_view_impl.h
@@ -755,6 +755,7 @@ class CONTENT_EXPORT RenderViewImpl
       const gfx::Size& size, const std::vector<unsigned char>& png);
   void OnSelectWordAroundCaret();
 #if defined(OS_ANDROID)
+  void OnExtractSmartClipData(const gfx::Rect& rect);

[Alexei Svitkine (slow), 2014/06/25 13:58:29]

Revert this part of the change.

[sarka, 2014/06/25 22:36:56]

I think Android still uses it.

[Alexei Svitkine (slow), 2014/06/26 13:06:26]

I understand that. My point is you should revert to the original version of the
code. If you look at the diffs in your CL, you'll see that you're actually
re-ordering the declaration of this method. Please change back this part of this
change. (And anywhere else in the CL where there's a diff as a result of some
previous thing you were doing with the CL but is not needed anymore.) Thanks!

   void OnActivateNearestFindResult(int request_id, float x, float y);
   void OnFindMatchRects(int current_version);
   void OnSelectPopupMenuItems(bool canceled,
@@ -763,8 +764,8 @@ class CONTENT_EXPORT RenderViewImpl
   void OnUpdateTopControlsState(bool enable_hiding,
                                 bool enable_showing,
                                 bool animate);
-  void OnExtractSmartClipData(const gfx::Rect& rect);
 #elif defined(OS_MACOSX)
+  void OnGetRenderedText();
   void OnPluginImeCompositionCompleted(const base::string16& text,
                                        int plugin_id);
   void OnSelectPopupMenuItem(int selected_index);