1374200 - Crash in java.lang.NoSuchMethodError: android.net.Uri.normalizeScheme at org.mozilla.gecko.IntentHelper.getOpenURIIntentInner(IntentHelper.java)

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Julien Cristau [:jcristau]]

This bug was filed from the Socorro interface and is 
report bp-07270d55-293f-48c9-8197-45ec00170619.
=============================================================

This signature seems to be spiking since 54 hit release (more than 600 crashes over the last 3 days).  Java exception:

java.lang.NoSuchMethodError: android.net.Uri.normalizeScheme
	at org.mozilla.gecko.IntentHelper.getOpenURIIntentInner(IntentHelper.java:284)
	at org.mozilla.gecko.IntentHelper.getOpenURIIntent(IntentHelper.java:241)
	at org.mozilla.gecko.GeckoApp.getHandlersForURL(GeckoApp.java:2984)
	at org.mozilla.gecko.GeckoAppShell.getHandlersForURL(GeckoAppShell.java:857)
	at org.mozilla.gecko.mozglue.GeckoLoader.nativeRun(Native Method)
	at org.mozilla.gecko.GeckoThread.run(GeckoThread.java:583)

Comment 1

[Gerry Chang [:gchang]]

Track 54+ as there is a spike in 54. 

Hi :snorp,
This might also need your help to look at?

Comment 3

[Marcia Knous [:marcia]]

All the crashes are API 15.

Comment 4

[Richard Newman [:rnewman]]

This method was added in API 16. Lint should have caught this.

Comment 5

[Richard Newman [:rnewman]]

Regression from Bug 1356893.

Comment 6

[Richard Newman [:rnewman]]

There's a handy method elsewhere in the file to avoid this issue:

https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/java/org/mozilla/gecko/IntentHelper.java#433

Should be an easy switchover. This will need uplift.

Comment 7

[Eugen Sawin [:esawin]]

Attachment: 0001-Bug-1374200-1.0-Use-custom-URI-scheme-normalization-.patch

Use custom URI scheme normalization.

I should have checked for the API requirement, it being such a rudimentary operation I assumed none.

Who's a good contact for the linter nowadays? We should look into why it hasn't caught the issue.

Comment 8

[Richard Newman [:rnewman]]

(In reply to Eugen Sawin [:esawin] from comment #7)

> Who's a good contact for the linter nowadays? We should look into why it
> hasn't caught the issue.

I think Grisha was the last person to run into this kind of thing.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a8bbfc7c430c
[1.0] Use custom URI scheme normalization to work around API level 16 restriction. r=rnewman

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/a8bbfc7c430c

Comment 11

[Julien Cristau [:jcristau]]

Please request uplift to beta (and maybe release) when you get a chance.

Comment 12

[Eugen Sawin [:esawin]]

Comment on attachment 8880045 [details] [diff] [review]
0001-Bug-1374200-1.0-Use-custom-URI-scheme-normalization-.patch

Approval Request Comment
[Feature/Bug causing the regression]: Bug 1356893.
[User impact if declined]: Crash on older devices.
[Is this code covered by automated tests?]: No.
[Has the fix been verified in Nightly?]: Yes.
[Needs manual test from QE? If yes, steps to reproduce]: 
[List of other uplifts needed for the feature/fix]:
[Is the change risky?]: No.
[Why is the change risky/not risky?]: We only use a custom (tested) function instead of an API call that is not available on older devices. 
[String changes made/needed]: None.

Comment 13

[Liz Henry (:lizzard) (relman/hg->git project)]

Comment on attachment 8880045 [details] [diff] [review]
0001-Bug-1374200-1.0-Use-custom-URI-scheme-normalization-.patch

Avoids a crash, let's take this for beta 5. Gerry, up to you for a 54 dot release

Comment 14

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/dab6493e5a5e

Comment 15

[Gerry Chang [:gchang]]

Comment on attachment 8880045 [details] [diff] [review]
0001-Bug-1374200-1.0-Use-custom-URI-scheme-normalization-.patch

Fix a crash. Release54+. Should be in 54.0.1.

Comment 16

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-release/rev/97f3f0804433