996528 - Gaia Keyboard could not show up when running in Firefox Nightly

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Keyboard, defect)

Description

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

STR
===
1. run Gaia with "DEBUG=1 make" + Firefox Nightly - 31.0a1 (2014-04-14).
2. Found that Keyboard could not show up.

After checking the system app could not get mozChrome event: inputmethod-contextchange.

Comment 1

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

Hi Jan, Yuan,

Have you encountered this situation? Any ideas?
Thanks.

Comment 2

[Jan Jongboom [:janjongboom] (inactive)]

I saw it today. Don't know why. Within the keyboard it never builds up. If any of you can dive deeper in it that'd be great, as I'm conference today and traveling tomorrow.

Comment 3

[Yuan Xulei [:yxl] (Leave from May 18)]

Bug 986992 landed. May that bug cause it? I will look into it later today.

Comment 4

[Yuan Xulei [:yxl] (Leave from May 18)]

Bug 963239 broke the keyboard on nightly. Backing out its patch, keyboard shows again.

Comment 5

[Alexandre Poirot [:ochameau]]

Attachment: Hack keyboard.jsm from gaia addon

Shipping SystemAppProxy.jsm in nightly would fix this issue but that's gonna be hard as it lives in b2g/ folder. It would also require calling SystemAppProxy.jsm from gaia addon.

I would prefer shipping SystemAppProxy.jsm in gaia addon, but that's hard to override its url (because it is resource: and not chrome:):
  resource://gre/modules/SystemAppProxy.jsm

At the end, this highlights how bad the nightly support is and how much we need the mulet (it is coming soon!). In the meantime I think we just have to pile up the hacks in the gaia addons :/

Comment 6

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 8408222 [details] [review]
Hack keyboard.jsm from gaia addon

One more hack. Hopefully the Mulet will resolve everything ;)

Comment 7

[Alexandre Poirot [:ochameau]]

master: https://github.com/mozilla-b2g/gaia/commit/2a380674ce0f055ced369e2eb449587208f7a710