Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(147)

Issue 6353005: Revert r70896 to move speech input back behind a flag. (Closed)

Created:
9 years, 11 months ago by Satish
Modified:
9 years, 7 months ago
Reviewers:
hans
CC:
chromium-reviews, Paweł Hajdan Jr., darin-cc_chromium.org, brettw-cc_chromium.org
Visibility:
Public.

Description

Revert http://src.chromium.org/viewvc/chrome?view=rev&revision=70896 to move speech input back behind a flag. We have more work to do and the feature isn't ready for M10 so moving it back behind a flag. BUG=61677 TEST=browser_tests --gtest_filter="SpeechInputBrowserTest.*" Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=71789

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+82 lines, -18 lines) Patch
M chrome/browser/renderer_host/browser_render_process_host.h View 1 chunk +4 lines, -0 lines 0 comments Download
M chrome/browser/renderer_host/browser_render_process_host.cc View 3 chunks +7 lines, -0 lines 0 comments Download
M chrome/browser/speech/speech_input_browsertest.cc View 1 chunk +0 lines, -4 lines 0 comments Download
M chrome/browser/speech/speech_input_dispatcher_host.cc View 1 chunk +23 lines, -12 lines 0 comments Download
M chrome/browser/speech/speech_input_manager.h View 1 chunk +4 lines, -0 lines 0 comments Download
M chrome/browser/speech/speech_input_manager.cc View 2 chunks +21 lines, -0 lines 0 comments Download
M chrome/common/chrome_switches.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/common/chrome_switches.cc View 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/common/render_messages_internal.h View 1 chunk +4 lines, -0 lines 0 comments Download
M chrome/renderer/render_thread.h View 2 chunks +6 lines, -0 lines 0 comments Download
M chrome/renderer/render_thread.cc View 3 chunks +9 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Satish
9 years, 11 months ago (2011-01-19 11:58:40 UTC) #1
hans
9 years, 11 months ago (2011-01-19 12:14:27 UTC) #2
On 2011/01/19 11:58:40, Satish wrote:

lgtm

Powered by Google App Engine
This is Rietveld 408576698