Remove DOMWindowLifecycleObserver::window()

Remove DOMWindowLifecycleObserver::window()

This is a preparation for landing https://codereview.chromium.org/822863003/. I'm planning to make LocalDOMWindow's destruction unobservable because the timing becomes undeterministic in a GC system. This CL removes DOMWindowLifecycleObserver::window(), which is a method whose behavior depends on a timing of LocalDOMWindow's destruction.

BUG=340522
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=187750

[haraken, 2014-12-29 15:28:09]

haraken@chromium.org changed reviewers:
+ oilpan-reviews@chromium.org, sigbjornf@opera.com

[haraken, 2014-12-29 15:28:15]

PTAL

[sof, 2014-12-29 15:32:38]

https://codereview.chromium.org/826223002/diff/1/Source/modules/gamepad/Navig...
File Source/modules/gamepad/NavigatorGamepad.cpp (right):

https://codereview.chromium.org/826223002/diff/1/Source/modules/gamepad/Navig...
Source/modules/gamepad/NavigatorGamepad.cpp:100: if (frame()->domWindow()) {
Add "frame() &&" to prevent detached accesses from breaking?

[haraken, 2014-12-29 16:42:07]

On 2014/12/29 15:32:38, sof wrote:
>
https://codereview.chromium.org/826223002/diff/1/Source/modules/gamepad/Navig...
> File Source/modules/gamepad/NavigatorGamepad.cpp (right):
> 
>
https://codereview.chromium.org/826223002/diff/1/Source/modules/gamepad/Navig...
> Source/modules/gamepad/NavigatorGamepad.cpp:100: if (frame()->domWindow()) {
> Add "frame() &&" to prevent detached accesses from breaking?

Done.

[sof, 2014-12-29 16:49:20]

lgtm

[haraken, 2014-12-29 17:08:00]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2014-12-29 17:08:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/826223002/20001

[commit-bot: I haz the power, 2014-12-29 18:31:03]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=187750

[haraken, 2015-03-03 12:00:32]

Patchset #3 (id:40001) has been deleted