Closed Bug 298794 Opened 19 years ago Closed 19 years ago

fastback interacts poorly with event queue prioritization

Categories

(Core :: DOM: Navigation, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: bryner, Assigned: bryner)

References

Details

Attachments

(1 file)

Docshell has code that decreases the relative priority of user events when
starting a load.  It's balanced by code in EndPageLoad that resets user events
to their original priority.  However, in the fastback case the calls aren't
balanced, so we'd likely stop decreasing user event priority during real loads
once you've done a fastback operation.  This could help explain some reports of
sites taking longer to load with fastback enabled.
Attached patch patchSplinter Review
Attachment #187370 - Flags: superreview?(darin)
Attachment #187370 - Flags: review?(darin)
Comment on attachment 187370 [details] [diff] [review]
patch

RestorePresentation is really getting to be a long function :-/

r+sr=darin
Attachment #187370 - Flags: superreview?(darin)
Attachment #187370 - Flags: superreview+
Attachment #187370 - Flags: review?(darin)
Attachment #187370 - Flags: review+
Comment on attachment 187370 [details] [diff] [review]
patch

requesting approval, very safe fix
Attachment #187370 - Flags: approval1.8b3?
Comment on attachment 187370 [details] [diff] [review]
patch

a=chase
Attachment #187370 - Flags: approval1.8b3? → approval1.8b3+
checked in
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: