[Webkit-unassigned] [Bug 33383] REGRESSION (r52082): Missing event handlers on JQuery demo page (33383)

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=33383





--- Comment #22 from Geoffrey Garen <ggaren at apple.com>  2010-02-04 10:54:29 PST ---
> >+        There were two bugs here:
> 
> And only one test?!

Since there's no way to make bug #2 happen in the context of this jQuery demo,
I thought I'd wait and write other tests along with the other bugs you've filed
(like the XPath bug). 

I don't mind adding a test to this patch, though.

> +function gc() {
> +    for (var i = 0; i < 5000; ++i)
> +        new Object;
> +}
> 
> A comment in other gc() functions states that Firefox collects after about 9000
> allocations, FWIW. And we definitely want a special case for DumpRenderTree
> (GCController.collect()).

I guess this function could use a better name, but I'm not sure what to call
it. We don't actually want to force a full GC (which is what
GCController.collect() would do.) That would hide the bug. Rather, we want to
move the collector through a full allocation phase. I guess I'll rename the
function to "allocate".

I'll submit a new patch with the rename and an additional test.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.