Ensure the @@species protector is updated for accessors

Ensure the @@species protector is updated for accessors

The initial species protector hooked into property declaration in an
incomplete place, and missed definitions of accessors. This patch repairs
them by calling out to update the protector from an additional location.

R=adamk
CC=verwaest,cbruni
BUG=v8:4093
LOG=Y

Committed: https://crrev.com/3f8af30ee7377e9013a3be93e6ca9e9ec51f926b
Cr-Commit-Position: refs/heads/master@{#34599}

[Dan Ehrenberg, 2016-02-29 23:03:11]

The CQ bit was checked by littledan@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-29 23:03:15]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1746323002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1746323002/1

[commit-bot: I haz the power, 2016-02-29 23:31:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-29 23:31:43]

Dry run: This issue passed the CQ dry run.

[adamk, 2016-03-01 02:24:27]

adamk@chromium.org changed reviewers:
+ verwaest@chromium.org

[adamk, 2016-03-01 02:24:28]

I'll defer to Toon here, I don't know enough about the current state of lookups
to say this is exhaustively correct.

[Dan Ehrenberg, 2016-03-04 00:35:27]

On 2016/03/01 at 02:24:28, adamk wrote:
> I'll defer to Toon here, I don't know enough about the current state of
lookups to say this is exhaustively correct.

Toon, any opinion on this patch? It fixes a bug that you reported to me.

[Toon Verwaest, 2016-03-07 19:38:23]

As discussed, please move the call into DefineAccessor. Otherwise lgtm

[Dan Ehrenberg, 2016-03-08 19:27:12]

The CQ bit was checked by littledan@chromium.org

[Dan Ehrenberg, 2016-03-08 19:27:12]

Done

[Dan Ehrenberg, 2016-03-08 19:27:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
verwaest@chromium.org
Link to the patchset: https://codereview.chromium.org/1746323002/#ps20001
(title: "Apply Toon's suggestion")

[commit-bot: I haz the power, 2016-03-08 19:27:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1746323002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1746323002/20001

[commit-bot: I haz the power, 2016-03-08 19:54:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-08 19:55:33]

Description was changed from

==========
Ensure the @@species protector is updated for accessors

The initial species protector hooked into property declaration in an
incomplete place, and missed definitions of accessors. This patch repairs
them by calling out to update the protector from an additional location.

R=adamk
CC=verwaest,cbruni
BUG=v8:4093
LOG=Y
==========

to

==========
Ensure the @@species protector is updated for accessors

The initial species protector hooked into property declaration in an
incomplete place, and missed definitions of accessors. This patch repairs
them by calling out to update the protector from an additional location.

R=adamk
CC=verwaest,cbruni
BUG=v8:4093
LOG=Y

Committed: https://crrev.com/3f8af30ee7377e9013a3be93e6ca9e9ec51f926b
Cr-Commit-Position: refs/heads/master@{#34599}
==========

[commit-bot: I haz the power, 2016-03-08 19:55:34]

Patchset 2 (id:??) landed as
https://crrev.com/3f8af30ee7377e9013a3be93e6ca9e9ec51f926b
Cr-Commit-Position: refs/heads/master@{#34599}