[v8-dev] Re: Make sure that idle scavenges are just performed when enough objects are allocated in new space. (issue 1138643003 by hpa...@chromium.org)

2015-05-15 Thread commit-bot
Committed patchset #2 (id:20001) https://codereview.chromium.org/1138643003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups "v8-dev" group. To unsubscribe from this group and

[v8-dev] Re: Make sure that idle scavenges are just performed when enough objects are allocated in new space. (issue 1138643003 by hpa...@chromium.org)

2015-05-15 Thread commit-bot
Patchset 2 (id:??) landed as https://crrev.com/6ead193b51105d1ed5796a7ba417bb62d0323176 Cr-Commit-Position: refs/heads/master@{#28428} https://codereview.chromium.org/1138643003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this messag

[v8-dev] Re: Make sure that idle scavenges are just performed when enough objects are allocated in new space. (issue 1138643003 by hpa...@chromium.org)

2015-05-15 Thread commit-bot
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1138643003/20001 https://codereview.chromium.org/1138643003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscr

[v8-dev] Re: Make sure that idle scavenges are just performed when enough objects are allocated in new space. (issue 1138643003 by hpa...@chromium.org)

2015-05-15 Thread erikcorry
lgtm https://codereview.chromium.org/1138643003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups "v8-dev" group. To unsubscribe from this group and stop receiving emails from

[v8-dev] Re: Make sure that idle scavenges are just performed when enough objects are allocated in new space. (issue 1138643003 by hpa...@chromium.org)

2015-05-15 Thread hpayer
https://codereview.chromium.org/1138643003/diff/1/src/heap/gc-idle-time-handler.cc File src/heap/gc-idle-time-handler.cc (right): https://codereview.chromium.org/1138643003/diff/1/src/heap/gc-idle-time-handler.cc#newcode141 src/heap/gc-idle-time-handler.cc:141: // The allocated new space limit t

[v8-dev] Re: Make sure that idle scavenges are just performed when enough objects are allocated in new space. (issue 1138643003 by hpa...@chromium.org)

2015-05-15 Thread commit-bot
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1138643003/20001 https://codereview.chromium.org/1138643003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscr

[v8-dev] Re: Make sure that idle scavenges are just performed when enough objects are allocated in new space. (issue 1138643003 by hpa...@chromium.org)

2015-05-15 Thread commit-bot
No LGTM from a valid reviewer yet. Only full committers are accepted. Even if an LGTM may have been provided, it was from a non-committer, _not_ a full super star committer. See http://www.chromium.org/getting-involved/become-a-committer Note that this has nothing to do with OWNERS files. https:/

[v8-dev] Re: Make sure that idle scavenges are just performed when enough objects are allocated in new space. (issue 1138643003 by hpa...@chromium.org)

2015-05-15 Thread erik . corry
https://codereview.chromium.org/1138643003/diff/1/src/heap/gc-idle-time-handler.cc File src/heap/gc-idle-time-handler.cc (right): https://codereview.chromium.org/1138643003/diff/1/src/heap/gc-idle-time-handler.cc#newcode141 src/heap/gc-idle-time-handler.cc:141: // The allocated new space limit t

[v8-dev] Re: Make sure that idle scavenges are just performed when enough objects are allocated in new space. (issue 1138643003 by hpa...@chromium.org)

2015-05-15 Thread erik . corry
lgtm https://codereview.chromium.org/1138643003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups "v8-dev" group. To unsubscribe from this group and stop receiving emails from