[Interpreter] Loads accumulator before calling StoreNamedProperty in ForInAssignment.

[Interpreter] Loads accumulator before calling StoreNamedProperty in ForInAssignment.

Fixed a bug in VisitForInAssignment. After visiting the object the value
to be stored was not loaded back to the accumulator. Also added two tests
to check this case.

BUG=v8:4280
LOG=N

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

[mythria, 2016-01-08 14:53:49]

mythria@chromium.org changed reviewers:
+ oth@chromium.org

[mythria, 2016-01-08 14:53:49]

PTAL,

Thanks,
Mythri

[oth, 2016-01-08 14:58:11]

LGTM. Thanks for fixing this and the test.

[mythria, 2016-01-08 15:15:44]

The CQ bit was checked by mythria@chromium.org

[commit-bot: I haz the power, 2016-01-08 15:15:52]

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

[commit-bot: I haz the power, 2016-01-08 15:42:48]

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

[commit-bot: I haz the power, 2016-01-08 15:42:48]

Try jobs failed on following builders:
  v8_presubmit on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/9566)

[mythria, 2016-01-08 15:50:41]

mythria@chromium.org changed reviewers:
+ rmcilroy@chromium.org

[mythria, 2016-01-08 15:50:42]

Ross, could you please take a look.

Thanks,
Mythri

[rmcilroy, 2016-01-08 16:04:13]

LGTM, thanks for fixing

[mythria, 2016-01-08 17:00:35]

The CQ bit was checked by mythria@chromium.org

[commit-bot: I haz the power, 2016-01-08 17:00:42]

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

[commit-bot: I haz the power, 2016-01-08 17:02:02]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-08 17:02:39]

Description was changed from

==========
[Interpreter] Loads accumulator before calling StoreNamedProperty in
ForInAssignment.

Fixed a bug in VisitForInAssignment. After visiting the object the value
to be stored was not loaded back to the accumulator. Also added two tests
to check this case.

BUG=v8:4280
LOG=N
==========

to

==========
[Interpreter] Loads accumulator before calling StoreNamedProperty in
ForInAssignment.

Fixed a bug in VisitForInAssignment. After visiting the object the value
to be stored was not loaded back to the accumulator. Also added two tests
to check this case.

BUG=v8:4280
LOG=N

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

[commit-bot: I haz the power, 2016-01-08 17:02:40]

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