[interpreter, debugger] add some missing statement positions.

[interpreter, debugger] add some missing statement positions.

R=rmcilroy@chromium.org, vogelheim@chromium.org
BUG=v8:4690
LOG=N

Committed: https://crrev.com/4a7722c9930a42ba0e8feeece286d74834211a7e
Cr-Commit-Position: refs/heads/master@{#34569}

Committed: https://crrev.com/5586ff666e69697416ffefcdf64081598b0b9c71
Cr-Commit-Position: refs/heads/master@{#34573}

[Yang, 2016-03-07 09:57:54]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2016-03-07 09:57:56]

The CQ bit was unchecked by yangguo@chromium.org

[Yang, 2016-03-07 09:57:57]

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

[commit-bot: I haz the power, 2016-03-07 09:58:07]

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

[commit-bot: I haz the power, 2016-03-07 10:24:33]

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

[commit-bot: I haz the power, 2016-03-07 10:24:34]

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

[Yang, 2016-03-07 10:39:43]

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

[commit-bot: I haz the power, 2016-03-07 10:39:47]

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

[commit-bot: I haz the power, 2016-03-07 10:58:42]

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

[commit-bot: I haz the power, 2016-03-07 10:58:43]

Dry run: This issue passed the CQ dry run.

[vogelheim, 2016-03-07 11:38:44]

lgtm

[Yang, 2016-03-07 11:39:32]

The CQ bit was checked by yangguo@chromium.org

[commit-bot: I haz the power, 2016-03-07 11:39:41]

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

[commit-bot: I haz the power, 2016-03-07 11:46:12]

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

[commit-bot: I haz the power, 2016-03-07 11:46:12]

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

[Yang, 2016-03-08 05:59:07]

yangguo@chromium.org changed reviewers:
+ mstarzinger@chromium.org

[Yang, 2016-03-08 05:59:08]

Michael, could you take a brief look at this? Thanks.

[rmcilroy, 2016-03-08 07:24:13]

Lgtm

[Yang, 2016-03-08 07:49:46]

The CQ bit was checked by yangguo@chromium.org

[commit-bot: I haz the power, 2016-03-08 07:49:48]

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

[commit-bot: I haz the power, 2016-03-08 07:51:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-08 07:53:12]

Description was changed from

==========
[interpreter, debugger] add some missing statement positions.


R=rmcilroy@chromium.org, vogelheim@chromium.org
BUG=v8:4690
LOG=N
==========

to

==========
[interpreter, debugger] add some missing statement positions.

R=rmcilroy@chromium.org, vogelheim@chromium.org
BUG=v8:4690
LOG=N

Committed: https://crrev.com/4a7722c9930a42ba0e8feeece286d74834211a7e
Cr-Commit-Position: refs/heads/master@{#34569}
==========

[commit-bot: I haz the power, 2016-03-08 07:53:13]

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

[Yang, 2016-03-08 08:28:51]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1772293002/ by yangguo@chromium.org.

The reason for reverting is: failing tests with ignition..

[Yang, 2016-03-08 08:29:34]

Description was changed from

==========
[interpreter, debugger] add some missing statement positions.

R=rmcilroy@chromium.org, vogelheim@chromium.org
BUG=v8:4690
LOG=N

Committed: https://crrev.com/4a7722c9930a42ba0e8feeece286d74834211a7e
Cr-Commit-Position: refs/heads/master@{#34569}
==========

to

==========
[interpreter, debugger] add some missing statement positions.

R=rmcilroy@chromium.org, vogelheim@chromium.org
BUG=v8:4690
LOG=N

Committed: https://crrev.com/4a7722c9930a42ba0e8feeece286d74834211a7e
Cr-Commit-Position: refs/heads/master@{#34569}
==========

[Yang, 2016-03-08 08:45:00]

On 2016/03/08 08:28:51, Yang wrote:
> A revert of this CL (patchset #1 id:1) has been created in
> https://codereview.chromium.org/1772293002/ by mailto:yangguo@chromium.org.
> 
> The reason for reverting is: failing tests with ignition..

Turns out the expectation changes clashed with another debugger CL. Ignition now
no longer fixes some of the tests.

[Yang, 2016-03-08 08:45:27]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2016-03-08 08:45:27]

The patchset sent to the CQ was uploaded after l-g-t-m from
rmcilroy@chromium.org, vogelheim@chromium.org
Link to the patchset: https://codereview.chromium.org/1770773002/#ps20001
(title: "update expectations")

[commit-bot: I haz the power, 2016-03-08 08:45:36]

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

[commit-bot: I haz the power, 2016-03-08 09:07:51]

Description was changed from

==========
[interpreter, debugger] add some missing statement positions.

R=rmcilroy@chromium.org, vogelheim@chromium.org
BUG=v8:4690
LOG=N

Committed: https://crrev.com/4a7722c9930a42ba0e8feeece286d74834211a7e
Cr-Commit-Position: refs/heads/master@{#34569}
==========

to

==========
[interpreter, debugger] add some missing statement positions.

R=rmcilroy@chromium.org, vogelheim@chromium.org
BUG=v8:4690
LOG=N

Committed: https://crrev.com/4a7722c9930a42ba0e8feeece286d74834211a7e
Cr-Commit-Position: refs/heads/master@{#34569}
==========

[commit-bot: I haz the power, 2016-03-08 09:07:52]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-08 09:09:22]

Description was changed from

==========
[interpreter, debugger] add some missing statement positions.

R=rmcilroy@chromium.org, vogelheim@chromium.org
BUG=v8:4690
LOG=N

Committed: https://crrev.com/4a7722c9930a42ba0e8feeece286d74834211a7e
Cr-Commit-Position: refs/heads/master@{#34569}
==========

to

==========
[interpreter, debugger] add some missing statement positions.

R=rmcilroy@chromium.org, vogelheim@chromium.org
BUG=v8:4690
LOG=N

Committed: https://crrev.com/4a7722c9930a42ba0e8feeece286d74834211a7e
Cr-Commit-Position: refs/heads/master@{#34569}

Committed: https://crrev.com/5586ff666e69697416ffefcdf64081598b0b9c71
Cr-Commit-Position: refs/heads/master@{#34573}
==========

[commit-bot: I haz the power, 2016-03-08 09:09:24]

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