Call ContextLifecycleObserver::contextDestroyed() when a window object is reset

Call ContextLifecycleObserver::contextDestroyed() when a window object is reset

Currently, when a window object is reset, DOMWindowProperty::frameDestroyed() is called but
ContextLifecycleObserver::contextDestroyed() is not called.
This CL makes the latter get called as well.

This is a preparation for replacing DOMWindowProperty with ContextLifecycleObsever.

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

[haraken, 2016-09-28 02:34:53]

Description was changed from

==========
Call contextDestroyed() when a window object is reset()

BUG=
==========

to

==========
Call contextDestroyed() when a window object is reset

BUG=
==========

[haraken, 2016-09-28 02:35:03]

The CQ bit was checked by haraken@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-28 02:35:17]

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, 2016-09-28 04:05:21]

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

[commit-bot: I haz the power, 2016-09-28 04:05:22]

Dry run: This issue passed the CQ dry run.

[haraken, 2016-09-28 04:25:58]

Description was changed from

==========
Call contextDestroyed() when a window object is reset

BUG=
==========

to

==========
Call ContextLifecycleObserver::contextDestroyed() when a window object is reset

Currently, when a window object is reset, DOMWindowProperty::frameDestroyed() is
called but
ContextLifecycleObserver::contextDestroyed() is not called.
This CL makes the latter get called as well.

This is a preparation for replacing DOMWindowProperty with
ContextLifecycleObsever.

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

[haraken, 2016-09-28 04:25:58]

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

[haraken, 2016-09-28 04:26:06]

PTAL

[dcheng, 2016-09-28 20:14:34]

On 2016/09/28 04:26:06, haraken wrote:
> PTAL

LGTM. I feel like the naming of reset() is a bit confusing, but that's not
something that should be fixed in this CL anyway.

[haraken, 2016-09-29 00:04:02]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-09-29 00:04:35]

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-09-29 00:10:29]

Description was changed from

==========
Call ContextLifecycleObserver::contextDestroyed() when a window object is reset

Currently, when a window object is reset, DOMWindowProperty::frameDestroyed() is
called but
ContextLifecycleObserver::contextDestroyed() is not called.
This CL makes the latter get called as well.

This is a preparation for replacing DOMWindowProperty with
ContextLifecycleObsever.

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

to

==========
Call ContextLifecycleObserver::contextDestroyed() when a window object is reset

Currently, when a window object is reset, DOMWindowProperty::frameDestroyed() is
called but
ContextLifecycleObserver::contextDestroyed() is not called.
This CL makes the latter get called as well.

This is a preparation for replacing DOMWindowProperty with
ContextLifecycleObsever.

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

[commit-bot: I haz the power, 2016-09-29 00:10:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-29 00:14:45]

Description was changed from

==========
Call ContextLifecycleObserver::contextDestroyed() when a window object is reset

Currently, when a window object is reset, DOMWindowProperty::frameDestroyed() is
called but
ContextLifecycleObserver::contextDestroyed() is not called.
This CL makes the latter get called as well.

This is a preparation for replacing DOMWindowProperty with
ContextLifecycleObsever.

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

to

==========
Call ContextLifecycleObserver::contextDestroyed() when a window object is reset

Currently, when a window object is reset, DOMWindowProperty::frameDestroyed() is
called but
ContextLifecycleObserver::contextDestroyed() is not called.
This CL makes the latter get called as well.

This is a preparation for replacing DOMWindowProperty with
ContextLifecycleObsever.

BUG=610176

Committed: https://crrev.com/4034fc1d8c67c3c6069c20047f55aee7b219e8cf
Cr-Commit-Position: refs/heads/master@{#421690}
==========

[commit-bot: I haz the power, 2016-09-29 00:14:46]

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