Closed
Bug 853819
Opened 12 years ago
Closed 12 years ago
VKB does not appear when tapping on text inputs on flash sites
Categories
(Firefox for Android Graveyard :: Keyboards and IME, defect)
Tracking
(firefox20 unaffected, firefox21 unaffected, firefox22 verified)
VERIFIED
FIXED
Firefox 22
| Tracking | Status | |
|---|---|---|
| firefox20 | --- | unaffected |
| firefox21 | --- | unaffected |
| firefox22 | --- | verified |
People
(Reporter: csuciu, Assigned: jchen)
References
()
Details
(Keywords: regression)
Attachments
(1 file)
|
2.45 KB,
patch
|
cpeterson
:
review+
|
Details | Diff | Splinter Review |
Nightly 22.0a1 (2013-03-21)
Galaxy Nexus (4.1.1), Samsung Galaxy Note (4.0.4)
Steps:
1. Go to http://www.zasmakujelegancji.pl/
2. Tap on one of the text inputs (day/month/year)
Expected: VKB appears
Actual: Nothing happens
Also happens on http://www.tizag.com/pics/flash/form1easy.swf
Regression window:
Good build: 2013-03-05
Bad build: 2013-03-06
Possible range: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=015da7030aab&tochange=216ec69cc531
Comment 1•12 years ago
|
||
Component: General → Keyboards and IME
Flags: needinfo?(nchen)
Updated•12 years ago
|
tracking-fennec: --- → ?
| Assignee | ||
Comment 2•12 years ago
|
||
Bug 843048 actually.
| Assignee | ||
Comment 3•12 years ago
|
||
In bug 843048, we switched to using an IME event to notify Java during SetInputContext calls. However, IME events are processed by the focused window, which is not necessarily the top window. So we need to let the focused window process the SetInputContext call too, to ensure saved states are consistent.
Attachment #728300 -
Flags: review?(cpeterson)
Comment 4•12 years ago
|
||
Comment on attachment 728300 [details] [diff] [review]
Use focused window to process SetInputContext and GetInputContext (v1)
Review of attachment 728300 [details] [diff] [review]:
-----------------------------------------------------------------
LGTM
Attachment #728300 -
Flags: review?(cpeterson) → review+
| Assignee | ||
Updated•12 years ago
|
Keywords: checkin-needed
Comment 5•12 years ago
|
||
Keywords: checkin-needed
Comment 6•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 22
Comment 7•12 years ago
|
||
Verified fixed on:
-build: Firefox for Android 22.0a1 (2013-03-28)
-device: Samsung Galaxy Nexus
-OS: Android 4.1.1
Status: RESOLVED → VERIFIED
Updated•12 years ago
|
tracking-fennec: ? → ---
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•