Closed Bug 480767 Opened 15 years ago Closed 15 years ago

Closing "Bookmark All Tabs" screws up focus

Categories

(Core :: DOM: UI Events & Focus Handling, defect, P1)

defect

Tracking

()

VERIFIED FIXED
mozilla1.9.2a1

People

(Reporter: jruderman, Assigned: smaug)

References

Details

(Keywords: regression, verified1.9.1)

Attachments

(1 file)

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2a1pre) Gecko/20090228 Minefield/3.2a1pre

Steps to reproduce:
1. Make sure you have two tabs open.
2. Make sure focus is on the page (not the address bar).
3. Press Cmd+Shift+D or select "Bookmark All Tabs..." from the Bookmarks menu.
4. Press Esc or click Cancel.

Result: The Firefox window goes from dark gray to light gray (indicating that it has lost focus).  Clicking does not restore focus completely.  Have to Cmd+Tab to recover.
Flags: blocking-firefox3.2?
I think this is the same as bug 480768, feel free to dupe either way... (note this happens on 1.9.1 as well).
I dunno.  I can't reproduce that one, and it doesn't really sound the same.
The underlying problem is the same, the focus isn't being taken away/restored from the parent window appropriately, the gray menu text is for windows only, mac doesn't have that (afaik).

Can you confirm the problem is in 3.1 as well?
Yes, the mentioned "bookmark all tabs" dialog is also a modal one. Due to we have more information in bug 480768 we should dupe this one against bug 480768.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → DUPLICATE
Blocks: 333198
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
I think this isn't actually the same problem as bug 480768.
I have a patch for this one, but it may not fix bug 480768 - waiting
tryserver builds to test.
This is a different approach to focus handling.
Allow NS_GOT/LOSTFOCUS and NS_ACTIVATE/DEACTIVATE to propagate to event state
manager, but don't dispatch focus/blur events.

The change to nsGlobalWindow isn't actually needed, but reduces the risk for
regressions, since event suppression is used then only when
there is sync XHR in the stack.

This fixes also bug 480768.

https://build.mozilla.org/tryserver-builds/2009-03-01_12:19-opettay@mozilla.com-dont_suppress_widget_events_and_suppress_only_when_needed/
Assignee: nobody → Olli.Pettay
Attachment #364809 - Flags: superreview?(jst)
Attachment #364809 - Flags: review?(jst)
Olli, I can see similar things on OS X and Windows when opening a new tab. Especially on OS X the whole application lose its focus. I have to switch to another application and back to get the focus back. This happens sporadically. Is it worth trying the build above to check?

Could you please adjust the product/component so we have the right one?
Status: REOPENED → ASSIGNED
OS: Mac OS X → All
Hardware: x86 → All
Attachment #364809 - Flags: superreview?(jst)
Attachment #364809 - Flags: superreview+
Attachment #364809 - Flags: review?(jst)
Attachment #364809 - Flags: review+
Marking this a P1 blocker, as it's a regression from P1 blocker bug 333198, we don't want one w/o the other.
Flags: blocking-firefox3.2? → blocking-firefox3.2+
Priority: -- → P1
Target Milestone: --- → Firefox 3.1b3
Component: Bookmarks & History → Event Handling
Flags: blocking-firefox3.2+
Product: Firefox → Core
QA Contact: bookmarks → events
Target Milestone: Firefox 3.1b3 → ---
(In reply to comment #9)
> Is it worth trying the build above to check?
Yes!
 
> Could you please adjust the product/component so we have the right one?
Done.
Flags: blocking1.9.2?
Target Milestone: --- → mozilla1.9.1b3
Flags: blocking1.9.2? → blocking1.9.2+
Flags: blocking1.9.2+ → blocking1.9.1+
Whiteboard: [needs landing]
http://hg.mozilla.org/mozilla-central/rev/0b0713f62e2a
Status: ASSIGNED → RESOLVED
Closed: 15 years ago15 years ago
Resolution: --- → FIXED
Blocks: 480768
Verified fixed on trunk and 1.9.1 with builds on OS X and Windows:

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2a1pre) Gecko/20090304 Minefield/3.2a1pre

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b3pre) Gecko/20090304 Shiretoko/3.1b3pre ID:20090304022008
Status: RESOLVED → VERIFIED
Whiteboard: [needs landing]
Target Milestone: mozilla1.9.1b3 → mozilla1.9.2a1
Component: Event Handling → User events and focus handling
You need to log in before you can comment on or make changes to this bug.