Revert of binding: Changes the association among global-proxy/global/window-instance.

Revert of binding: Changes the association among global-proxy/global/window-instance. (patchset #15 id:270001 of https://codereview.chromium.org/2617733004/ )

Reason for revert:
This CL is suspicious to be causing crashes: https://crbug.com/691269


Original issue's description:
> binding: Changes the association among global-proxy/global/window-instance.
> 
> Since V8 now supports internal fields for the global proxy objects,
> let's change the mappings among global proxy object / global object /
> C++ instance in Blink.
> 
> With this CL, a global proxy object and C++ instance point to each
> other, and a global object points to the C++ instance (without
> reverse pointer).
> 
> Note that this CL makes one big difference.  For non-window objects,
> Blink objects always live longer than V8 wrappers.  This CL
> introduces an exception; DOMWindow may be collected before the V8
> wrapper object gets collected because the V8 wrapper is the global
> proxy object which remains the same while navigations.
> 
> BUG=675872
> 
> Review-Url: https://codereview.chromium.org/2617733004
> Cr-Commit-Position: refs/heads/master@{#449616}
> Committed: https://chromium.googlesource.com/chromium/src/+/76ec9e6ea20fb4a3c1f242e39abea0067afd235a

TBR=haraken@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=675872, 691269
Review-Url: https://codereview.chromium.org/2690943002
Cr-Commit-Position: refs/heads/master@{#449919}
Committed: https://chromium.googlesource.com/chromium/src/+/a8e0f8228466461fb0dbbce6eb4373625a39b787

[Yuki, 2017-02-13 06:53:09]

The CQ bit was checked by yukishiino@chromium.org

[Yuki, 2017-02-13 06:53:09]

Created Revert of binding: Changes the association among
global-proxy/global/window-instance.

[commit-bot: I haz the power, 2017-02-13 06:53:20]

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

[Yuki, 2017-02-13 06:53:29]

Description was changed from

==========
Revert of binding: Changes the association among
global-proxy/global/window-instance. (patchset #15 id:270001 of
https://codereview.chromium.org/2617733004/ )

Reason for revert:
This CL is suspicious to be causing crashes: https://crbug.com/691269


Original issue's description:
> binding: Changes the association among global-proxy/global/window-instance.
> 
> Since V8 now supports internal fields for the global proxy objects,
> let's change the mappings among global proxy object / global object /
> C++ instance in Blink.
> 
> With this CL, a global proxy object and C++ instance point to each
> other, and a global object points to the C++ instance (without
> reverse pointer).
> 
> Note that this CL makes one big difference.  For non-window objects,
> Blink objects always live longer than V8 wrappers.  This CL
> introduces an exception; DOMWindow may be collected before the V8
> wrapper object gets collected because the V8 wrapper is the global
> proxy object which remains the same while navigations.
> 
> BUG=675872
> 
> Review-Url: https://codereview.chromium.org/2617733004
> Cr-Commit-Position: refs/heads/master@{#449616}
> Committed:
https://chromium.googlesource.com/chromium/src/+/76ec9e6ea20fb4a3c1f242e39abe...

TBR=haraken@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=675872
==========

to

==========
Revert of binding: Changes the association among
global-proxy/global/window-instance. (patchset #15 id:270001 of
https://codereview.chromium.org/2617733004/ )

Reason for revert:
This CL is suspicious to be causing crashes: https://crbug.com/691269


Original issue's description:
> binding: Changes the association among global-proxy/global/window-instance.
> 
> Since V8 now supports internal fields for the global proxy objects,
> let's change the mappings among global proxy object / global object /
> C++ instance in Blink.
> 
> With this CL, a global proxy object and C++ instance point to each
> other, and a global object points to the C++ instance (without
> reverse pointer).
> 
> Note that this CL makes one big difference.  For non-window objects,
> Blink objects always live longer than V8 wrappers.  This CL
> introduces an exception; DOMWindow may be collected before the V8
> wrapper object gets collected because the V8 wrapper is the global
> proxy object which remains the same while navigations.
> 
> BUG=675872
> 
> Review-Url: https://codereview.chromium.org/2617733004
> Cr-Commit-Position: refs/heads/master@{#449616}
> Committed:
https://chromium.googlesource.com/chromium/src/+/76ec9e6ea20fb4a3c1f242e39abe...

TBR=haraken@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=675872, 691269
==========

[haraken, 2017-02-13 06:54:19]

LGTM

[commit-bot: I haz the power, 2017-02-13 08:33:33]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486968789381750, "parent_rev":
"062539120d0c3585f8a0a393246c5b98e0cd26f7", "commit_rev":
"a8e0f8228466461fb0dbbce6eb4373625a39b787"}

[commit-bot: I haz the power, 2017-02-13 08:34:14]

Description was changed from

==========
Revert of binding: Changes the association among
global-proxy/global/window-instance. (patchset #15 id:270001 of
https://codereview.chromium.org/2617733004/ )

Reason for revert:
This CL is suspicious to be causing crashes: https://crbug.com/691269


Original issue's description:
> binding: Changes the association among global-proxy/global/window-instance.
> 
> Since V8 now supports internal fields for the global proxy objects,
> let's change the mappings among global proxy object / global object /
> C++ instance in Blink.
> 
> With this CL, a global proxy object and C++ instance point to each
> other, and a global object points to the C++ instance (without
> reverse pointer).
> 
> Note that this CL makes one big difference.  For non-window objects,
> Blink objects always live longer than V8 wrappers.  This CL
> introduces an exception; DOMWindow may be collected before the V8
> wrapper object gets collected because the V8 wrapper is the global
> proxy object which remains the same while navigations.
> 
> BUG=675872
> 
> Review-Url: https://codereview.chromium.org/2617733004
> Cr-Commit-Position: refs/heads/master@{#449616}
> Committed:
https://chromium.googlesource.com/chromium/src/+/76ec9e6ea20fb4a3c1f242e39abe...

TBR=haraken@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=675872, 691269
==========

to

==========
Revert of binding: Changes the association among
global-proxy/global/window-instance. (patchset #15 id:270001 of
https://codereview.chromium.org/2617733004/ )

Reason for revert:
This CL is suspicious to be causing crashes: https://crbug.com/691269


Original issue's description:
> binding: Changes the association among global-proxy/global/window-instance.
> 
> Since V8 now supports internal fields for the global proxy objects,
> let's change the mappings among global proxy object / global object /
> C++ instance in Blink.
> 
> With this CL, a global proxy object and C++ instance point to each
> other, and a global object points to the C++ instance (without
> reverse pointer).
> 
> Note that this CL makes one big difference.  For non-window objects,
> Blink objects always live longer than V8 wrappers.  This CL
> introduces an exception; DOMWindow may be collected before the V8
> wrapper object gets collected because the V8 wrapper is the global
> proxy object which remains the same while navigations.
> 
> BUG=675872
> 
> Review-Url: https://codereview.chromium.org/2617733004
> Cr-Commit-Position: refs/heads/master@{#449616}
> Committed:
https://chromium.googlesource.com/chromium/src/+/76ec9e6ea20fb4a3c1f242e39abe...

TBR=haraken@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=675872, 691269

Review-Url: https://codereview.chromium.org/2690943002
Cr-Commit-Position: refs/heads/master@{#449919}
Committed:
https://chromium.googlesource.com/chromium/src/+/a8e0f8228466461fb0dbbce6eb43...
==========

[commit-bot: I haz the power, 2017-02-13 08:34:15]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/a8e0f8228466461fb0dbbce6eb43...