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.


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.


Reply via email to