922329 - HelperApps.getAppsForUri does not check for empty results and throws

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: Check for valid return data

We assume that "Intent:GetHandlers" will always work, but GeckoView does not implement it, and may never implement it. This causes the JSON.parse to throw and breaks the initialization of many other parts of code. Effectively killing GeckoView.

In the long term, I want to split out code from browser.js into 3 code paths:
* Firefox (default)
* WebApp
* GeckoView

Some parts of the code won't be needed for WebApps and GeckoView, so let's not even add it.

In the meantime, this fixes the current problem. With this patch, GeckoView example apps work on everything but the Nexus 4 (bug 921792)

Comment 1

[Wesley Johnston (:wesj)]

Comment on attachment 812258 [details] [diff] [review]
Check for valid return data

Review of attachment 812258 [details] [diff] [review]:
-----------------------------------------------------------------

::: mobile/android/chrome/content/HelperApps.js
@@ +60,5 @@
>        packageName: "",
>        className: ""
>      };
> +    let data = sendMessageToJava(msg);
> +    if (data) {

Early return instead?

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to Wesley Johnston (:wesj) from comment #1)

> > +    let data = sendMessageToJava(msg);
> > +    if (data) {
> 
> Early return instead?

OK

https://hg.mozilla.org/integration/fx-team/rev/dd3dd64aa432

Comment 3

[Ed Morley [:emorley]]

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

Comment 5

[Brian Nicholson (:bnicholson)]

This fix needs uplift to Aurora.

Comment 6

[Brian Nicholson (:bnicholson)]

Err...wrong flag. Apologize for bugspam.

Comment 7

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 812258 [details] [diff] [review]
Check for valid return data

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: things like bug 920544
Testing completed (on m-c, etc.): landed and working OK
Risk to taking this patch (and alternatives if risky): low risk. we just check for empty data.
String or IDL/UUID changes made by this patch: none

When uplifting, please use the checked in patch since it is more simple:
https://hg.mozilla.org/mozilla-central/rev/dd3dd64aa432

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/0857b261c4e1