Closed
Bug 1121430
Opened 10 years ago
Closed 10 years ago
Backspace doesn't work any more
Categories
(Firefox for Android Graveyard :: Keyboards and IME, defect)
Tracking
(firefox37+ fixed, firefox38 fixed, fennec37+)
VERIFIED
FIXED
Firefox 38
People
(Reporter: kats, Assigned: kats)
References
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
patch
|
kats
:
review+
lmandel
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
In today's nightly using the default Google keyboard on a nexus4 the backspace key doesn't work. Also on this keyboard i have the option enabled to allow swype-like word entry by sliding through the letters and usually the keyboard auto-inserts spaces between each word but that's not happening today.
Comment 2•10 years ago
|
||
No issues over here on my Nexus 6 (5.0.1), trunk (01/14). I ran through both content and our address bar and could not reproduce. The default keyboard in Lollipop has the gesture input enabled by default and it provided a gap between words by default. WFM.
If this is truly reproducible it should be simple enough for the reporter to help out finding a range. You will need 0.31 based on the new APK directory layout on our FTP: https://pypi.python.org/pypi/mozregression/0.31
Assignee | ||
Comment 3•10 years ago
|
||
Regression range on m-c:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=286e1f883fdb&tochange=643589c3ef94
I'll bisect inbound; there's a few IME patches by jim in there that look likely.
Assignee | ||
Comment 4•10 years ago
|
||
Assignee | ||
Comment 5•10 years ago
|
||
Kind of surprising since I suspected bug 1112212 but I double-checked that the build at https://hg.mozilla.org/integration/mozilla-inbound/rev/3cfe8379a8a5 doesn't show the problem.
Updated•10 years ago
|
Assignee | ||
Comment 6•10 years ago
|
||
Any idea how bug 1103036 might have affected keyboard input in Fennec? As far as I know the ContentChild/ContentParent stuff isn't even used in Fennec...
I don't know much about Fennec, but that seems really surprising. Jim would know, but unfortunately he's on vacation for a while.
Flags: needinfo?(wmccloskey)
Comment 8•10 years ago
|
||
(In reply to Bill McCloskey (:billm) from comment #7)
> I don't know much about Fennec, but that seems really surprising. Jim would
> know, but unfortunately he's on vacation for a while.
For a long while.
Kats - If you're confident a backout fixes this problem, we should be considering it.
Comment 9•10 years ago
|
||
(In reply to Mark Finkle (:mfinkle) from comment #8)
> Kats - If you're confident a backout fixes this problem, we should be
> considering it.
If you do back it out, you'll probably also have to back out bug 1121263, bug 1087613, bug 1083897 and bug 1091766, because I'm pretty sure all of the e10s leak checking stuff depends on it.
Assignee | ||
Comment 10•10 years ago
|
||
So I was also very surprised that the bisection pointed to bug 1103036 instead of bug 1112212, and so I did a try push [1] at cset 3cfe8379a8a5 to get a clobber-build and re-verify. And that build *does* manifest the problem. And then I triple-checked the inbound build for 3cfe8379a8a5 [2] and it *also* manifests the problem. So I don't know what I checked and double-checked before but it was likely not the right build.
I also triple-checked cset ab841f36d62f and that does not have the problem, so the regression is actually from bug 1112212 and the world makes sense again. Sorry for the screwup!
[1] https://treeherder.mozilla.org/#/jobs?repo=try&revision=977fdfe1b633
[2] http://ftp.mozilla.org/pub/mozilla.org/mobile/tinderbox-builds/mozilla-inbound-android-api-11/1420911932/
Assignee | ||
Comment 11•10 years ago
|
||
Try push at https://treeherder.mozilla.org/#/jobs?repo=try&revision=6aad8c1ee2d4 which I'll verify fixes the problem before I land.
Assignee: nobody → bugmail.mozilla
Attachment #8549625 -
Flags: review+
Assignee | ||
Comment 12•10 years ago
|
||
The try build with this backout patch fixes the problem, so I landed the backout:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a220d54c8c14
I had to rebase around cset e1dd58a149d2 which just landed on inbound; hopefully I didn't screw that up.
Comment 13•10 years ago
|
||
Assignee | ||
Comment 14•10 years ago
|
||
Comment on attachment 8549625 [details] [diff] [review]
Backout of bug 1112212
Approval Request Comment
[Feature/regressing bug #]: bug 1112212
[User impact if declined]: on some devices/keyboards it's impossible to delete text in a text field or textarea using the backspace key.
[Describe test coverage new/current, TBPL]: local testing only so far
[Risks and why]: is a backout of a relatively recent landing so it should be low risk and will restore the bugs it was fixing (which are less severe than the regression)
[String/UUID change made/needed]: none
Attachment #8549625 -
Flags: approval-mozilla-aurora?
Updated•10 years ago
|
tracking-fennec: ? → 37+
Assignee | ||
Comment 15•10 years ago
|
||
This already got merged.
https://hg.mozilla.org/mozilla-central/rev/a220d54c8c14
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•10 years ago
|
Flags: needinfo?(nchen)
Updated•10 years ago
|
tracking-firefox37:
--- → +
Comment 17•10 years ago
|
||
Comment on attachment 8549625 [details] [diff] [review]
Backout of bug 1112212
Aurora+ for the backout.
Attachment #8549625 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 18•10 years ago
|
||
Target Milestone: --- → Firefox 38
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
•