Re-re-land FastAccessorBuilder.

Re-re-land FastAccessorBuilder.

... using the RawMachineAssembler and the work in crrev.com/1407313004.

The original change collided with crrev.com/1513543003.

BUG=chromium:508898
LOG=Y

Committed: https://crrev.com/515d9ccd8e6df7bf2ca01e2a55aaad30226399e1
Cr-Commit-Position: refs/heads/master@{#32742}

patch from issue 1474543004 at patchset 260001 (http://crrev.com/1474543004#ps260001)

Committed: https://crrev.com/ee5c38d7db907ff86dd4049721c0cb4bc90a6c4d
Cr-Commit-Position: refs/heads/master@{#32753}

patch from issue 1504713012 at patchset 20001 (http://crrev.com/1504713012#ps20001)

Committed: https://crrev.com/5819e4be5bd128556b0d735abe6b58173da67079
Cr-Commit-Position: refs/heads/master@{#32786}

[vogelheim, 2015-12-10 15:07:53]

vogelheim@chromium.org changed reviewers:
+ epertoso@chromium.org, mstarzinger@chromium.org

[vogelheim, 2015-12-10 15:07:54]

Oh hai again, please have a look. :-/

When the test was written, the idea was that an accessor would be wither JS/fast
OR C++/native, but not both. In the other test, we work around that by forcing
it to be in a single call site and running it a lot. This test doesn't
(explicitly) do that. So I just made sure the C++ and fast versions return the
same value. Please let me know whether that makes any sense...

[Michael Starzinger, 2015-12-10 15:10:58]

LGTM.

[epertoso, 2015-12-10 15:22:08]

lgtm

[epertoso, 2015-12-10 15:22:34]

On 2015/12/10 at 15:22:08, epertoso wrote:
> lgtm

But, of course, test-api-accessors.cc should be updated.

[vogelheim, 2015-12-11 09:40:04]

The CQ bit was checked by vogelheim@chromium.org

[vogelheim, 2015-12-11 09:40:05]

The patchset sent to the CQ was uploaded after l-g-t-m from
epertoso@chromium.org, mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1518703002/#ps40001
(title: "Add test restrictions, as for test-api-fast-accessor-builder")

[commit-bot: I haz the power, 2015-12-11 09:40:10]

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

[commit-bot: I haz the power, 2015-12-11 09:53:55]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-12-11 09:54:04]

Description was changed from

==========
Re-re-land FastAccessorBuilder.

... using the RawMachineAssembler and the work in crrev.com/1407313004.

The original change collided with crrev.com/1513543003.

BUG=chromium:508898
LOG=Y

Committed: https://crrev.com/515d9ccd8e6df7bf2ca01e2a55aaad30226399e1
Cr-Commit-Position: refs/heads/master@{#32742}

patch from issue 1474543004 at patchset 260001
(http://crrev.com/1474543004#ps260001)

Committed: https://crrev.com/ee5c38d7db907ff86dd4049721c0cb4bc90a6c4d
Cr-Commit-Position: refs/heads/master@{#32753}

patch from issue 1504713012 at patchset 20001
(http://crrev.com/1504713012#ps20001)
==========

to

==========
Re-re-land FastAccessorBuilder.

... using the RawMachineAssembler and the work in crrev.com/1407313004.

The original change collided with crrev.com/1513543003.

BUG=chromium:508898
LOG=Y

Committed: https://crrev.com/515d9ccd8e6df7bf2ca01e2a55aaad30226399e1
Cr-Commit-Position: refs/heads/master@{#32742}

patch from issue 1474543004 at patchset 260001
(http://crrev.com/1474543004#ps260001)

Committed: https://crrev.com/ee5c38d7db907ff86dd4049721c0cb4bc90a6c4d
Cr-Commit-Position: refs/heads/master@{#32753}

patch from issue 1504713012 at patchset 20001
(http://crrev.com/1504713012#ps20001)

Committed: https://crrev.com/5819e4be5bd128556b0d735abe6b58173da67079
Cr-Commit-Position: refs/heads/master@{#32786}
==========

[commit-bot: I haz the power, 2015-12-11 09:54:05]

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