Closed Bug 573796 Opened 10 years ago Closed 10 years ago

gFocusedWindow should be cleared on window destruction

Categories

(Core :: Widget: Android, defect)

All
Android
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: mwu, Assigned: mwu)

Details

Attachments

(1 file)

Attached patch FixSplinter 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.