Closed Bug 83992 Opened 24 years ago Closed 24 years ago

Holding down a cursor key to scroll in a form's <select> freezes Mozilla.

Categories

(Core :: Graphics: ImageLib, defect, P2)

defect

Tracking

()

VERIFIED FIXED
mozilla0.9.2

People

(Reporter: kbh7, Assigned: pavlov)

References

()

Details

(Keywords: hang, Whiteboard: r=vakesi, sr=blizzard need a=asa)

Attachments

(3 files)

From Bugzilla Helper: User-Agent: Mozilla/5.0 (X11; U; Linux 2.2.19 i686; en-US; rv:0.9+) Gecko/20010601 BuildID: 2001060121 If I hold down the down-cursor-key in this particular <select> form to scroll down through the entries, it freezes. Reproducible: Always Steps to Reproduce: 1. Go to http://www.pactecenclosures.com 2. Click on "ENCLOSURES" (left side). 3. Click the first size from the list on the left, "1.4 x 2.3 x 0.4", to give this list keyboard focus. 4. Hold down the down-arrow key. Actual Results: After it gets down a couple lines, the browser completely stops responding to any input, and won't even redraw itself: time to restart Mozilla. Expected Results: It should keep scrolling down the list. There's a lot of Javascript on this page. The Javascript updates an image with the user's selection; perhaps asking it to redraw too quickly is causing this?
Severity: normal → critical
Keywords: hang
reasigning to core layout, not sure who's bug this should be
Assignee: rods → attinasi
Status: UNCONFIRMED → NEW
Ever confirmed: true
I get the exact same stack. Looks like a problem with the imgRequestProxy to me, so over to imglib.
Assignee: attinasi → pavlov
Component: HTML Form Controls → ImageLib
QA Contact: vladimire → tpreston
targeting 0.9.2. the lock isn't needed (right now) so here comes a patch to remove it.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.2
Attached patch patch to remove the lock (deleted) — Splinter Review
OS: Linux → All
Priority: -- → P2
Hardware: PC → All
Okay, pav sez his thread patch has this change, so it won't get lost when it is needed. r=saari
Keywords: patch
Whiteboard: r=saari, need sr= and a=
Attached patch better fix (deleted) — Splinter Review
r=valeski
sr=blizzard
Whiteboard: r=saari, need sr= and a= → r=vakesi, sr=blizzard need a=
Blocks: 83989
a= asa@mozilla.org for checkin to the trunk. (on behalf of drivers)
Whiteboard: r=vakesi, sr=blizzard need a= → r=vakesi, sr=blizzard need a=asa
fix checked in
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Verified fixed w2k build 2001062004 Verified fixed linux build 2001062008 Verified fixed mac build 2001062005
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: