Simplify MediaStreamAudioSourceNode ownerships.

Simplify MediaStreamAudioSourceNode ownerships.

The processing that happens on the audio thread for this object
doesn't access the MediaStream. Hence the reference to it and the
track can be kept directly on the node object, thereby avoiding the
use of a pair of Persistent<>s.

R=haraken
BUG=693988
Review-Url: https://codereview.chromium.org/2699403002
Cr-Commit-Position: refs/heads/master@{#451726}
Committed: https://chromium.googlesource.com/chromium/src/+/e27847a80deaea0674c951a7be26dda066a05cb8

[sof, 2017-02-19 20:34:58]

The CQ bit was checked by sigbjornf@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-02-19 20:35:05]

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, 2017-02-19 22:12:31]

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

[commit-bot: I haz the power, 2017-02-19 22:12:32]

Dry run: This issue passed the CQ dry run.

[sof, 2017-02-20 07:24:34]

sigbjornf@opera.com changed reviewers:
+ chongz@chromium.org, haraken@chromium.org, rtoy@chromium.org

[sof, 2017-02-20 07:24:35]

please take a look.

while reviewing the use of Persistent<>s in Blink, I came across these. As the
audio thread doesn't make use of these objects, it seems better to have the
"Node" rather than the "Handler" store them. Fewer Persistent<>s results.

[sof, 2017-02-20 07:46:12]

Description was changed from

==========
Simplify MediaStreamAudioSourceNode ownerships.

The processing that happens on the audio thread for this object
doesn't access the MediaStream. Hence the reference to it and the
track can be kept directly on the node object, thereby avoiding the
use a pair of Persistent<>s.

R=
BUG=
==========

to

==========
Simplify MediaStreamAudioSourceNode ownerships.

The processing that happens on the audio thread for this object
doesn't access the MediaStream. Hence the reference to it and the
track can be kept directly on the node object, thereby avoiding the
use a pair of Persistent<>s.

R=
BUG=
==========

[sof, 2017-02-20 07:46:12]

sigbjornf@opera.com changed reviewers:
+ hongchan@chromium.org
- chongz@chromium.org

[haraken, 2017-02-20 09:34:07]

LGTM

[sof, 2017-02-20 10:05:16]

Description was changed from

==========
Simplify MediaStreamAudioSourceNode ownerships.

The processing that happens on the audio thread for this object
doesn't access the MediaStream. Hence the reference to it and the
track can be kept directly on the node object, thereby avoiding the
use a pair of Persistent<>s.

R=
BUG=
==========

to

==========
Simplify MediaStreamAudioSourceNode ownerships.

The processing that happens on the audio thread for this object
doesn't access the MediaStream. Hence the reference to it and the
track can be kept directly on the node object, thereby avoiding the
use of a pair of Persistent<>s.

R=haraken
BUG=
==========

[sof, 2017-02-21 08:41:26]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2017-02-21 08:41:39]

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

[sof, 2017-02-21 11:00:59]

Description was changed from

==========
Simplify MediaStreamAudioSourceNode ownerships.

The processing that happens on the audio thread for this object
doesn't access the MediaStream. Hence the reference to it and the
track can be kept directly on the node object, thereby avoiding the
use of a pair of Persistent<>s.

R=haraken
BUG=
==========

to

==========
Simplify MediaStreamAudioSourceNode ownerships.

The processing that happens on the audio thread for this object
doesn't access the MediaStream. Hence the reference to it and the
track can be kept directly on the node object, thereby avoiding the
use of a pair of Persistent<>s.

R=haraken
BUG=693988
==========

[commit-bot: I haz the power, 2017-02-21 11:17:07]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487666486329530, "parent_rev":
"8cfb8018660f8a412fed47d066f0d92913c3556c", "commit_rev":
"e27847a80deaea0674c951a7be26dda066a05cb8"}

[commit-bot: I haz the power, 2017-02-21 11:18:13]

Description was changed from

==========
Simplify MediaStreamAudioSourceNode ownerships.

The processing that happens on the audio thread for this object
doesn't access the MediaStream. Hence the reference to it and the
track can be kept directly on the node object, thereby avoiding the
use of a pair of Persistent<>s.

R=haraken
BUG=693988
==========

to

==========
Simplify MediaStreamAudioSourceNode ownerships.

The processing that happens on the audio thread for this object
doesn't access the MediaStream. Hence the reference to it and the
track can be kept directly on the node object, thereby avoiding the
use of a pair of Persistent<>s.

R=haraken
BUG=693988

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

[commit-bot: I haz the power, 2017-02-21 11:18:14]

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