Closed
Bug 498609
Opened 16 years ago
Closed 16 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•16 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•16 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•16 years ago
|
Attachment #383710 -
Flags: superreview?(Olli.Pettay)
Attachment #383710 -
Flags: superreview+
Attachment #383710 -
Flags: review?(Olli.Pettay)
Attachment #383710 -
Flags: review+
Comment 3•16 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•16 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
| Assignee | ||
Updated•16 years ago
|
Flags: in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•