Use NullAudioSink in WebMediaPlayerImpl if the audio sink is not OK.

Use NullAudioSink in WebMediaPlayerImpl if the audio sink is not OK.

This implements the recommendation given in http://crbug.com/626862#c16

BUG=626862
Committed: https://crrev.com/36e6930d420b5549b3dfdde51ae22da03b3a01ff
Cr-Commit-Position: refs/heads/master@{#424982}

[Guido Urdaneta, 2016-10-12 09:33:40]

Description was changed from

==========
Use NullAudioSink in WebMediaPlayerImpl if the audio sink is not OK.

This implements the recommendation given in http://crbug.com/626862#c16

BUG=626862
==========

to

==========
Use NullAudioSink in WebMediaPlayerImpl if the audio sink is not OK.

This implements the recommendation given in http://crbug.com/626862#c16

BUG=626862
==========

[Guido Urdaneta, 2016-10-12 09:33:41]

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

[Guido Urdaneta, 2016-10-12 09:34:07]

Hi, PTAL

[o1ka, 2016-10-12 09:36:01]

On 2016/10/12 09:34:07, Guido Urdaneta wrote:
> Hi, PTAL

LGTM - thanks for doing that!

[Guido Urdaneta, 2016-10-12 09:38:25]

Patchset #1 (id:1) has been deleted

[Guido Urdaneta, 2016-10-12 09:39:29]

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

[commit-bot: I haz the power, 2016-10-12 09:39:41]

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-10-12 11:47:24]

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

[commit-bot: I haz the power, 2016-10-12 11:47:25]

Dry run: This issue passed the CQ dry run.

[chcunningham, 2016-10-12 23:46:36]

lgtm

[Guido Urdaneta, 2016-10-13 08:22:50]

The CQ bit was checked by guidou@chromium.org

[Guido Urdaneta, 2016-10-13 08:22:51]

The patchset sent to the CQ was uploaded after l-g-t-m from olka@chromium.org
Link to the patchset: https://codereview.chromium.org/2413653002/#ps20001
(title: "Update comment")

[commit-bot: I haz the power, 2016-10-13 08:23:09]

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-10-13 08:27:45]

Description was changed from

==========
Use NullAudioSink in WebMediaPlayerImpl if the audio sink is not OK.

This implements the recommendation given in http://crbug.com/626862#c16

BUG=626862
==========

to

==========
Use NullAudioSink in WebMediaPlayerImpl if the audio sink is not OK.

This implements the recommendation given in http://crbug.com/626862#c16

BUG=626862
==========

[commit-bot: I haz the power, 2016-10-13 08:27:46]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-10-13 08:30:18]

Description was changed from

==========
Use NullAudioSink in WebMediaPlayerImpl if the audio sink is not OK.

This implements the recommendation given in http://crbug.com/626862#c16

BUG=626862
==========

to

==========
Use NullAudioSink in WebMediaPlayerImpl if the audio sink is not OK.

This implements the recommendation given in http://crbug.com/626862#c16

BUG=626862

Committed: https://crrev.com/36e6930d420b5549b3dfdde51ae22da03b3a01ff
Cr-Commit-Position: refs/heads/master@{#424982}
==========

[commit-bot: I haz the power, 2016-10-13 08:30:19]

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