v8binding: Fixes static readonly wrapper-returning attributes.

v8binding: Fixes static readonly wrapper-returning attributes.

static attributes must not be "keep_alive_gc".  keep_alive_gc
is a mechanism to associate a returned object with the receiver
object, however static attributes must not use the receiver
object.

BUG=

Review-Url: https://codereview.chromium.org/2913403003
Cr-Commit-Position: refs/heads/master@{#476231}
Committed: https://chromium.googlesource.com/chromium/src/+/808a9e88c43fb74ac5554d5dc7e621158937bcce

[Yuki, 2017-06-01 07:35:57]

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

[Yuki, 2017-06-01 07:35:58]

There was no use of static readonly wrapper-returning attributes, however,
https://crrev.com/2896773002 is going to use them.  Then, it turned out that
keep_alive_gc has been broken in that case.

This CL simply fixes it disabling keep_alive_gc for static attributes.

Could you review this CL?

[peria, 2017-06-01 07:40:07]

lgtm

[Yuki, 2017-06-01 07:44:11]

The CQ bit was checked by yukishiino@chromium.org

[commit-bot: I haz the power, 2017-06-01 07:44:20]

CQ is trying da patch.

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

[haraken, 2017-06-01 07:53:56]

LGTM

[commit-bot: I haz the power, 2017-06-01 09:17:17]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496303051573030, "parent_rev":
"e90a0487386316a1e97f69159f20fbe5933cb3b9", "commit_rev":
"808a9e88c43fb74ac5554d5dc7e621158937bcce"}

[commit-bot: I haz the power, 2017-06-01 09:17:25]

Description was changed from

==========
v8binding: Fixes static readonly wrapper-returning attributes.

static attributes must not be "keep_alive_gc".  keep_alive_gc
is a mechanism to associate a returned object with the receiver
object, however static attributes must not use the receiver
object.

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

to

==========
v8binding: Fixes static readonly wrapper-returning attributes.

static attributes must not be "keep_alive_gc".  keep_alive_gc
is a mechanism to associate a returned object with the receiver
object, however static attributes must not use the receiver
object.

BUG=

Review-Url: https://codereview.chromium.org/2913403003
Cr-Commit-Position: refs/heads/master@{#476231}
Committed:
https://chromium.googlesource.com/chromium/src/+/808a9e88c43fb74ac5554d5dc7e6...
==========

[commit-bot: I haz the power, 2017-06-01 09:17:26]

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