Context menu for "Voice recognition options"

content/browser/renderer_host/render_view_host.cc

Index: content/browser/renderer_host/render_view_host.cc
diff --git a/content/browser/renderer_host/render_view_host.cc b/content/browser/renderer_host/render_view_host.cc
index 2376d887da51e22987dc3f71df514e1fe0a1a0a8..c5bb00724470ee48525db162edcd7cf1a434afa1 100644
--- a/content/browser/renderer_host/render_view_host.cc
+++ b/content/browser/renderer_host/render_view_host.cc
@@ -15,7 +15,9 @@
 #include "base/time.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
+#include "chrome/browser/google/google_util.h"

[jam, 2011/07/19 23:06:54]

you're going to get a deps failure for including chrome paths from content (see
http://dev.chromium.org/developers/content-module).

it seems unnecessary to add a method to RVH that is called by chrome code which
then just calls out to chrome. so in the right click menu handler (in chrome),
make it do whatever you want directly

[gshires, 2011/07/20 13:48:43]

Done.

[jam, 2011/07/20 15:49:27]

thanks!

 #include "chrome/browser/profiles/profile.h"
+#include "chrome/common/url_constants.h"
 #include "content/browser/browser_message_filter.h"
 #include "content/browser/child_process_security_policy.h"
 #include "content/browser/content_browser_client.h"
@@ -511,6 +513,13 @@ void RenderViewHost::SelectAll() {
   UserMetrics::RecordAction(UserMetricsAction("SelectAll"));
 }
 
+void RenderViewHost::SpeechInputAbout() {
+  GURL about_url(chrome::kSpeechInputAboutURL);
+  GURL localized_about_url = google_util::AppendGoogleLocaleParam(about_url);
+  // Open URL with no referrer field (because user clicked on menu item).
+  delegate_->RequestOpenURL(localized_about_url, GURL(), NEW_FOREGROUND_TAB);
+}
+
 void RenderViewHost::JavaScriptDialogClosed(IPC::Message* reply_msg,
                                             bool success,
                                             const string16& user_input) {