Closed Bug 573796 Opened 10 years ago Closed 10 years ago

gFocusedWindow should be cleared on window destruction

Categories

(Core :: Widget: Android, defect)

Product:
Core
Component:
Widget: Android
Platform:
All
Android
Type:
defect
Priority:
Not set

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: mwu, Assigned: mwu)

Details

Attachments

(1 file)

Fix
461 bytes, patch
blassey
: review+
Details | Diff | Splinter Review
Attached patch Fix — — Splinter Review 
As reported by vlad.
Attachment #453131 - Flags: review?(blassey.bugs)
Comment on attachment 453131 [details] [diff] [review]
Fix

makes sense
Attachment #453131 - Flags: review?(blassey.bugs) → review+
looks like we could potentially be dropping key and ime events on the floor.  Do you know when this would happen and that its okay to do so?
(In reply to comment #2)
> looks like we could potentially be dropping key and ime events on the floor. 
> Do you know when this would happen and that its okay to do so?
I'd imagine that the new window should get focus so it shouldn't happen.. but it does since vlad hit it. IME events are actually pretty dangerous to drop. I'll investigate further (and fix in another bug).
http://hg.mozilla.org/mozilla-central/rev/7a66915cab32
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.