Closed Bug 498609 Opened 16 years ago Closed 16 years ago

Cannot focus location bar after cancelling/pausing a download

Categories

(Core :: XUL, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: phiw2, Assigned: enndeakin)

References

Details

(Keywords: regression)

Attachments

(1 file, 1 obsolete file)

STR 1. start downloading a file 2. download manager opens in front of main window 3. cancel or pause the download 4. switch to main window At this point it is impossible * to focus and input something the location or search bar (I normally use the keyboard shortcut, but mouse click fails) * to set focus in a form field. fails 20090610110529-cabb8925dcd3-firefox-3.6a1pre.en-US.mac.dmg 10-Jun-2009 works 20090610090902-1057ca8f2a91-firefox-3.6a1pre.en-US.mac.dmg 10-Jun-2009 --> bug 178324 note 1. right-click on a file in the download manager doesn't seem to cause problems 2. I have have experienced a similar issue (I think) while debugging a webpage and switching between 2 tabs - possibly having used Firebug. But I could not reproduce that.
Attached patch fix (obsolete) — Splinter Review
Seems to work but can't figure out how to get a test for it
Assignee: nobody → enndeakin
Status: NEW → ASSIGNED
Blocks: 498643
Attached patch include a testSplinter Review
Clear out mFocusedContent if it isn't in a document. This problem occurs when a focused element is in a binding which goes away.
Attachment #383497 - Attachment is obsolete: true
Attachment #383710 - Flags: superreview?(Olli.Pettay)
Attachment #383710 - Flags: review?(Olli.Pettay)
Attachment #383710 - Flags: superreview?(Olli.Pettay)
Attachment #383710 - Flags: superreview+
Attachment #383710 - Flags: review?(Olli.Pettay)
Attachment #383710 - Flags: review+
Comment on attachment 383710 [details] [diff] [review] include a test > PRBool > nsFocusManager::Blur(nsPIDOMWindow* aWindowToClear, > nsPIDOMWindow* aAncestorWindowToFocus, > PRBool aIsLeavingDocument) > { Please document in .h what the return value means. >+<window title="Focus Switch Binding Test" >+ onload="setTimeout(switchBinding, 0);" Could you perhaps move this timeout to the ctor of the first binding, after .focus()?
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: