530624 - TM: fix JS_DUMP_PROPTREE_STATS

Status: RESOLVED WONTFIX

(Core :: JavaScript Engine, defect)

Description

[Gregor Wagner [:gwagner]]

I always hit an assertion during the shutdown process of the browser when  JS_DUMP_PROPTREE_STATS is enabled.

It in
js_SweepScopeProperties(JSContext *cx)

#ifdef JS_DUMP_PROPTREE_STATS
    JSBasicStats bs;
    uint32 livePropCapacity = 0, totalLiveCount = 0;
    static FILE *logfp;
    if (!logfp)
        logfp = fopen("/tmp/proptree.stats", "w");

    JS_BASIC_STATS_INIT(&bs);
    MeterKidCount(&bs, rt->propertyTreeHash.entryCount);
    JS_DHashTableEnumerate(&rt->propertyTreeHash, js_MeterPropertyTree, &bs);

    {
        double props, nodes, mean, sigma;

        props = rt->liveScopePropsPreSweep;
        nodes = rt->livePropTreeNodes;
        JS_ASSERT(nodes == bs.sum);

The last assertion is not always true. I printed the numbers and they differ by one.