Closed Bug 43428 Opened 25 years ago Closed 25 years ago

Tree auto-scrolling crashes in StopTracking()

Categories

(Core :: XUL, defect, P3)

defect

Tracking

()

VERIFIED FIXED

People

(Reporter: mikepinkerton, Assigned: mikepinkerton)

Details

(Keywords: crash, Whiteboard: [nsbeta2-][nsbeta3+])

Here's how to repro this: - launch navigator with sidebar open to bookmarks - open the bookmarks window - drag a bookmark in the BM window quickly out of the window to the side and over something where the drop isn't allowed (a dos window is a good place). be sure to do this quickly since the bug is dependent on an event not being triggered. - close the bookmarks window - start a drag in the navigator sidebar. boom! The reason for this is that the bookmark tree is not unregistering itself with the drag session because event capture does not correctly give us the notification that the mouse has left the window. This is probably because mouse capture does not work for drag events on win32 and linux (note, this works on macos because i know how to fix that). cc'ing rod because he knows how the mouse capture works on win32, and pav for linux.
adding keywords, milestone, accepting.
Status: NEW → ASSIGNED
Keywords: crash, nsbeta2
Target Milestone: --- → M17
Target Milestone: M17 → M18
Putting on [nsbeta2+] radar for beta2 fix.
Whiteboard: [nsbeta2+]
this isn't going to make it for beta2, too many uncertainties. let's hold off for beta3.
Keywords: nsbeta3
Whiteboard: [nsbeta2+]
Target Milestone: M18 → M20
Putting on [nsbeta2-] radar. Not critical to beta2.
Whiteboard: [nsbeta2-]
adding nsbeta3+
Whiteboard: [nsbeta2-] → [nsbeta2-][nsbeta3+]
fix checked in. we no longer use capturing or timers or anything of the sort, so there's nothing to miss or not unregister and crash.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
verified fixed mac/linux/win32 2000090508, for pink's steps to reproduce, above.
Status: RESOLVED → VERIFIED
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.