Remove AudioOutputDeviceEnumerator. Replace usage with MediaDevicesManager.

Remove AudioOutputDeviceEnumerator. Replace usage with MediaDevicesManager.

Drive-by:
  Stop using std::array literals to specify devices to enumerate. Use device types as indexes.
  This makes it easier to understand what is being requested and removes reliance on the order of elements in the array.

BUG=647660
Committed: https://crrev.com/63931c20853b821d960649bb5ccd9b2f0332f245
Cr-Commit-Position: refs/heads/master@{#420959}

[Guido Urdaneta, 2016-09-26 10:26:53]

Description was changed from

==========
Remove AudioOutputDeviceEnumerator. Replace usage with MediaDevicesManager.

BUG=647660
==========

to

==========
Remove AudioOutputDeviceEnumerator. Replace usage with MediaDevicesManager.

Drive-by: 
  Stop using std::array literals to specify devices to enumerate. Use device
types as indexes.
  This makes it easier to understand what is being requested and removes
reliance on the order in the array.


BUG=647660
==========

[Guido Urdaneta, 2016-09-26 10:27:49]

Patchset #1 (id:1) has been deleted

[Guido Urdaneta, 2016-09-26 10:29:45]

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

[Guido Urdaneta, 2016-09-26 10:29:55]

Hi, PTAL

[Guido Urdaneta, 2016-09-26 10:30:13]

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

[commit-bot: I haz the power, 2016-09-26 10:30:39]

Dry run: 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-26 11:54:24]

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

[commit-bot: I haz the power, 2016-09-26 11:54:25]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[tommi (sloooow) - chröme, 2016-09-26 12:41:18]

lgtm but I would like it if someone else could take a look to make sure...
olka@?

[Guido Urdaneta, 2016-09-26 13:33:13]

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

[Guido Urdaneta, 2016-09-26 13:34:01]

Moved olka@ from cc to actual reviewer.

[o1ka, 2016-09-26 14:28:18]

On 2016/09/26 13:34:01, Guido Urdaneta wrote:
> Moved olka@ from cc to actual reviewer.

Just give me some time to catch up on MediaDevicesManager :)

[Guido Urdaneta, 2016-09-26 15:26:53]

Description was changed from

==========
Remove AudioOutputDeviceEnumerator. Replace usage with MediaDevicesManager.

Drive-by: 
  Stop using std::array literals to specify devices to enumerate. Use device
types as indexes.
  This makes it easier to understand what is being requested and removes
reliance on the order in the array.


BUG=647660
==========

to

==========
Remove AudioOutputDeviceEnumerator. Replace usage with MediaDevicesManager.

Drive-by: 
  Stop using std::array literals to specify devices to enumerate. Use device
types as indexes.
  This makes it easier to understand what is being requested and removes
reliance on the order of elements in the array.


BUG=647660
==========

[o1ka, 2016-09-26 15:36:40]

On 2016/09/26 14:28:18, o1ka wrote:
> On 2016/09/26 13:34:01, Guido Urdaneta wrote:
> > Moved olka@ from cc to actual reviewer.
> 
> Just give me some time to catch up on MediaDevicesManager :)

lgtm
nice cleanup!

(cc miu@ in case he is interested in ARH modifications)

[Guido Urdaneta, 2016-09-26 16:24:44]

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

[Guido Urdaneta, 2016-09-26 16:26:32]

avi@: can you rs / look at BUILD.gn ?

[Guido Urdaneta, 2016-09-26 16:38:25]

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

[commit-bot: I haz the power, 2016-09-26 16:38:46]

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

[Avi (use Gerrit), 2016-09-26 17:23:36]

stampity stamp lgtm

[Guido Urdaneta, 2016-09-26 17:25:02]

The CQ bit was unchecked by guidou@chromium.org

[Guido Urdaneta, 2016-09-26 17:25:14]

The CQ bit was checked by guidou@chromium.org

[Guido Urdaneta, 2016-09-26 17:25:14]

The patchset sent to the CQ was uploaded after l-g-t-m from tommi@chromium.org,
olka@chromium.org
Link to the patchset: https://codereview.chromium.org/2368213002/#ps40001
(title: "Remove BoolDeviceTypes std::array literals")

[commit-bot: I haz the power, 2016-09-26 17:25:42]

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-26 19:22:23]

Description was changed from

==========
Remove AudioOutputDeviceEnumerator. Replace usage with MediaDevicesManager.

Drive-by: 
  Stop using std::array literals to specify devices to enumerate. Use device
types as indexes.
  This makes it easier to understand what is being requested and removes
reliance on the order of elements in the array.


BUG=647660
==========

to

==========
Remove AudioOutputDeviceEnumerator. Replace usage with MediaDevicesManager.

Drive-by: 
  Stop using std::array literals to specify devices to enumerate. Use device
types as indexes.
  This makes it easier to understand what is being requested and removes
reliance on the order of elements in the array.


BUG=647660
==========

[commit-bot: I haz the power, 2016-09-26 19:22:24]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-09-26 19:24:49]

Description was changed from

==========
Remove AudioOutputDeviceEnumerator. Replace usage with MediaDevicesManager.

Drive-by: 
  Stop using std::array literals to specify devices to enumerate. Use device
types as indexes.
  This makes it easier to understand what is being requested and removes
reliance on the order of elements in the array.


BUG=647660
==========

to

==========
Remove AudioOutputDeviceEnumerator. Replace usage with MediaDevicesManager.

Drive-by:
  Stop using std::array literals to specify devices to enumerate. Use device
types as indexes.
  This makes it easier to understand what is being requested and removes
reliance on the order of elements in the array.

BUG=647660

Committed: https://crrev.com/63931c20853b821d960649bb5ccd9b2f0332f245
Cr-Commit-Position: refs/heads/master@{#420959}
==========

[commit-bot: I haz the power, 2016-09-26 19:24:50]

Patchset 2 (id:??) landed as
https://crrev.com/63931c20853b821d960649bb5ccd9b2f0332f245
Cr-Commit-Position: refs/heads/master@{#420959}