[Webkit-unassigned] [Bug 107954] EventSender isn't suspended when page load is deferred

[bugzilla-daemon at webkit.org]

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





--- Comment #13 from Yong Li <yoli at rim.com>  2013-01-29 06:57:51 PST ---
(In reply to comment #10)
> > Yes, it does sound a little bit confusing. However event it is just about loading, JS execution can definitely trigger some loading jobs.
> 
> Honestly, this is more than a little confusing to me. Personally, I would not dare to make patches in this area without deeply investigating both existing and desired behavior.

We already have a lot of code (like ActiveDOMObjects, SuspendableTimer, DocumentEventQueue and other classes/functions) that suspends and resumes scheduled timers accordingly.

I think the existing behavior for these load events is:

1) schedule event dispatching with timer. Dispatch events when the timer fires
2) Force dispatching the events in Document.implicitClose() and XMLDocumentParser::append()

I wouldn't change the behavior. My proposed patch misses 2), which I think can be done by keeping the event sender lists in the document object (rather than using global static lists).

Thanks Dan. I'll also try to make a test case even a manual one to show the JS re-entrancy when time allowing.

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