Closed Bug 527870 Opened 11 years ago Closed 11 years ago

Bug in async code

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: sicking, Assigned: sicking)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Patch to fix (obsolete) — Splinter Review
I'm not sure if there is a way to actually make this matter, bug just in case the loop that executes async scripts has a bug when it's attempting to reset its counter. Patch attached
Attachment #411617 - Flags: review?(jst)
Comment on attachment 411617 [details] [diff] [review]
Patch to fix

r=jst
Attachment #411617 - Flags: review?(jst) → review+
Attached patch better fixSplinter Review
This is an even better fix. What we really want to do is to look for *any* script (async or not) that's ready to execute after we've executed a script.
Assignee: nobody → jonas
Attachment #411617 - Attachment is obsolete: true
Comment on attachment 411620 [details] [diff] [review]
better fix

Yup, that's better yet! r=jst for that as well.
Attachment #411620 - Flags: review+
This was fixed a long time ago

http://hg.mozilla.org/mozilla-central/rev/d2429d660af8
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.