Fix receiver in Runtime::GetFrameDetails

[debug] Handle OOM events in debugger tests

Map OOM breaks generated by inspector to DebugEvent.OOM.
This avoids generating unintentional DebugEvent.Break events.

Also be more future-proof in event categorization.

On a related note, this CL also fixes a DCHECK in 
Runtime::GetFrameDetails.

The receiver needs to be grabbed from the inlined frame, not 
the outer optimized frame. Optimized frames only provide the
receiver on a best-effort basis.

BUG=v8:5950
Review-Url: https://codereview.chromium.org/2696173002
Cr-Commit-Position: refs/heads/master@{#43244}
Committed: https://chromium.googlesource.com/v8/v8/+/e9f5e1e99ed6d5c4930700be848f205bb1758369

[jgruber, 2017-02-15 14:03:36]

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

[commit-bot: I haz the power, 2017-02-15 14:03:43]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-15 14:21:53]

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

[commit-bot: I haz the power, 2017-02-15 14:21:54]

Dry run: Try jobs failed on following builders:
  v8_linux64_avx2_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng/buil...)
  v8_linux64_avx2_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng_trig...)
  v8_win_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_rel_ng/builds/22714)
  v8_win_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_rel_ng_triggered/bui...)

[jgruber, 2017-02-16 10:34:46]

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

[commit-bot: I haz the power, 2017-02-16 10:34:51]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jgruber, 2017-02-16 10:37:45]

Description was changed from

==========
Fix receiver in Runtime::GetFrameDetails

The receiver needs to be grabbed from the inlined frame, not the outer
optimized frame.

BUG=v8:5950
==========

to

==========
[debug] Handle OOM events in debugger tests

Map OOM breaks generated by inspector to DebugEvent.OOM.
This avoids generating unintentional DebugEvent.Break events.

Also be more future-proof in event categorization.

On a related note, this CL also fixes a DCHECK in 
Runtime::GetFrameDetails.

The receiver needs to be grabbed from the inlined frame, not 
the outer optimized frame, and it can be undefined for strict 
frames in construct calls.

BUG=v8:5950
==========

[commit-bot: I haz the power, 2017-02-16 10:56:20]

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

[commit-bot: I haz the power, 2017-02-16 10:56:21]

Dry run: Try jobs failed on following builders:
  v8_linux_verify_csa_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_verify_csa_rel_ng/...)
  v8_linux_verify_csa_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_verify_csa_rel_ng_...)
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/17556)
  v8_mac_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng_triggered/bui...)
  v8_win_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_rel_ng/builds/22780)
  v8_win_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_rel_ng_triggered/bui...)

[jgruber, 2017-02-16 11:04:43]

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

[commit-bot: I haz the power, 2017-02-16 11:04:51]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-16 11:31:37]

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

[commit-bot: I haz the power, 2017-02-16 11:31:39]

Dry run: This issue passed the CQ dry run.

[jgruber, 2017-02-16 11:35:26]

Description was changed from

==========
[debug] Handle OOM events in debugger tests

Map OOM breaks generated by inspector to DebugEvent.OOM.
This avoids generating unintentional DebugEvent.Break events.

Also be more future-proof in event categorization.

On a related note, this CL also fixes a DCHECK in 
Runtime::GetFrameDetails.

The receiver needs to be grabbed from the inlined frame, not 
the outer optimized frame, and it can be undefined for strict 
frames in construct calls.

BUG=v8:5950
==========

to

==========
[debug] Handle OOM events in debugger tests

Map OOM breaks generated by inspector to DebugEvent.OOM.
This avoids generating unintentional DebugEvent.Break events.

Also be more future-proof in event categorization.

On a related note, this CL also fixes a DCHECK in 
Runtime::GetFrameDetails.

The receiver needs to be grabbed from the inlined frame, not 
the outer optimized frame. Optimized frames only provide the
receiver on a best-effort basis.

BUG=v8:5950
==========

[jgruber, 2017-02-16 11:35:36]

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

[Yang, 2017-02-16 11:42:49]

On 2017/02/16 11:35:37, jgruber wrote:

Awesome! Nice work investigating the cause. LGTM.

[jgruber, 2017-02-16 11:46:24]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2017-02-16 11:46:46]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-16 11:48:22]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1487245584104750,
"parent_rev": "140ec9d7cc3bf5098ed26e1e14ed7cca85ba4fe7", "commit_rev":
"e9f5e1e99ed6d5c4930700be848f205bb1758369"}

[commit-bot: I haz the power, 2017-02-16 11:48:27]

Description was changed from

==========
[debug] Handle OOM events in debugger tests

Map OOM breaks generated by inspector to DebugEvent.OOM.
This avoids generating unintentional DebugEvent.Break events.

Also be more future-proof in event categorization.

On a related note, this CL also fixes a DCHECK in 
Runtime::GetFrameDetails.

The receiver needs to be grabbed from the inlined frame, not 
the outer optimized frame. Optimized frames only provide the
receiver on a best-effort basis.

BUG=v8:5950
==========

to

==========
[debug] Handle OOM events in debugger tests

Map OOM breaks generated by inspector to DebugEvent.OOM.
This avoids generating unintentional DebugEvent.Break events.

Also be more future-proof in event categorization.

On a related note, this CL also fixes a DCHECK in 
Runtime::GetFrameDetails.

The receiver needs to be grabbed from the inlined frame, not 
the outer optimized frame. Optimized frames only provide the
receiver on a best-effort basis.

BUG=v8:5950

Review-Url: https://codereview.chromium.org/2696173002
Cr-Commit-Position: refs/heads/master@{#43244}
Committed:
https://chromium.googlesource.com/v8/v8/+/e9f5e1e99ed6d5c4930700be848f205bb17...
==========

[commit-bot: I haz the power, 2017-02-16 11:48:29]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/v8/v8/+/e9f5e1e99ed6d5c4930700be848f205bb17...