705993 - read recommended-addons.json from the profile dir

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Comment 1

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: patch

the content of the recommended-addons.json file in my profile is '{"addons":[]}', this shouldn't land until that has content

Comment 2

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

(In reply to Brad Lassey [:blassey] from comment #1)
> Created attachment 577872 [details] [diff] [review] [diff] [details] [review]
> patch
> 
> the content of the recommended-addons.json file in my profile is
> '{"addons":[]}', this shouldn't land until that has content

This is probably right. AMO has zero support for Fennec Native. There are zero add-ons in AMO for the app.. In fact, the two add-ons you have in the hard coded list will not install in Fennec Native.

Comment 3

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

Comment on attachment 577872 [details] [diff] [review]
patch

>diff --git a/mobile/android/base/AboutHomeContent.java b/mobile/android/base/AboutHomeContent.java
>@@ -152,17 +152,37 @@ public class AboutHomeContent extends Li
>         });
>     }
> 
>+
>+    InputStream getProfileRecommendedAddonsStream() {

nit: Remove the extra line

>+        }
>+        return null;
>+    }
>+    InputStream getRecommendedAddonsStream(Activity activity) throws Exception{
>+        InputStream is = getProfileRecommendedAddonsStream();

nit: Add a line break between methods

Comment 4

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

Based on comment 2, you should feel free to land this when the dependent bug is ready.

Comment 5

[Brad Lassey [:blassey] (use needinfo?)]

pushed https://hg.mozilla.org/projects/birch/rev/21ee0cd78262