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

Unified Diff: chromecast/media/cma/backend/alsa/stream_mixer_alsa.cc

Issue 1767603003: [chromecast] Add cast_shell --accept-resource-provider switch (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: bool -> int Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chromecast/media/cma/backend/alsa/stream_mixer_alsa.cc
diff --git a/chromecast/media/cma/backend/alsa/stream_mixer_alsa.cc b/chromecast/media/cma/backend/alsa/stream_mixer_alsa.cc
index 2f1afc7f9d41a45cb4cbe59754ca1fec65ae1e95..aec64cf2b3831e6f3d31dc2ee1fd3d16efe50a96 100644
--- a/chromecast/media/cma/backend/alsa/stream_mixer_alsa.cc
+++ b/chromecast/media/cma/backend/alsa/stream_mixer_alsa.cc
@@ -256,9 +256,13 @@ void StreamMixerAlsa::DefineAlsaParameters() {
}
alsa_avail_min_ = avail_min;
+ // --accept-resource-provider should imply a check close timeout of 0.
+ int default_close_timeout =
+ base::GetSwitchValueBoolean(switches::kAcceptResourceProvider)
+ ? 0
+ : kDefaultCheckCloseTimeoutMs;
byungchul 2016/03/07 17:19:40 Is this a right indentation? Why isn't it: int de
jyw 2016/03/08 02:09:24 git cl format is a no-op.
GetSwitchValueAsNonNegativeInt(switches::kAlsaCheckCloseTimeout,
- kDefaultCheckCloseTimeoutMs,
- &check_close_timeout_);
+ default_close_timeout, &check_close_timeout_);
}
int StreamMixerAlsa::SetAlsaPlaybackParams() {
« chromecast/base/chromecast_switches.cc ('K') | « chromecast/base/chromecast_switches.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698