Remove ContextLifecycleObserver from RemotePlayback

Remove ContextLifecycleObserver from RemotePlayback

We have made RemotePlayback a ContextLifecycleObserver to make
RemotePlayback::hasPendingActivity return false after the context gets destroyed.
We can do the same thing by checking executionContext->isContextDestroyed
in RemotePlayback::hasPendingActivity. This CL does that and removes
ContextLifecycleObserver from RemotePlayback.

BUG=610176
Committed: https://crrev.com/ee53c427022b948b552f9c4492d8ff8667e277a9
Cr-Commit-Position: refs/heads/master@{#436900}

[haraken, 2016-12-07 08:00:11]

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

[haraken, 2016-12-07 08:00:11]

PTAL

I'm sorry -- I know this CL is something you tried to land but I declined before
:/

Now I'm trying to reduce # of ContextLifecycleObservers and noticed that your
approach is smarter from that perspective.

[Michael Lippautz, 2016-12-07 08:09:08]

lgtm, no worries

[haraken, 2016-12-07 08:15:36]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-12-07 08:15:55]

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 09:41:27]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481098536379060, "parent_rev":
"1d1eef81f9ec8adcbd3a0174738a002308164c81", "commit_rev":
"76ad88387a420020286114a713f7334a1422422e"}

[commit-bot: I haz the power, 2016-12-07 09:41:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-07 09:47:26]

Description was changed from

==========
Remove ContextLifecycleObserver from RemotePlayback

We have made RemotePlayback a ContextLifecycleObserver to make
RemotePlayback::hasPendingActivity return false after the context gets
destroyed.
We can do the same thing by checking executionContext->isContextDestroyed
in RemotePlayback::hasPendingActivity. This CL does that and removes
ContextLifecycleObserver from RemotePlayback.

BUG=610176
==========

to

==========
Remove ContextLifecycleObserver from RemotePlayback

We have made RemotePlayback a ContextLifecycleObserver to make
RemotePlayback::hasPendingActivity return false after the context gets
destroyed.
We can do the same thing by checking executionContext->isContextDestroyed
in RemotePlayback::hasPendingActivity. This CL does that and removes
ContextLifecycleObserver from RemotePlayback.

BUG=610176

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

[commit-bot: I haz the power, 2016-12-07 09:47:27]

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