Reland of Remove dangling PlayingState pointers in WebRtcAudioRenderer.

Reland of Remove dangling PlayingState pointers in WebRtcAudioRenderer. (patchset #1 id:1 of https://codereview.chromium.org/2856943003/ )

Reason for revert:
Relanding this as reverting didn't improve the metrics.

Original issue's description:
> Revert of Remove dangling PlayingState pointers in WebRtcAudioRenderer. (patchset #4 id:60001 of https://codereview.chromium.org/2758453004/ )
> 
> Reason for revert:
> This causes a memory use increase across the board on Android, even when WebRTC isn't used.
> 
> Original issue's description:
> > Remove dangling PlayingState pointers in WebRtcAudioRenderer.
> > 
> > This solution is a bit of a hack, but it works. Did manual testing and ran
> > unit tests. Bug 697256 does not reproduce with this patch.
> > 
> > R=tommi@chromium.org
> > 
> > BUG=697256
> > 
> > Review-Url: https://codereview.chromium.org/2758453004
> > Cr-Commit-Position: refs/heads/master@{#458025}
> > Committed: https://chromium.googlesource.com/chromium/src/+/ba21ffd704aebcc8a5e5e1ab1f072e9b79b51fcb
> 
> TBR=tommi@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=697256, 703743
> 
> Review-Url: https://codereview.chromium.org/2856943003
> Cr-Commit-Position: refs/heads/master@{#468984}
> Committed: https://chromium.googlesource.com/chromium/src/+/a5f87e7a6177d32e5cf9cfdc39753fd7753814e2

TBR=tommi@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=697256, 703743
Review-Url: https://codereview.chromium.org/2868013002
Cr-Commit-Position: refs/heads/master@{#470288}
Committed: https://chromium.googlesource.com/chromium/src/+/abd2c36fd30e3b814a4f0f5ce024dd583e71af1a

[Max Morin, 2017-05-09 10:40:36]

The CQ bit was checked by maxmorin@chromium.org

[Max Morin, 2017-05-09 10:40:36]

Created Reland of move dangling PlayingState pointers in WebRtcAudioRenderer.

[commit-bot: I haz the power, 2017-05-09 10:41:01]

CQ is trying da patch.

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

[Max Morin, 2017-05-09 10:49:26]

Description was changed from

==========
Reland of move dangling PlayingState pointers in WebRtcAudioRenderer. (patchset
#1 id:1 of https://codereview.chromium.org/2856943003/ )

Reason for revert:
Relanding this as reverting didn't improve the metrics.

Original issue's description:
> Revert of Remove dangling PlayingState pointers in WebRtcAudioRenderer.
(patchset #4 id:60001 of https://codereview.chromium.org/2758453004/ )
> 
> Reason for revert:
> This causes a memory use increase across the board on Android, even when
WebRTC isn't used.
> 
> Original issue's description:
> > Remove dangling PlayingState pointers in WebRtcAudioRenderer.
> > 
> > This solution is a bit of a hack, but it works. Did manual testing and ran
> > unit tests. Bug 697256 does not reproduce with this patch.
> > 
> > R=tommi@chromium.org
> > 
> > BUG=697256
> > 
> > Review-Url: https://codereview.chromium.org/2758453004
> > Cr-Commit-Position: refs/heads/master@{#458025}
> > Committed:
https://chromium.googlesource.com/chromium/src/+/ba21ffd704aebcc8a5e5e1ab1f07...
> 
> TBR=tommi@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=697256,703743
> 
> Review-Url: https://codereview.chromium.org/2856943003
> Cr-Commit-Position: refs/heads/master@{#468984}
> Committed:
https://chromium.googlesource.com/chromium/src/+/a5f87e7a6177d32e5cf9cfdc3975...

TBR=tommi@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=697256,703743
==========

to

==========
Reland of Remove dangling PlayingState pointers in WebRtcAudioRenderer.
(patchset #1 id:1 of https://codereview.chromium.org/2856943003/ )

Reason for revert:
Relanding this as reverting didn't improve the metrics.

Original issue's description:
> Revert of Remove dangling PlayingState pointers in WebRtcAudioRenderer.
(patchset #4 id:60001 of https://codereview.chromium.org/2758453004/ )
> 
> Reason for revert:
> This causes a memory use increase across the board on Android, even when
WebRTC isn't used.
> 
> Original issue's description:
> > Remove dangling PlayingState pointers in WebRtcAudioRenderer.
> > 
> > This solution is a bit of a hack, but it works. Did manual testing and ran
> > unit tests. Bug 697256 does not reproduce with this patch.
> > 
> > R=tommi@chromium.org
> > 
> > BUG=697256
> > 
> > Review-Url: https://codereview.chromium.org/2758453004
> > Cr-Commit-Position: refs/heads/master@{#458025}
> > Committed:
https://chromium.googlesource.com/chromium/src/+/ba21ffd704aebcc8a5e5e1ab1f07...
> 
> TBR=tommi@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=697256,703743
> 
> Review-Url: https://codereview.chromium.org/2856943003
> Cr-Commit-Position: refs/heads/master@{#468984}
> Committed:
https://chromium.googlesource.com/chromium/src/+/a5f87e7a6177d32e5cf9cfdc3975...

TBR=tommi@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=697256,703743
==========

[commit-bot: I haz the power, 2017-05-09 12:11:26]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494326436357520, "parent_rev":
"6fc6c3d5071745fe9d37f94a772b1e70231f96ee", "commit_rev":
"abd2c36fd30e3b814a4f0f5ce024dd583e71af1a"}

[commit-bot: I haz the power, 2017-05-09 12:11:44]

Description was changed from

==========
Reland of Remove dangling PlayingState pointers in WebRtcAudioRenderer.
(patchset #1 id:1 of https://codereview.chromium.org/2856943003/ )

Reason for revert:
Relanding this as reverting didn't improve the metrics.

Original issue's description:
> Revert of Remove dangling PlayingState pointers in WebRtcAudioRenderer.
(patchset #4 id:60001 of https://codereview.chromium.org/2758453004/ )
> 
> Reason for revert:
> This causes a memory use increase across the board on Android, even when
WebRTC isn't used.
> 
> Original issue's description:
> > Remove dangling PlayingState pointers in WebRtcAudioRenderer.
> > 
> > This solution is a bit of a hack, but it works. Did manual testing and ran
> > unit tests. Bug 697256 does not reproduce with this patch.
> > 
> > R=tommi@chromium.org
> > 
> > BUG=697256
> > 
> > Review-Url: https://codereview.chromium.org/2758453004
> > Cr-Commit-Position: refs/heads/master@{#458025}
> > Committed:
https://chromium.googlesource.com/chromium/src/+/ba21ffd704aebcc8a5e5e1ab1f07...
> 
> TBR=tommi@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=697256,703743
> 
> Review-Url: https://codereview.chromium.org/2856943003
> Cr-Commit-Position: refs/heads/master@{#468984}
> Committed:
https://chromium.googlesource.com/chromium/src/+/a5f87e7a6177d32e5cf9cfdc3975...

TBR=tommi@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=697256,703743
==========

to

==========
Reland of Remove dangling PlayingState pointers in WebRtcAudioRenderer.
(patchset #1 id:1 of https://codereview.chromium.org/2856943003/ )

Reason for revert:
Relanding this as reverting didn't improve the metrics.

Original issue's description:
> Revert of Remove dangling PlayingState pointers in WebRtcAudioRenderer.
(patchset #4 id:60001 of https://codereview.chromium.org/2758453004/ )
> 
> Reason for revert:
> This causes a memory use increase across the board on Android, even when
WebRTC isn't used.
> 
> Original issue's description:
> > Remove dangling PlayingState pointers in WebRtcAudioRenderer.
> > 
> > This solution is a bit of a hack, but it works. Did manual testing and ran
> > unit tests. Bug 697256 does not reproduce with this patch.
> > 
> > R=tommi@chromium.org
> > 
> > BUG=697256
> > 
> > Review-Url: https://codereview.chromium.org/2758453004
> > Cr-Commit-Position: refs/heads/master@{#458025}
> > Committed:
https://chromium.googlesource.com/chromium/src/+/ba21ffd704aebcc8a5e5e1ab1f07...
> 
> TBR=tommi@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=697256,703743
> 
> Review-Url: https://codereview.chromium.org/2856943003
> Cr-Commit-Position: refs/heads/master@{#468984}
> Committed:
https://chromium.googlesource.com/chromium/src/+/a5f87e7a6177d32e5cf9cfdc3975...

TBR=tommi@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=697256,703743

Review-Url: https://codereview.chromium.org/2868013002
Cr-Commit-Position: refs/heads/master@{#470288}
Committed:
https://chromium.googlesource.com/chromium/src/+/abd2c36fd30e3b814a4f0f5ce024...
==========

[commit-bot: I haz the power, 2017-05-09 12:11:46]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/abd2c36fd30e3b814a4f0f5ce024...