Closed
Bug 498609
Opened 15 years ago
Closed 15 years ago
Cannot focus location bar after cancelling/pausing a download
Categories
(Core :: XUL, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: phiw2, Assigned: enndeakin)
References
Details
(Keywords: regression)
Attachments
(1 file, 1 obsolete file)
5.07 KB,
patch
|
smaug
:
review+
smaug
:
superreview+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•15 years ago
|
||
Seems to work but can't figure out how to get a test for it
Assignee: nobody → enndeakin
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•15 years ago
|
||
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)
Updated•15 years ago
|
Attachment #383710 -
Flags: superreview?(Olli.Pettay)
Attachment #383710 -
Flags: superreview+
Attachment #383710 -
Flags: review?(Olli.Pettay)
Attachment #383710 -
Flags: review+
Comment 3•15 years ago
|
||
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()?
Assignee | ||
Comment 4•15 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/45cf3a994809
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•15 years ago
|
Flags: in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•