1082876 - TypeError: context is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 84

Status: RESOLVED DUPLICATE of bug 1079810

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

Description

[Anshul]

[Blocking Requested - why for this release]: This exception is preventing us to run our unit test to ensure the quality of this release.

I am seeing the following exception while running marionette test cases related to SIM card locks, which is causing the test to fail. 

JavascriptException: TypeError: context is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 84

Comment 1

[Doug Sherk (:drs) (inactive)]

Please provide a STR.

Comment 2

[Anshul]

I don't have STR as the issue doesn't happen persistently.

Comment 3

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

triage: I'm afraid we can't + it without being able to reproduce it.
Could you provide some more information?

Comment 4

[Anshul]

Doug, the error is not easily reproducible and after spending quite some time I couldn't produce STR. I am still seeing the error 1/20 times I run my test script. Could you point me to the file that might be throwing this error, I can add some debug message on console.trace() to possibly help debug?

Comment 5

[Doug Sherk (:drs) (inactive)]

I believe that the error is somewhere in this file:
https://github.com/mozilla-b2g/gaia/blob/master/shared/js/dialer/tone_player.js

Comment 6

[bhavana bajaj [:bajaj]]

waiting for anshul to see if he has more logs to investigate this further per comment #4,5.

Comment 7

[Gabriele Svelto [:gsvelto]]

I think this might be caused by a race condition whereas we trash the audio context and then try to play. See bug 834530 comment 50 of why this might be happening. The changes I'm doing in that bug should prevent those races so it might be worth re-testing this after that bug lands.

Comment 8

[Doug Sherk (:drs) (inactive)]

Anshul, could you try the patches in bug 834530 and see if they fix the issue here?

That would be attachment 8508778 [details] [diff] [review] and attachment 8511646 [details] [diff] [review].

Comment 9

[Doug Sherk (:drs) (inactive)]

Toggling needinfo off and on again for visibility.

Comment 10

[Anshul]

Doug I am not able to reproduce this error anymore even without the patch. I am marking this issue closed for now.

Comment 11

[Doug Sherk (:drs) (inactive)]

Thanks for checking. I could definitely be wrong here, but I strongly suspect that bug 1079810 fixed this.