Make SystemMonitor report all audio device changes instead of just audio capture.

Make SystemMonitor report all audio device changes instead of just audio capture.

Linux and Windows were already reporting audio output device changes as
DEVTYPE_AUDIO_CAPTURE.

This CL basically renames DEVTYPE_AUDIO_CAPTURE to DEVTYPE_AUDIO and
makes the CrOS device monitor to report audio-output device changes in
addition to audio-capture devices.

The only platform that needs further work is Mac, which currently does
not report device changes for output-only devices such as USB
speakers.

BUG=587396
Committed: https://crrev.com/d6789a38d48d6e5fcf4703a1cbf7d1525706b082
Cr-Commit-Position: refs/heads/master@{#419136}

[Guido Urdaneta, 2016-09-09 16:29:44]

Description was changed from

==========
Make SystemMonitor report all audio device changes instead of just audio
capture.

Linux and Windows were already reporting audio output device changes as
DEVTYPE_AUDIO_CAPTURE.

This CL basically renames DEVTYPE_AUDIO_CAPTURE to DEVTYPE_AUDIO and
makes the CrOS device monitor to report audio-output device changes in
addition to audio-capture devices.

The only platform that needs further work is Mac, which currently does
not report audio-output changes for output-only devices such as USB
speakers.

BUG=587396
==========

to

==========
Make SystemMonitor report all audio device changes instead of just audio
capture.

Linux and Windows were already reporting audio output device changes as
DEVTYPE_AUDIO_CAPTURE.

This CL basically renames DEVTYPE_AUDIO_CAPTURE to DEVTYPE_AUDIO and
makes the CrOS device monitor to report audio-output device changes in
addition to audio-capture devices.

The only platform that needs further work is Mac, which currently does
not report device changes for output-only devices such as USB
speakers.

BUG=587396
==========

[Guido Urdaneta, 2016-09-09 16:31:22]

Patchset #1 (id:1) has been deleted

[Guido Urdaneta, 2016-09-09 16:31:39]

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

[commit-bot: I haz the power, 2016-09-09 16:32:01]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Guido Urdaneta, 2016-09-09 16:54:21]

guidou@chromium.org changed reviewers:
+ dalecurtis@chromium.org, dcheng@chromium.org, jennyz@chromium.org,
tommi@chromium.org

[Guido Urdaneta, 2016-09-09 16:54:22]

jennyz@chromium.org: Please review changes in chromeos/
audio
dcheng@chromium.org: Please review changes in base/

tommi@chromium.org: Please review changes in content/

dalecurtis@chromium.org: Please review changes in media/

mek@chromium.org: Please review changes in chrome/browser/extensions

[DaleCurtis, 2016-09-09 17:36:58]

If this is just a name change, media/ lgtm

[Guido Urdaneta, 2016-09-09 17:38:44]

On 2016/09/09 17:36:58, DaleCurtis wrote:
> If this is just a name change, media/ lgtm

It's just a name change, except for chromeos/audio/cras_audio_handler.cc

[commit-bot: I haz the power, 2016-09-09 18:29:32]

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

[commit-bot: I haz the power, 2016-09-09 18:29:33]

Dry run: This issue passed the CQ dry run.

[dcheng, 2016-09-09 18:41:42]

rs lgtm for //base

[tommi (sloooow) - chröme, 2016-09-09 18:49:50]

The CQ bit was checked by tommi@chromium.org

[tommi (sloooow) - chröme, 2016-09-09 18:49:50]

lgtm

[commit-bot: I haz the power, 2016-09-09 18:50:14]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-09 18:56:33]

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

[commit-bot: I haz the power, 2016-09-09 18:56:34]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Guido Urdaneta, 2016-09-13 10:08:55]

guidou@chromium.org changed reviewers:
+ mek@google.com, steel@chromium.org
- jennyz@chromium.org

[Guido Urdaneta, 2016-09-13 10:09:32]

guidou@chromium.org changed reviewers:
- mek@google.com

[Guido Urdaneta, 2016-09-13 10:12:13]

guidou@chromium.org changed reviewers:
+ lazyboy@chromium.org

[Guido Urdaneta, 2016-09-13 10:13:12]

steel@chromium.org: please review changes in chromeos/audio

lazyboy@chromium.org: please review changes in chrome/browser/extensions

[lazyboy, 2016-09-13 18:56:29]

chrome/browser/extensions/* lgtm.

[Rahul Chaturvedi, 2016-09-13 21:44:32]

steel@chromium.org changed reviewers:
+ jennyz@chromium.org

[Rahul Chaturvedi, 2016-09-13 21:44:33]

Adding Jenny for audio review. I'll be removing myself from OWNERs for audio, it
has been years since I worked on it.

[Guido Urdaneta, 2016-09-15 09:24:49]

guidou@chromium.org changed reviewers:
+ hshi@chromium.org

[Guido Urdaneta, 2016-09-15 09:25:33]

hshi@: can you also take a look at chromos/audio?

[jennyz, 2016-09-15 21:26:45]

lgtm

[Guido Urdaneta, 2016-09-16 08:11:28]

The CQ bit was checked by guidou@chromium.org

[commit-bot: I haz the power, 2016-09-16 08:11:43]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-16 09:29:40]

Description was changed from

==========
Make SystemMonitor report all audio device changes instead of just audio
capture.

Linux and Windows were already reporting audio output device changes as
DEVTYPE_AUDIO_CAPTURE.

This CL basically renames DEVTYPE_AUDIO_CAPTURE to DEVTYPE_AUDIO and
makes the CrOS device monitor to report audio-output device changes in
addition to audio-capture devices.

The only platform that needs further work is Mac, which currently does
not report device changes for output-only devices such as USB
speakers.

BUG=587396
==========

to

==========
Make SystemMonitor report all audio device changes instead of just audio
capture.

Linux and Windows were already reporting audio output device changes as
DEVTYPE_AUDIO_CAPTURE.

This CL basically renames DEVTYPE_AUDIO_CAPTURE to DEVTYPE_AUDIO and
makes the CrOS device monitor to report audio-output device changes in
addition to audio-capture devices.

The only platform that needs further work is Mac, which currently does
not report device changes for output-only devices such as USB
speakers.

BUG=587396
==========

[commit-bot: I haz the power, 2016-09-16 09:29:41]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-09-16 09:31:11]

Description was changed from

==========
Make SystemMonitor report all audio device changes instead of just audio
capture.

Linux and Windows were already reporting audio output device changes as
DEVTYPE_AUDIO_CAPTURE.

This CL basically renames DEVTYPE_AUDIO_CAPTURE to DEVTYPE_AUDIO and
makes the CrOS device monitor to report audio-output device changes in
addition to audio-capture devices.

The only platform that needs further work is Mac, which currently does
not report device changes for output-only devices such as USB
speakers.

BUG=587396
==========

to

==========
Make SystemMonitor report all audio device changes instead of just audio
capture.

Linux and Windows were already reporting audio output device changes as
DEVTYPE_AUDIO_CAPTURE.

This CL basically renames DEVTYPE_AUDIO_CAPTURE to DEVTYPE_AUDIO and
makes the CrOS device monitor to report audio-output device changes in
addition to audio-capture devices.

The only platform that needs further work is Mac, which currently does
not report device changes for output-only devices such as USB
speakers.

BUG=587396

Committed: https://crrev.com/d6789a38d48d6e5fcf4703a1cbf7d1525706b082
Cr-Commit-Position: refs/heads/master@{#419136}
==========

[commit-bot: I haz the power, 2016-09-16 09:31:13]

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

[Guido Urdaneta, 2017-03-02 18:07:22]

Patchset #2 (id:40001) has been deleted

[Guido Urdaneta, 2017-03-02 18:07:34]

Patchset #2 (id:60001) has been deleted