Closed Bug 912748 Opened 11 years ago Closed 11 years ago

Geolocation crashes Nightly and Aurora running on Froyo

Categories

(Firefox for Android Graveyard :: General, defect)

All
Android
defect
Not set
normal

Tracking

(firefox25- wontfix, firefox26- fixed, fennec+)

RESOLVED FIXED
Firefox 26
Tracking Status
firefox25 - wontfix
firefox26 - fixed
fennec + ---

People

(Reporter: cpeterson, Assigned: cpeterson)

References

Details

Attachments

(1 file)

Fennec is calling GsmCellLocation.getPsc() which was added in Gingerbread: https://developer.android.com/reference/android/telephony/gsm/GsmCellLocation.html#getPsc%28%29 We need a version check for Froyo. All the other GsmCellLocation methods are supported by Froyo; we just want to skip getPsc().
This code path is (currently) #ifdef'd for just the Nightly and Aurora channels, so Beta and Release users are unaffected.
STR: - Install nightly on your android - Go to maps.google.com - Ask to be geolocated - See nightly crash. Here are the logs from adb logcat: E/AndroidRuntime( 973): FATAL EXCEPTION: Thread-9 E/AndroidRuntime( 973): java.lang.NoSuchMethodError: android.telephony.gsm.GsmCellLocation.getPsc E/AndroidRuntime( 973): at org.mozilla.mozstumbler.Scanner.getCellInfo(Scanner.java:264) E/AndroidRuntime( 973): at org.mozilla.mozstumbler.Scanner.onLocationChanged(Scanner.java:227) E/AndroidRuntime( 973): at android.location.LocationManager$ListenerTransport._handleMessage(Locat ionManager.java:191) E/AndroidRuntime( 973): at android.location.LocationManager$ListenerTransport.access$000(LocationM anager.java:124) E/AndroidRuntime( 973): at android.location.LocationManager$ListenerTransport$1.handleMessage(Loca tionManager.java:140) E/AndroidRuntime( 973): at android.os.Handler.dispatchMessage(Handler.java:99) E/AndroidRuntime( 973): at android.os.Looper.loop(Looper.java:144) E/AndroidRuntime( 973): at org.mozilla.mozstumbler.ScannerService$LooperThread.run(ScannerService. java:110)
Whiteboard: [good first bug][lang=java][mentor=cpeterson]
tracking-fennec: --- → ?
Blocks: 866957
tracking-fennec: ? → +
Assignee: nobody → cpeterson
Status: NEW → ASSIGNED
Whiteboard: [good first bug][lang=java][mentor=cpeterson]
Attached patch fix-froyo-crash.patch (deleted) — Splinter Review
GsmCellLocation.getPsc() is only supported on GB+.
Attachment #800348 - Flags: review?(blassey.bugs)
Attachment #800348 - Flags: review?(blassey.bugs) → review+
I don't think we need to track (or uplift) this fix for Aurora 25. This Froyo crashing feature is #ifdef'd only for the Nightly and Aurora channels. Nightly 26 will be promoted to Aurora in about one week and then both Nightly and Aurora will be fixed (and the feature will still be #ifdef'd off in Beta).
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: