Gardening: Revert "Use AudioStreamMonitor to control power save blocking."

Gardening: Revert "Use AudioStreamMonitor to control power save blocking."

This reverts commit 484d29dcb45ca362fc9049c37dd3a51e11492f14.

This patch made media/media-controller-playbackrate.html test crash on DEBUG.

it crashes at content/browser/web_contents/web_contents_impl.cc:4296 on DCHECK
Looks like AddMediaPlayerEntry was called twice for the same player_cookie

It is not clear how it happens due to async nature of the notification.
The author is sleeping at the moment, so I decided to roll back the patch.

BUG=
TBR=dalecurtis@chromium.org

Committed: https://crrev.com/0381a74131887eb13b5d457cce756fb7066411ea
Cr-Commit-Position: refs/heads/master@{#294370}

[loislo, 2014-09-11 08:47:12]

The CQ bit was checked by loislo@chromium.org

[commit-bot: I haz the power, 2014-09-11 08:48:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/562863002/1

[commit-bot: I haz the power, 2014-09-11 10:16:39]

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

[commit-bot: I haz the power, 2014-09-11 10:16:40]

Try jobs failed on following builders:
  linux_chromium_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[loislo, 2014-09-11 10:21:10]

The CQ bit was checked by loislo@chromium.org

[commit-bot: I haz the power, 2014-09-11 10:21:42]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/562863002/1

[commit-bot: I haz the power, 2014-09-11 10:59:17]

Committed patchset #1 (id:1) as 78e91a8ed12501f8df63333c769d08a5e91a6d9d

[commit-bot: I haz the power, 2014-09-11 11:03:57]

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

[DaleCurtis, 2014-09-11 18:00:54]

Seems the BUG= got dropped when you reverted this.  Did you remove that
intentionally?

[loislo, 2014-09-11 18:35:48]

On 2014/09/11 18:00:54, DaleCurtis wrote:
> Seems the BUG= got dropped when you reverted this.  Did you remove that
> intentionally?

Ups, I reverted it manually (because the patch had no revert checkbox) and just
forgot to enter it. sorry.

[DaleCurtis, 2014-09-11 19:39:27]

Turns out the new DCHECKs were catching real bugs that prexisted my patch,
https://codereview.chromium.org/564943002/

I'll revert this revert once it lands.

[loislo, 2014-09-11 19:44:05]

great!!! :)

On Thu Sep 11 2014 at 11:39:29 PM <dalecurtis@chromium.org> wrote:

> Turns out the new DCHECKs were catching real bugs that prexisted my patch,
> https://codereview.chromium.org/564943002/
>
> I'll revert this revert once it lands.
>
> https://codereview.chromium.org/562863002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[DaleCurtis, 2014-09-12 17:03:36]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/569713002/ by dalecurtis@chromium.org.

The reason for reverting is: Bugs should be fixed now... will CQ this revert..