Improve rendering of callsite with non-function target.

Improve rendering of callsite with non-function target.

This hackily disambiguates multiple calls for the iterator protocols in ForOf / Yield* by adding -2 / -1 to the pos.

BUG=v8:3953
LOG=y

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

[Toon Verwaest, 2015-12-02 17:13:36]

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

[Toon Verwaest, 2015-12-02 17:13:37]

ptal

[Toon Verwaest, 2015-12-02 17:14:31]

Description was changed from

==========
Improve rendering of callsite with non-function target.

BUG=v8:3953
LOG=y
==========

to

==========
Improve rendering of callsite with non-function target.

This hackily disambiguates multiple calls for the iterator protocols in ForOf /
Yield* by adding -2 / -1 to the pos.

BUG=v8:3953
LOG=y
==========

[Camillo Bruni, 2015-12-02 17:33:50]

lgtm

https://codereview.chromium.org/1491923003/diff/40001/src/runtime/runtime-int...
File src/runtime/runtime-internal.cc (right):

https://codereview.chromium.org/1491923003/diff/40001/src/runtime/runtime-int...
src/runtime/runtime-internal.cc:417: if (!it.done()) {
early return!

[Toon Verwaest, 2015-12-02 17:42:20]

The CQ bit was checked by verwaest@chromium.org

[commit-bot: I haz the power, 2015-12-02 17:42:31]

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

[commit-bot: I haz the power, 2015-12-02 17:53:52]

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

[commit-bot: I haz the power, 2015-12-02 17:53:58]

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

[Toon Verwaest, 2015-12-02 18:16:10]

The CQ bit was checked by verwaest@chromium.org

[Toon Verwaest, 2015-12-02 18:16:10]

The patchset sent to the CQ was uploaded after l-g-t-m from cbruni@chromium.org
Link to the patchset: https://codereview.chromium.org/1491923003/#ps60001
(title: "Fix tests")

[commit-bot: I haz the power, 2015-12-02 18:16:18]

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

[commit-bot: I haz the power, 2015-12-02 18:36:44]

Description was changed from

==========
Improve rendering of callsite with non-function target.

This hackily disambiguates multiple calls for the iterator protocols in ForOf /
Yield* by adding -2 / -1 to the pos.

BUG=v8:3953
LOG=y
==========

to

==========
Improve rendering of callsite with non-function target.

This hackily disambiguates multiple calls for the iterator protocols in ForOf /
Yield* by adding -2 / -1 to the pos.

BUG=v8:3953
LOG=y
==========

[commit-bot: I haz the power, 2015-12-02 18:36:44]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-12-02 18:37:33]

Description was changed from

==========
Improve rendering of callsite with non-function target.

This hackily disambiguates multiple calls for the iterator protocols in ForOf /
Yield* by adding -2 / -1 to the pos.

BUG=v8:3953
LOG=y
==========

to

==========
Improve rendering of callsite with non-function target.

This hackily disambiguates multiple calls for the iterator protocols in ForOf /
Yield* by adding -2 / -1 to the pos.

BUG=v8:3953
LOG=y

Committed: https://crrev.com/a6ed24d61c597c0e78d2d1c830d86ce33e091531
Cr-Commit-Position: refs/heads/master@{#32527}
==========

[commit-bot: I haz the power, 2015-12-02 18:37:34]

Patchset 4 (id:??) landed as
https://crrev.com/a6ed24d61c597c0e78d2d1c830d86ce33e091531
Cr-Commit-Position: refs/heads/master@{#32527}