[Chromecast] Set use_alsa=true by default on Chromecast platforms.

[Chromecast] Set use_alsa=true by default on Chromecast platforms.

Chromecast code depends on media:: alsa code, so enable it by default.
use_pulseaudio will continue to be false when is_chromecast=true.

BUG= b/27105215
TEST= Builds on several Cast platforms.

Committed: https://crrev.com/09f6c021d48dd68fc24f4c1d4a1061a5564ea171
Cr-Commit-Position: refs/heads/master@{#374780}

[slan, 2016-02-10 00:03:29]

Description was changed from

==========
[Chromecast] Set use_alsa=true by defauly on Chromecast platforms.

Chromecast code depends on media:: alsa code, so enable it by default.
use_pulseaudio will continue to be false when is_chromecast=true.

BUG= b/27105215
TEST= Builds on several Cast platforms.
==========

to

==========
[Chromecast] Set use_alsa=true by defauly on Chromecast platforms.

Chromecast code depends on media:: alsa code, so enable it by default.
use_pulseaudio will continue to be false when is_chromecast=true.

BUG= b/27105215
TEST= Builds on several Cast platforms.
==========

[slan, 2016-02-10 00:03:47]

Description was changed from

==========
[Chromecast] Set use_alsa=true by defauly on Chromecast platforms.

Chromecast code depends on media:: alsa code, so enable it by default.
use_pulseaudio will continue to be false when is_chromecast=true.

BUG= b/27105215
TEST= Builds on several Cast platforms.
==========

to

==========
[Chromecast] Set use_alsa=true by default on Chromecast platforms.

Chromecast code depends on media:: alsa code, so enable it by default.
use_pulseaudio will continue to be false when is_chromecast=true.

BUG= b/27105215
TEST= Builds on several Cast platforms.
==========

[slan, 2016-02-10 00:04:08]

The CQ bit was checked by slan@chromium.org to run a CQ dry run

[slan, 2016-02-10 00:04:48]

slan@chromium.org changed reviewers:
+ alokp@chromium.org, dalecurtis@chromium.org, hallielaine@chromium.org

[commit-bot: I haz the power, 2016-02-10 00:06:45]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1678103007/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1678103007/1

[slan, 2016-02-10 01:14:41]

slan@chromium.org changed reviewers:
+ halliwell@chromium.org
- hallielaine@chromium.org

[commit-bot: I haz the power, 2016-02-10 01:42:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-10 01:42:51]

Dry run: This issue passed the CQ dry run.

[alokp, 2016-02-10 06:03:13]

lgtm

https://codereview.chromium.org/1678103007/diff/1/media/media.gyp
File media/media.gyp (right):

https://codereview.chromium.org/1678103007/diff/1/media/media.gyp#newcode16
media/media.gyp:16: ['(OS=="linux" or OS=="freebsd" or OS=="solaris") and
(embedded==0 or chromecast==1)', {
I am curious about the difference between gyp and gn. GN does not use embedded
flag?

[slan, 2016-02-10 16:05:23]

https://codereview.chromium.org/1678103007/diff/1/media/media.gyp
File media/media.gyp (right):

https://codereview.chromium.org/1678103007/diff/1/media/media.gyp#newcode16
media/media.gyp:16: ['(OS=="linux" or OS=="freebsd" or OS=="solaris") and
(embedded==0 or chromecast==1)', {
On 2016/02/10 06:03:13, alokp wrote:
> I am curious about the difference between gyp and gn. GN does not use embedded
> flag?

I *thought* that |embedded| was not used by builds other than Chromecast, but I
found out that that was an incorrect assumption the last time I touched this
file (https://codereview.chromium.org/1530713006), so I'm being proactive.
|embedded| will not be ported to GN.

[DaleCurtis, 2016-02-10 23:13:56]

lgtm

[slan, 2016-02-10 23:26:14]

The CQ bit was checked by slan@chromium.org

[commit-bot: I haz the power, 2016-02-10 23:28:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1678103007/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1678103007/1

[commit-bot: I haz the power, 2016-02-10 23:37:25]

Description was changed from

==========
[Chromecast] Set use_alsa=true by default on Chromecast platforms.

Chromecast code depends on media:: alsa code, so enable it by default.
use_pulseaudio will continue to be false when is_chromecast=true.

BUG= b/27105215
TEST= Builds on several Cast platforms.
==========

to

==========
[Chromecast] Set use_alsa=true by default on Chromecast platforms.

Chromecast code depends on media:: alsa code, so enable it by default.
use_pulseaudio will continue to be false when is_chromecast=true.

BUG= b/27105215
TEST= Builds on several Cast platforms.
==========

[commit-bot: I haz the power, 2016-02-10 23:37:26]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-16 22:32:33]

Description was changed from

==========
[Chromecast] Set use_alsa=true by default on Chromecast platforms.

Chromecast code depends on media:: alsa code, so enable it by default.
use_pulseaudio will continue to be false when is_chromecast=true.

BUG= b/27105215
TEST= Builds on several Cast platforms.
==========

to

==========
[Chromecast] Set use_alsa=true by default on Chromecast platforms.

Chromecast code depends on media:: alsa code, so enable it by default.
use_pulseaudio will continue to be false when is_chromecast=true.

BUG= b/27105215
TEST= Builds on several Cast platforms.

Committed: https://crrev.com/09f6c021d48dd68fc24f4c1d4a1061a5564ea171
Cr-Commit-Position: refs/heads/master@{#374780}
==========

[commit-bot: I haz the power, 2016-02-16 22:32:34]

Patchset 1 (id:??) landed as
https://crrev.com/09f6c021d48dd68fc24f4c1d4a1061a5564ea171
Cr-Commit-Position: refs/heads/master@{#374780}