902176 - Gaia Firefox development environment is broken

Status: RESOLVED FIXED

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

Description

[Kevin Grandon :kgrandon]

I'm seeing the following errors being logged:

System JS : ERROR (null):0
                     uncaught exception: 2147500033
No permission to use the keyboard API for http://keyboard.gaiamobile.org:8080
JavaScript error: http://keyboard.gaiamobile.org:8080/js/keyboard.js, line 1753: navigator.mozKeyboard is null

Comment 1

[Kevin Grandon :kgrandon]

After bisecting, I've found that the patch for bug 894287 is breaking gaia in nightly.

Comment 2

[Kevin Grandon :kgrandon]

Attachment: Github Pull Request

Hey - adding a simple shim to gaia. Can someone review?

Comment 3

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

Comment on attachment 786520 [details]
Github Pull Request

Next time you likely want to ask etienne, he really likes the telephony API. It has consumed a big part of his time during the last year :)

Comment 4

[Kevin Grandon :kgrandon]

Landed in master: https://github.com/mozilla-b2g/gaia/commit/70a9797adb9f38efc3f40a4ac8748dd401fca861

Comment 5

[Axel Hecht [:Pike]]

bug 894287 will land on v1-train, so this one should, too?

Comment 6

[Kevin Grandon :kgrandon]

This is "NPOTB", so it wouldn't hurt to uplift as well.

Comment 7

[John Ford [:jhford] CET/CEST Berlin Time]

I was not able to uplift this bug to v1-train.  If this bug has dependencies which are not marked in this bug, please comment on this bug.  If this bug depends on patches that aren't approved for v1-train, we need to re-evaluate the approval.  Otherwise, if this is just a merge conflict, you might be able to resolve it with:

  git checkout v1-train
  git cherry-pick -x -m1 70a9797adb9f38efc3f40a4ac8748dd401fca861
  <RESOLVE MERGE CONFLICTS>
  git commit

Comment 8

[Kevin Grandon :kgrandon]

I haven't heard any complaints from users who are unable to use this in v1-train. As we're now focusing on 1.2, this is really not important. Clearing the leo flag, please re-nom if you *really* need this.