Remove RefPtr<ScriptState> from MediaSession

Remove RefPtr<ScriptState> from MediaSession

RefPtr<ScriptState> stored in DOM objects has a risk of leaking ScriptStates
to unexpected worlds. We should explicitly pass in ScriptState when we need it.

This CL makes MediaSession a ContextLifecycleObserver.

BUG=669812
Committed: https://crrev.com/a1229b7ad6ec062281bcd389dcc1ca93b49f5d2f
Cr-Commit-Position: refs/heads/master@{#436853}

[haraken, 2016-12-02 07:23:43]

haraken@chromium.org changed reviewers:
+ yukishiino@chromium.org

[haraken, 2016-12-02 07:23:44]

PTAL

[haraken, 2016-12-02 07:30:04]

As far as I grep, I think this is the final CL to remove RefPtr<ScriptState>
from DOM objects that have IDL interfaces.

[Yuki, 2016-12-02 07:45:55]

The CQ bit was checked by yukishiino@chromium.org

[Yuki, 2016-12-02 07:45:55]

lgtm

[commit-bot: I haz the power, 2016-12-02 07:46:17]

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-12-02 09:37:22]

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

[commit-bot: I haz the power, 2016-12-02 09:37:22]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[haraken, 2016-12-06 05:58:50]

The CQ bit was checked by haraken@chromium.org

[haraken, 2016-12-06 05:58:51]

The patchset sent to the CQ was uploaded after l-g-t-m from
yukishiino@chromium.org
Link to the patchset: https://codereview.chromium.org/2545893003/#ps60001
(title: "temp")

[commit-bot: I haz the power, 2016-12-06 05:59:08]

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-12-06 06:45:38]

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

[commit-bot: I haz the power, 2016-12-06 06:45:38]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[haraken, 2016-12-07 02:11:26]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-12-07 02:11:51]

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-12-07 03:42:52]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1481076686573740,
"parent_rev": "1db823fc7ae771016033c893f7dcfcdcef73ec9f", "commit_rev":
"32fc849ab39857f25f1a6e99cddf46e866f526f3"}

[commit-bot: I haz the power, 2016-12-07 03:43:15]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-12-07 03:47:10]

Description was changed from

==========
Remove RefPtr<ScriptState> from MediaSession

RefPtr<ScriptState> stored in DOM objects has a risk of leaking ScriptStates
to unexpected worlds. We should explicitly pass in ScriptState when we need it.

This CL makes MediaSession a ContextLifecycleObserver.

BUG=669812
==========

to

==========
Remove RefPtr<ScriptState> from MediaSession

RefPtr<ScriptState> stored in DOM objects has a risk of leaking ScriptStates
to unexpected worlds. We should explicitly pass in ScriptState when we need it.

This CL makes MediaSession a ContextLifecycleObserver.

BUG=669812

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

[commit-bot: I haz the power, 2016-12-07 03:47:11]

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