Simplify window.closed testing.

Simplify window.closed testing.

Instead of keeping a separate Frame flag, have LocalDOMWindow::closed
consider the presence of a FrameHost on the frame. It'll be cleared
during detach and impending frame closure, and is observationally
equivalent to keeping an unnecessary extra flag.

R=dcheng,ager
BUG=414658
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=182201

[sof, 2014-09-17 12:48:05]

sigbjornf@opera.com changed reviewers:
+ ager@chromium.org, dcheng@chromium.org, tkent@chromium.org

[sof, 2014-09-17 12:48:06]

Please take a look.

As per dcheng's suggestion ( https://codereview.chromium.org/538323003/#msg22 )
to consider FrameHost instead.

[Mads Ager (chromium), 2014-09-17 12:50:13]

LGTM

Nice simplification; thanks for pointing that out dcheng. :)

[dcheng, 2014-09-17 17:12:58]

Thanks! LGTM

[sof, 2014-09-17 19:07:22]

On 2014/09/17 12:50:13, Mads Ager (chromium) wrote:
> LGTM
> 
> Nice simplification; thanks for pointing that out dcheng. :)

Indeed so; I'm trying to better grasp frame handling mechanics as I go along
here, so nudges in the right direction are most welcome.

[tkent, 2014-09-17 23:19:43]

The CQ bit was checked by tkent@chromium.org

[tkent, 2014-09-17 23:19:43]

lgtm

[commit-bot: I haz the power, 2014-09-17 23:19:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/544013005/1

[commit-bot: I haz the power, 2014-09-18 00:01:09]

Committed patchset #1 (id:1) as 182201

[haraken, 2014-09-20 08:14:39]

LGTM