[MacViews] Implemented text context menu

content/browser/frame_host/render_widget_host_view_guest.cc

Index: content/browser/frame_host/render_widget_host_view_guest.cc
diff --git a/content/browser/frame_host/render_widget_host_view_guest.cc b/content/browser/frame_host/render_widget_host_view_guest.cc
index 2b9f1e3ed1be83b5c73eeeaa12f313ad59e12d4d..50f4aaccd1b7d5ecaf76a5398b5454fac303ae94 100644
--- a/content/browser/frame_host/render_widget_host_view_guest.cc
+++ b/content/browser/frame_host/render_widget_host_view_guest.cc
@@ -471,21 +471,10 @@ void RenderWidgetHostViewGuest::ShowDefinitionForSelection() {
   helper.ShowDefinitionForSelection();
 }
 
-bool RenderWidgetHostViewGuest::SupportsSpeech() const {
-  return platform_view_->SupportsSpeech();
-}
-
 void RenderWidgetHostViewGuest::SpeakSelection() {

[nasko, 2017/01/31 15:37:29]

Why does this method stay around and the others are removed? I'd expect they all
go together.

[spqchan, 2017/02/01 00:32:39]

In Mac, there is a menu item in the main menu that lets you select "Speak Text".
If nothing is selection, a message gets sent to the renderer to get all of the
text in the web content. This is async, so we need the method in
RenderWidgetHostView. IsSpeaking() and StopSpeaking() can be removed and handled
in text_services_context_menu because they're not async.

   platform_view_->SpeakSelection();
 }
 
-bool RenderWidgetHostViewGuest::IsSpeaking() const {
-  return platform_view_->IsSpeaking();
-}
-
-void RenderWidgetHostViewGuest::StopSpeaking() {
-  platform_view_->StopSpeaking();
-}
 #endif  // defined(OS_MACOSX)
 
 void RenderWidgetHostViewGuest::LockCompositingSurface() {