Remove existingWindowProxy()

Remove existingWindowProxy()

The only call site of existingWindowProxy() is V8Initializer::messageHandlerInMainThread().
However, as commented in the code, that code should not be needed because
if the context is during initialization it should already have bailed out at
the if(!scriptState->contextIsValid()) check at the very beginning of
V8Initializer::messageHandlerInMainThread().

If the reasoning is correct, we can simply remove all code around existingWindowProxy().

BUG=677253
Committed: https://crrev.com/45830933ad06e7f8a213e97269522469b1602863
Cr-Commit-Position: refs/heads/master@{#440840}

[haraken, 2016-12-28 08:28:08]

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

[haraken, 2016-12-28 08:28:09]

PTAL

[Yuki, 2016-12-28 08:42:29]

LGTM.

[haraken, 2016-12-28 08:46:10]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-12-28 08:46:19]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[haraken, 2016-12-28 08:52:08]

Description was changed from

==========
Remove existingWindowProxy()

The only call site of existingWindowProxy() is
V8Initializer::messageHandlerInMainThread().
However, as commented in the code, that code should not be needed because
if the context is during initialization it should already have bailed out at
the if(!scriptState->contextIsValid()) check at the very beginning of
V8Initializer::messageHandlerInMainThread().

If the reasoning is correct, we can simply remove all code around
existingWindowProxy().

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

to

==========
Remove existingWindowProxy()

The only call site of existingWindowProxy() is
V8Initializer::messageHandlerInMainThread().
However, as commented in the code, that code should not be needed because
if the context is during initialization it should already have bailed out at
the if(!scriptState->contextIsValid()) check at the very beginning of
V8Initializer::messageHandlerInMainThread().

If the reasoning is correct, we can simply remove all code around
existingWindowProxy().

BUG=677253
==========

[commit-bot: I haz the power, 2016-12-28 10:08:17]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482914770951480, "parent_rev":
"3d281cdb2ed2d4e6825662e2e090f4542d85c940", "commit_rev":
"30666d80dc0ddc5cf5d559bdfa9c9d16237a8aad"}

[commit-bot: I haz the power, 2016-12-28 10:08:43]

Description was changed from

==========
Remove existingWindowProxy()

The only call site of existingWindowProxy() is
V8Initializer::messageHandlerInMainThread().
However, as commented in the code, that code should not be needed because
if the context is during initialization it should already have bailed out at
the if(!scriptState->contextIsValid()) check at the very beginning of
V8Initializer::messageHandlerInMainThread().

If the reasoning is correct, we can simply remove all code around
existingWindowProxy().

BUG=677253
==========

to

==========
Remove existingWindowProxy()

The only call site of existingWindowProxy() is
V8Initializer::messageHandlerInMainThread().
However, as commented in the code, that code should not be needed because
if the context is during initialization it should already have bailed out at
the if(!scriptState->contextIsValid()) check at the very beginning of
V8Initializer::messageHandlerInMainThread().

If the reasoning is correct, we can simply remove all code around
existingWindowProxy().

BUG=677253

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

[commit-bot: I haz the power, 2016-12-28 10:08:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2017-01-02 15:48:46]

Description was changed from

==========
Remove existingWindowProxy()

The only call site of existingWindowProxy() is
V8Initializer::messageHandlerInMainThread().
However, as commented in the code, that code should not be needed because
if the context is during initialization it should already have bailed out at
the if(!scriptState->contextIsValid()) check at the very beginning of
V8Initializer::messageHandlerInMainThread().

If the reasoning is correct, we can simply remove all code around
existingWindowProxy().

BUG=677253

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

to

==========
Remove existingWindowProxy()

The only call site of existingWindowProxy() is
V8Initializer::messageHandlerInMainThread().
However, as commented in the code, that code should not be needed because
if the context is during initialization it should already have bailed out at
the if(!scriptState->contextIsValid()) check at the very beginning of
V8Initializer::messageHandlerInMainThread().

If the reasoning is correct, we can simply remove all code around
existingWindowProxy().

BUG=677253

Committed: https://crrev.com/45830933ad06e7f8a213e97269522469b1602863
Cr-Commit-Position: refs/heads/master@{#440840}
==========

[commit-bot: I haz the power, 2017-01-02 15:48:47]

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