summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorsatish@chromium.org <satish@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-08-28 08:29:40 +0000
committersatish@chromium.org <satish@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-08-28 08:29:40 +0000
commit0478d01629f03e3040cb2aee03c1fc6ace51bb8b (patch)
tree372f84dd3df628dbf849c7659920829de0d82d63
parentc0b5909a5e619cb174b5f601e49472939693dbc1 (diff)
downloadchromium_src-0478d01629f03e3040cb2aee03c1fc6ace51bb8b.zip
chromium_src-0478d01629f03e3040cb2aee03c1fc6ace51bb8b.tar.gz
chromium_src-0478d01629f03e3040cb2aee03c1fc6ace51bb8b.tar.bz2
Enable speech input by default on windows.
Makes --enable-speech-input with available only on non-windows builds and adds --disable-speech-input for windows. BUG=none TEST=browser_tests --gtest_filter="SpeechInputBrowserTest.*" Review URL: http://codereview.chromium.org/3261001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57781 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r--chrome/browser/renderer_host/browser_render_process_host.cc1
-rw-r--r--chrome/browser/speech/enable_speech_input_switch_browsertest.cc5
-rw-r--r--chrome/common/chrome_switches.cc3
-rw-r--r--chrome/common/chrome_switches.h1
-rw-r--r--chrome/renderer/render_thread.cc11
5 files changed, 19 insertions, 2 deletions
diff --git a/chrome/browser/renderer_host/browser_render_process_host.cc b/chrome/browser/renderer_host/browser_render_process_host.cc
index 8a355f6..b4727c7 100644
--- a/chrome/browser/renderer_host/browser_render_process_host.cc
+++ b/chrome/browser/renderer_host/browser_render_process_host.cc
@@ -543,6 +543,7 @@ void BrowserRenderProcessHost::PropagateBrowserCommandLineToRenderer(
switches::kDisableApplicationCache,
switches::kDisableDeviceOrientation,
switches::kEnableIndexedDatabase,
+ switches::kDisableSpeechInput,
switches::kEnableSpeechInput,
switches::kDisableGeolocation,
switches::kShowPaintRects,
diff --git a/chrome/browser/speech/enable_speech_input_switch_browsertest.cc b/chrome/browser/speech/enable_speech_input_switch_browsertest.cc
index 018cc013..72a6a72 100644
--- a/chrome/browser/speech/enable_speech_input_switch_browsertest.cc
+++ b/chrome/browser/speech/enable_speech_input_switch_browsertest.cc
@@ -18,6 +18,10 @@ class SpeechInputEnableSwitchTest : public InProcessBrowserTest {
}
};
+// Speech input is enabled by default in windows & mac builds, so this test
+// is not relevant for them.
+#if !defined(OS_WIN) && !defined(OS_MACOSX)
+
IN_PROC_BROWSER_TEST_F(SpeechInputEnableSwitchTest, UnavailabilityTest) {
// Test that speech input is not available to a web page if
// the runtime switch is disabled.
@@ -34,3 +38,4 @@ IN_PROC_BROWSER_TEST_F(SpeechInputEnableSwitchTest, UnavailabilityTest) {
EXPECT_EQ("pass", status) << "Page detected speech input properties.";
}
+#endif
diff --git a/chrome/common/chrome_switches.cc b/chrome/common/chrome_switches.cc
index b2c3073..d6cace2 100644
--- a/chrome/common/chrome_switches.cc
+++ b/chrome/common/chrome_switches.cc
@@ -252,6 +252,9 @@ const char kDisableSharedWorkers[] = "disable-shared-workers";
// Disable site-specific tailoring to compatibility issues in WebKit.
const char kDisableSiteSpecificQuirks[] = "disable-site-specific-quirks";
+// Disables speech input.
+const char kDisableSpeechInput[] = "disable-speech-input";
+
// Disable False Start in SSL and TLS connections.
const char kDisableSSLFalseStart[] = "disable-ssl-false-start";
diff --git a/chrome/common/chrome_switches.h b/chrome/common/chrome_switches.h
index 1cb299f..f6f4025 100644
--- a/chrome/common/chrome_switches.h
+++ b/chrome/common/chrome_switches.h
@@ -84,6 +84,7 @@ extern const char kDisableRestoreBackgroundContents[];
extern const char kDisableSessionStorage[];
extern const char kDisableSharedWorkers[];
extern const char kDisableSiteSpecificQuirks[];
+extern const char kDisableSpeechInput[];
extern const char kDisableSSLFalseStart[];
extern const char kDisableSync[];
extern const char kDisableSyncApps[];
diff --git a/chrome/renderer/render_thread.cc b/chrome/renderer/render_thread.cc
index d45a925..4ad6a3f 100644
--- a/chrome/renderer/render_thread.cc
+++ b/chrome/renderer/render_thread.cc
@@ -205,6 +205,14 @@ class RenderViewZoomer : public RenderViewVisitor {
DISALLOW_COPY_AND_ASSIGN(RenderViewZoomer);
};
+
+bool IsSpeechInputEnabled(const CommandLine& command_line) {
+#if defined(OS_WIN) || defined(OS_MACOSX)
+ return !command_line.HasSwitch(switches::kDisableSpeechInput);
+#else
+ return command_line.HasSwitch(switches::kEnableSpeechInput);
+#endif
+}
} // namespace
// When we run plugins in process, we actually run them on the render thread,
@@ -933,8 +941,7 @@ void RenderThread::EnsureWebKitInitialized() {
WebRuntimeFeatures::enableDeviceOrientation(
!command_line.HasSwitch(switches::kDisableDeviceOrientation));
- WebRuntimeFeatures::enableSpeechInput(
- command_line.HasSwitch(switches::kEnableSpeechInput));
+ WebRuntimeFeatures::enableSpeechInput(IsSpeechInputEnabled(command_line));
}
void RenderThread::IdleHandler() {