Restricting output device authorization timeout to Windows only.

Restricting output device authorization timeout to Windows only.

Previous CL on authorization timeout decrease is being reverted https://codereview.chromium.org/2106373002/, this one should be landed instead.

The reason is we hit authorization timeout on platforms other than Windows (https://uma.googleplex.com/p/chrome/timeline_v2?sid=74e6fc3b3d0ef041f64f37272402ebb1), but the goal was to address windows driver hangs only.

BUG=615589
Committed: https://crrev.com/b9debcdd14086d59327dcb0cfd2044b41c998ae8
Cr-Commit-Position: refs/heads/master@{#403164}

[Guido Urdaneta, 2016-06-30 09:11:02]

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

[Guido Urdaneta, 2016-06-30 09:11:02]

lgtm

https://codereview.chromium.org/2116443002/diff/1/content/renderer/media/audi...
File content/renderer/media/audio_device_factory.cc (right):

https://codereview.chromium.org/2116443002/diff/1/content/renderer/media/audi...
content/renderer/media/audio_device_factory.cc:28: const int64_t
kMaxAuthorizationTimeoutMs = 900;
Add a comment explaining why the timeout is necessary on Windows.
Make a reference to the bug.

[o1ka, 2016-06-30 10:31:01]

Patchset #2 (id:20001) has been deleted

[o1ka, 2016-06-30 10:59:45]

Patchset #2 (id:40001) has been deleted

[o1ka, 2016-06-30 11:04:09]

Description was changed from

==========
Restricting output device authorization timeout to Windows only.

BUG=615589
==========

to

==========
Restricting output device authorization timeout to Windows only.

Previous CL on authorization timeout decrease is being reverted
https://codereview.chromium.org/2106373002/, this one should be landed instead.

The reason is we hit authorization timeout on platforms other than Windows
(https://uma.googleplex.com/p/chrome/timeline_v2?sid=74e6fc3b3d0ef041f64f37272...),
but the goal was to address windows driver hangs only.

BUG=615589
==========

[o1ka, 2016-06-30 11:04:28]

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

[o1ka, 2016-06-30 11:04:28]

tommi@ PTAL

[tommi (sloooow) - chröme, 2016-06-30 11:13:44]

lgtm

[o1ka, 2016-06-30 12:03:07]

The CQ bit was checked by olka@chromium.org

[o1ka, 2016-06-30 12:03:07]

The patchset sent to the CQ was uploaded after l-g-t-m from guidou@chromium.org,
tommi@chromium.org
Link to the patchset: https://codereview.chromium.org/2116443002/#ps80001
(title: "rebasing to the revert")

[commit-bot: I haz the power, 2016-06-30 12:03:20]

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-06-30 13:29:20]

Description was changed from

==========
Restricting output device authorization timeout to Windows only.

Previous CL on authorization timeout decrease is being reverted
https://codereview.chromium.org/2106373002/, this one should be landed instead.

The reason is we hit authorization timeout on platforms other than Windows
(https://uma.googleplex.com/p/chrome/timeline_v2?sid=74e6fc3b3d0ef041f64f37272...),
but the goal was to address windows driver hangs only.

BUG=615589
==========

to

==========
Restricting output device authorization timeout to Windows only.

Previous CL on authorization timeout decrease is being reverted
https://codereview.chromium.org/2106373002/, this one should be landed instead.

The reason is we hit authorization timeout on platforms other than Windows
(https://uma.googleplex.com/p/chrome/timeline_v2?sid=74e6fc3b3d0ef041f64f37272...),
but the goal was to address windows driver hangs only.

BUG=615589
==========

[commit-bot: I haz the power, 2016-06-30 13:29:22]

Committed patchset #3 (id:80001)

[commit-bot: I haz the power, 2016-06-30 13:30:53]

Description was changed from

==========
Restricting output device authorization timeout to Windows only.

Previous CL on authorization timeout decrease is being reverted
https://codereview.chromium.org/2106373002/, this one should be landed instead.

The reason is we hit authorization timeout on platforms other than Windows
(https://uma.googleplex.com/p/chrome/timeline_v2?sid=74e6fc3b3d0ef041f64f37272...),
but the goal was to address windows driver hangs only.

BUG=615589
==========

to

==========
Restricting output device authorization timeout to Windows only.

Previous CL on authorization timeout decrease is being reverted
https://codereview.chromium.org/2106373002/, this one should be landed instead.

The reason is we hit authorization timeout on platforms other than Windows
(https://uma.googleplex.com/p/chrome/timeline_v2?sid=74e6fc3b3d0ef041f64f37272...),
but the goal was to address windows driver hangs only.

BUG=615589

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

[commit-bot: I haz the power, 2016-06-30 13:30:54]

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