Make Edit>Speech>Speak Selection work for PDFs on Mac

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 9fb8cacb4d6b54e30c79a99b5d674ff53ad2378e..59a0ede0dbfcb2d036ce30fb60df3b7a661fbf42 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -2560,18 +2560,20 @@ void WebContentsImpl::Find(int request_id,
                            const base::string16& search_text,
                            const blink::WebFindOptions& options) {
   // See if a top level browser plugin handles the find request first.
-  if (browser_plugin_embedder_ &&
-      browser_plugin_embedder_->Find(request_id, search_text, options)) {
-    return;
+  if (browser_plugin_embedder_) {
+    BrowserPluginGuest* guest = browser_plugin_embedder_->GetFullPageGuest();

[Sam McNally, 2015/04/01 06:04:49]

Is this the right behavior for all guest types?

[raymes, 2015/04/13 11:00:31]

Yes, only the MimeHandlerViewGuest should turn true for Find below, so for other
guests it will drop through to the default behavior

+    if (guest && guest->Find(request_id, search_text, options))
+      return;
   }
   Send(new ViewMsg_Find(GetRoutingID(), request_id, search_text, options));
 }
 
 void WebContentsImpl::StopFinding(StopFindAction action) {
   // See if a top level browser plugin handles the stop finding request first.
-  if (browser_plugin_embedder_ &&
-      browser_plugin_embedder_->StopFinding(action)) {
-    return;
+  if (browser_plugin_embedder_) {
+    BrowserPluginGuest* guest = browser_plugin_embedder_->GetFullPageGuest();
+    if (guest && guest->StopFinding(action))
+      return;
   }
   Send(new ViewMsg_StopFinding(GetRoutingID(), action));
 }