InstalledAppController should use ContextLifecycleObserver instead of DOMWindowProperty

InstalledAppController should use ContextLifecycleObserver instead of DOMWindowProperty

DOMWindowProperty::frameDestroyed is now equivalent to ContextLifecycleObserver::contextDestroyed.
This CL does the replacement for InstalledAppController.

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

[haraken, 2016-12-13 07:35:10]

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

[haraken, 2016-12-13 07:35:11]

PTAL

[sof, 2016-12-13 08:27:33]

lgtm

update the description, changing the last occurrence of InstalledAppController
to be ContextLifecycleObserver instead.

(The title would perhaps be more informative if you say what
InstalledAppController has switched to being instead.)

[haraken, 2016-12-13 08:45:13]

Description was changed from

==========
Remove DOMWindowProperty from InstalledAppController

We're deprecating DOMWindowProperty.
We should use InstalledAppController instead.

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

to

==========
InstalledAppController should use ContextLifecycleObserver instead of
DOMWindowProperty

DOMWindowProperty::frameDestroyed is now equivalent to
ContextLifecycleObserver::contextDestroyed.
This CL does the replacement for InstalledAppController.

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

[haraken, 2016-12-13 08:45:23]

> update the description, changing the last occurrence of InstalledAppController
> to be ContextLifecycleObserver instead.
> 
> (The title would perhaps be more informative if you say what
> InstalledAppController has switched to being instead.)

done.

[haraken, 2016-12-13 08:45:27]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-12-13 08:45:43]

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-13 11:20:18]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481618727076210,
"parent_rev": "995647973bc46a2bd784e3100428103d9534b82b", "commit_rev":
"629bd4a438511507fdcfa7e1deb84867c6f43332"}

[commit-bot: I haz the power, 2016-12-13 11:20:44]

Description was changed from

==========
InstalledAppController should use ContextLifecycleObserver instead of
DOMWindowProperty

DOMWindowProperty::frameDestroyed is now equivalent to
ContextLifecycleObserver::contextDestroyed.
This CL does the replacement for InstalledAppController.

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

to

==========
InstalledAppController should use ContextLifecycleObserver instead of
DOMWindowProperty

DOMWindowProperty::frameDestroyed is now equivalent to
ContextLifecycleObserver::contextDestroyed.
This CL does the replacement for InstalledAppController.

BUG=610176

Review-Url: https://codereview.chromium.org/2571513003
==========

[commit-bot: I haz the power, 2016-12-13 11:20:45]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-13 11:22:26]

Description was changed from

==========
InstalledAppController should use ContextLifecycleObserver instead of
DOMWindowProperty

DOMWindowProperty::frameDestroyed is now equivalent to
ContextLifecycleObserver::contextDestroyed.
This CL does the replacement for InstalledAppController.

BUG=610176

Review-Url: https://codereview.chromium.org/2571513003
==========

to

==========
InstalledAppController should use ContextLifecycleObserver instead of
DOMWindowProperty

DOMWindowProperty::frameDestroyed is now equivalent to
ContextLifecycleObserver::contextDestroyed.
This CL does the replacement for InstalledAppController.

BUG=610176

Committed: https://crrev.com/4256ecc52fd229daa6e9c7aff0ccbe275cb71a49
Cr-Commit-Position: refs/heads/master@{#438123}
==========

[commit-bot: I haz the power, 2016-12-13 11:22:27]

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