Use std::deque for storing edges and children in heap snapshot.

Use std::deque for storing edges and children in heap snapshot.

This patch fixes OOM crash that happens for large heap where
the total size of edges exceeds 2GB, which is the hard limit
for v8::internal::List allocated using tcmalloc.

BUG=chromium:675911
Review-Url: https://codereview.chromium.org/2595003002
Cr-Commit-Position: refs/heads/master@{#42004}
Committed: https://chromium.googlesource.com/v8/v8/+/b00fc8be8a2ff527d8462bb9c0fff163bfce850f

[ulan, 2016-12-21 14:49:40]

Description was changed from

==========
Use std::deque for storing edges and children in heap snapshot.

BUG=chromium:675911
==========

to

==========
Use std::deque for storing edges and children in heap snapshot.

This patch fixes OOM crash that happens for large heap where
the total size of edges exceeds 2GB, which is the hard limit
for v8::internal::List allocated using tcmalloc.

BUG=chromium:675911
==========

[ulan, 2016-12-21 14:51:29]

ulan@chromium.org changed reviewers:
+ kozyatinskiy@chromium.org

[ulan, 2016-12-21 14:51:29]

PTAL. This is the CL that we worked on together today.

[kozy, 2016-12-21 15:05:46]

thanks! lgtm.

[ulan, 2016-12-21 15:07:40]

The CQ bit was checked by ulan@chromium.org

[commit-bot: I haz the power, 2016-12-21 15:07:49]

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, 2016-12-21 15:15:19]

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

[commit-bot: I haz the power, 2016-12-21 15:15:20]

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

[ulan, 2016-12-30 16:02:47]

The CQ bit was checked by ulan@chromium.org

[ulan, 2016-12-30 16:02:47]

The patchset sent to the CQ was uploaded after l-g-t-m from
kozyatinskiy@chromium.org
Link to the patchset: https://codereview.chromium.org/2595003002/#ps40001
(title: "fix ce")

[commit-bot: I haz the power, 2016-12-30 16:02:58]

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, 2016-12-30 16:27:16]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1483113767671570,
"parent_rev": "a93aab375a0ecf938128fe097b8b6abd429bb364", "commit_rev":
"b00fc8be8a2ff527d8462bb9c0fff163bfce850f"}

[commit-bot: I haz the power, 2016-12-30 16:27:19]

Description was changed from

==========
Use std::deque for storing edges and children in heap snapshot.

This patch fixes OOM crash that happens for large heap where
the total size of edges exceeds 2GB, which is the hard limit
for v8::internal::List allocated using tcmalloc.

BUG=chromium:675911
==========

to

==========
Use std::deque for storing edges and children in heap snapshot.

This patch fixes OOM crash that happens for large heap where
the total size of edges exceeds 2GB, which is the hard limit
for v8::internal::List allocated using tcmalloc.

BUG=chromium:675911

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

[commit-bot: I haz the power, 2016-12-30 16:27:20]

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