Closed Bug 956428 Opened 7 years ago Closed 7 years ago

JavaScript Error: "conn.getPreferredNetworkType is not a function"

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(firefox29 fixed, b2g-v1.3 affected)

RESOLVED FIXED
Tracking Status
firefox29 --- fixed
b2g-v1.3 --- affected

People

(Reporter: gkw, Assigned: arthurcc)

References

Details

(Keywords: regression)

Attachments

(1 file)

JavaScript error: http://system.gaiamobile.org:8080/js/emergency_callback_manager.js, line 191: conn.getPreferredNetworkType is not a function

Not sure if this is likely to be related to:

https://github.com/mozilla-b2g/gaia/blob/f55764b3cfb5937e8ca58835342a67059cd3ad8f/apps/system/js/emergency_callback_manager.js#L191

The line was merged in https://github.com/mozilla-b2g/gaia/commit/3508d5c030f3c3474a6566140aa663d07801d333#diff-bdc40303b623075bfb9cb8f7074db7c8R191 so cc'ing :jaoo and setting needinfo.

I found this from poking around marifuzz by starting B2G Desktop from rev 4ecd4f2ba149 with Gaia profile rev f55764b.
Flags: needinfo?(josea.olivera)
Component: Gaia → Gaia::System
So, is this happening in B2G desktop builds?
Flags: needinfo?(josea.olivera) → needinfo?(gary)
(In reply to José Antonio Olivera Ortega [:jaoo] from comment #1)
> So, is this happening in B2G desktop builds?

Yes. See comment 0:

(In reply to Gary Kwong [:gkw] [:nth10sd] from comment #0)
> I found this from poking around marifuzz by starting B2G Desktop from rev
> 4ecd4f2ba149 with Gaia profile rev f55764b.
Flags: needinfo?(gary)
Assignee: nobody → arthur.chen
Jose, could you help review the change? Thanks!
Attachment #8360321 - Flags: review?(josea.olivera)
Comment on attachment 8360321 [details]
Link to https://github.com/mozilla-b2g/gaia/pull/15351

This issue happens on desktop builds. I've noticed Kevin added a fix in bug 959107 so there wouldn't be necessary this change, am I right?
Attachment #8360321 - Flags: review?(josea.olivera)
(In reply to José Antonio Olivera Ortega [:jaoo] from comment #4)
> Comment on attachment 8360321 [details]
> Link to https://github.com/mozilla-b2g/gaia/pull/15351
> 
> This issue happens on desktop builds. I've noticed Kevin added a fix in bug
> 959107 so there wouldn't be necessary this change, am I right?

Oops, sorry I meant bug 956012.

As this bug could be fixed by that bug It would be great to check whether it is still happening. Gary, could you give it a try please? Thanks.
Flags: needinfo?(gary)
See Also: → 956012
Comment on attachment 8360321 [details]
Link to https://github.com/mozilla-b2g/gaia/pull/15351

It is still better to check the existence of the APIs in case the code is running on environments without them.
Attachment #8360321 - Flags: review?(josea.olivera)
> As this bug could be fixed by that bug It would be great to check whether it
> is still happening. Gary, could you give it a try please? Thanks.

I'd prefer that the fix (comment 6) be verified later after it has landed.
Flags: needinfo?(gary)
Comment on attachment 8360321 [details]
Link to https://github.com/mozilla-b2g/gaia/pull/15351

LGTM. Thanks Arthur.

r=me
Attachment #8360321 - Flags: review?(josea.olivera) → review+
Thanks Jose!

master: d82724dde400928ad889084c9d9571f6861d8ffe
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.