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

Issue 7827016: Enable low-latency audio by default. (Closed)

Created:
9 years, 3 months ago by enal
Modified:
9 years, 2 months ago
CC:
chromium-reviews, hclam+watch_chromium.org, ddorwin+watch_chromium.org, fischman+watch_chromium.org, acolwell+watch_chromium.org, jam, annacc+watch_chromium.org, joi+watch-content_chromium.org, darin-cc_chromium.org, ajwong+watch_chromium.org, vrk (LEFT CHROMIUM), scherkus (not reviewing)
Visibility:
Public.

Description

Enable low-latency audio by default. Still keep old high-latency code, but it works only when "--disable-low-latency-audio" flag specified. Rename command-line flag. BUG=http://code.google.com/p/chromium/issues/detail?id=61022 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=103143

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -7 lines) Patch
M content/browser/renderer_host/browser_render_process_host.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M content/common/content_switches.h View 1 1 chunk +1 line, -1 line 0 comments Download
M content/common/content_switches.cc View 1 1 chunk +3 lines, -3 lines 0 comments Download
M content/renderer/media/audio_renderer_impl.cc View 1 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
enal
9 years, 3 months ago (2011-09-01 16:47:49 UTC) #1
brettw
LGTM, we sure we still have a bug around on removing the old high latency ...
9 years, 3 months ago (2011-09-01 16:50:42 UTC) #2
wjia(left Chromium)
lgtm.
9 years, 3 months ago (2011-09-01 16:59:21 UTC) #3
commit-bot: I haz the power
Try job failure for 7827016-1 (previous was lost) on win for steps "update_scripts, update". http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number=52778 ...
9 years, 3 months ago (2011-09-02 01:11:42 UTC) #4
commit-bot: I haz the power
Change committed as 99326
9 years, 3 months ago (2011-09-02 05:48:41 UTC) #5
commit-bot: I haz the power
CQ is trying tha patch. Follow status at https://chromium-status.appspot.com/cq/enal@chromium.org/7827016/5001
9 years, 2 months ago (2011-09-28 17:35:27 UTC) #6
commit-bot: I haz the power
9 years, 2 months ago (2011-09-28 18:34:48 UTC) #7
Change committed as 103143

Powered by Google App Engine
This is Rietveld 408576698