Remove ContextLifecycleObserver from PresentationConnectionList

Remove ContextLifecycleObserver from PresentationConnectionList

Currently PresentationConnectionList is inheriting from ContextLifecycleObserver
just to implement EventTarget::getExecutionContext(). Instead, PresentationConnectionList
should just hold a strong member to ExecutionContext.

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

[haraken, 2016-12-07 07:37:09]

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

[haraken, 2016-12-07 07:37:10]

PTAL

https://codereview.chromium.org/2558643004/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/presentation/PresentationConnectionList.h
(right):

https://codereview.chromium.org/2558643004/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/presentation/PresentationConnectionList.h:53:
Member<ExecutionContext> m_executionContext;

Before this CL, it was a weak member. But I don't see any benefit to make it
weak, so changed it to a strong member.

[sof, 2016-12-07 07:42:12]

lgtm

https://codereview.chromium.org/2558643004/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/presentation/PresentationConnectionList.h
(right):

https://codereview.chromium.org/2558643004/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/presentation/PresentationConnectionList.h:8:
#include "core/dom/ContextLifecycleObserver.h"
Unused

[haraken, 2016-12-07 08:01:40]

On 2016/12/07 07:42:12, sof wrote:
> lgtm
> 
>
https://codereview.chromium.org/2558643004/diff/1/third_party/WebKit/Source/m...
> File
third_party/WebKit/Source/modules/presentation/PresentationConnectionList.h
> (right):
> 
>
https://codereview.chromium.org/2558643004/diff/1/third_party/WebKit/Source/m...
> third_party/WebKit/Source/modules/presentation/PresentationConnectionList.h:8:
> #include "core/dom/ContextLifecycleObserver.h"
> Unused

Fixed.

[haraken, 2016-12-07 08:01:44]

The CQ bit was checked by haraken@chromium.org

[haraken, 2016-12-07 08:01:44]

The patchset sent to the CQ was uploaded after l-g-t-m from sigbjornf@opera.com
Link to the patchset: https://codereview.chromium.org/2558643004/#ps20001
(title: "temp")

[commit-bot: I haz the power, 2016-12-07 08:01:56]

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:49:21]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481097704569700,
"parent_rev": "76ad88387a420020286114a713f7334a1422422e", "commit_rev":
"721e076a94305cd8a628ee9e3195d3e9833e6502"}

[commit-bot: I haz the power, 2016-12-07 09:49:45]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-07 09:51:13]

Description was changed from

==========
Remove ContextLifecycleObserver from PresentationConnectionList

Currently PresentationConnectionList is inheriting from ContextLifecycleObserver
just to implement EventTarget::getExecutionContext(). Instead,
PresentationConnectionList
should just hold a strong member to ExecutionContext.

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

to

==========
Remove ContextLifecycleObserver from PresentationConnectionList

Currently PresentationConnectionList is inheriting from ContextLifecycleObserver
just to implement EventTarget::getExecutionContext(). Instead,
PresentationConnectionList
should just hold a strong member to ExecutionContext.

BUG=610176

Committed: https://crrev.com/3a0ebdbbfbde6acd2782c30671d42292740762f7
Cr-Commit-Position: refs/heads/master@{#436901}
==========

[commit-bot: I haz the power, 2016-12-07 09:51:14]

Patchset 2 (id:??) landed as
https://crrev.com/3a0ebdbbfbde6acd2782c30671d42292740762f7
Cr-Commit-Position: refs/heads/master@{#436901}