Closed Bug 851406 Opened 12 years ago Closed 12 years ago

MozKeyboard initialization fails in emulator with multithreaded image decoding patches applied; Gaia needs updating?

Categories

(Firefox OS Graveyard :: General, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: joe, Assigned: joe)

References

Details

Attachments

(2 files)

Attached patch rollup patch (deleted) — Splinter Review
If you apply the rollup patch attached here (I emailed a version of it to Fabrice) and run Try emulator builds on it, you get very frequent errors: No permission to use the keyboard API for app://system.gaiamobile.org This is insane, since the system app has the keyboard permission. *Except* - the Gaia patches in bug 847105 that ostensibly grant the keyboard and system app permission to the keyboard app have not been applied to the Gaia we ship with our emulator. I suspect (but have not yet proven) that my patches change initialization ordering, which causes the keyboard to be initialized. This would be fine except for the above. I have relatively conclusively proven that the keyboard isn't being initialized normally on mochitest and crashtest and the like; see this try build: https://tbpl.mozilla.org/?tree=Try&rev=8ebd786a5421 Fabrice tells me that he's going to run my patch against an actual device for more information, but I'm filing this to get more eyes.
rail/jgriffin, I'm not sure which of you usually builds new emulator builds, but can one of you build a new emulator with an updated gecko for me so that I can run it against try?
Flags: needinfo?(rail)
Flags: needinfo?(jgriffin)
Emulator builds are usually provided by jgriffin. Buildduty can upload them for you so you can push to try. Just ping buildduty or me on IRC.
Flags: needinfo?(rail)
size: 626324746 hash: 8f071f6d8295e1ecb2fbe04ae2f556d0f3617233a96538611b3a4c0f7e81703487c954c60e00c0cfa4245aad29c1f5f0fb9f64be23a7863b97e1f4e7cad1eb9c
So this does indeed fix the MozKeyboard bug, but unfortunately it makes several other unrelated text-related reftests fail: https://tbpl.mozilla.org/?tree=Try&rev=423b9fd6be30
These failures are caused by the emulator update, they also show up in bug 852111. We'll need someone to investigate them, or we can disable them. :(
If we can get someone to investigate this *very soon* we can go ahead without disabling, but I'm betting chances of that are not good . so - Do you want to disable or shall I?
cc'ing Jonas. Jonas, it looks like some code in the emulator has changed at some point which makes some fonts appear differently, which is breaking a lot of reftests. We need someone to investigate this, or alternately, we need to disable all those failing tests so :joedrew can land his patch, which requires a new emulator be uploaded to buildbot. Which do you prefer?
Flags: needinfo?(jonas)
Depends on: 852656
Attached patch update the emulator (deleted) — Splinter Review
This is just an update of the emulator to the version that jgriffin built and ran through try with the font update backed out. It seems to be green!
Attachment #727026 - Flags: review?(rail)
(In reply to Joe Drew (:JOEDREW! \o/) from comment #9) > Created attachment 727026 [details] [diff] [review] > update the emulator > > This is just an update of the emulator to the version that jgriffin built > and ran through try with the font update backed out. It seems to be green! You should be good to push that.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: