On 2013/07/10 17:13:06, loislo wrote:
On 2013/07/10 16:27:48, Yury Semikhatsky wrote:
> I'm wondering if there was some problem that prompted adding this test?
This check is a downstream version of our expectation about heap-snapshot
structure.
5 days ago danno landed a patch
https://code.google.com/p/v8/source/detail?r=15512
And in the combination with today's patch
https://chromiumcodereview.appspot.com/18751003/ we got ~20 arrays in the
heap
snapshot with no retainers.
As a result our blink tests for heap profiler became red.
Actually they were timing out and it was very hard to understand the
reason of
that.
Thanks for the clarification! Next time please include such information
into the
CL description or into a bug.
https://codereview.chromium.org/18996004/
--
--
v8-dev mailing list
[email protected]
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 it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.