remove security check from Window

remove security check from Window

it should be covered by the global object access check already

R=haraken@chromium.org

BUG=

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

[dcarney, 2015-02-26 08:31:48]

Patchset #1 (id:1) has been deleted

[dcarney, 2015-02-26 08:31:54]

Patchset #1 (id:20001) has been deleted

[dcarney, 2015-02-26 08:38:25]

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

[dcarney, 2015-02-26 08:38:39]

ptal

[haraken, 2015-02-26 08:40:07]

LGTM

[marja, 2015-02-26 09:53:31]

marja@chromium.org changed reviewers:
+ marja@chromium.org

[marja, 2015-02-26 09:53:31]

drive by comment: is it possible to write a meaningful test here? (that stuff
still gets blocked even if the access check is removed)

[dcarney, 2015-02-26 10:18:13]

On 2015/02/26 09:53:31, marja wrote:
> drive by comment: is it possible to write a meaningful test here? (that stuff
> still gets blocked even if the access check is removed)

it would be absurdly complicated

[dcarney, 2015-02-26 10:18:18]

The CQ bit was checked by dcarney@chromium.org

[commit-bot: I haz the power, 2015-02-26 10:18:48]

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

[marja, 2015-02-26 10:23:12]

(Offline discussion: this should already be covered by existing tests and I also
wasn't suggesting to write a test which changes from fail -> pass after this
change :)

[commit-bot: I haz the power, 2015-02-26 10:53:12]

Committed patchset #1 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=190905

[boliu, 2015-02-27 16:34:03]

This caused an assertion failure in an android webview test. More details in
crbug.com/462306. Anyone here can help?

02-26 20:27:13.793  2753  2791 W WebKit  : ASSERTION FAILED:
scriptState->contextIsValid()
02-26 20:27:13.793  2753  2791 W WebKit  :
../../third_party/WebKit/Source/bindings/core/v8/ScriptState.h(35) :
blink::ScriptState::Scope::Scope(blink::ScriptState *)