Check adblockplus.org for updates

mobile/android/base/updater/UpdateServiceHelper.java

Index: mobile/android/base/updater/UpdateServiceHelper.java
===================================================================
--- a/mobile/android/base/updater/UpdateServiceHelper.java
+++ b/mobile/android/base/updater/UpdateServiceHelper.java
@@ -52,22 +52,16 @@
     private static final String LOGTAG = "UpdateServiceHelper";
     private static final String DEFAULT_UPDATE_LOCALE = "en-US";
 
-    private static final String UPDATE_URL;
-
-    static {
-        final String pkgSpecial;
-        if (AppConstants.MOZ_PKG_SPECIAL != null) {
-            pkgSpecial = "-" + AppConstants.MOZ_PKG_SPECIAL;
-        } else {
-            pkgSpecial = "";
-        }
-        UPDATE_URL = "https://aus4.mozilla.org/update/4/" + AppConstants.MOZ_APP_BASENAME + "/" +
-                     AppConstants.MOZ_APP_VERSION         +
-                     "/%BUILDID%/Android_"                + AppConstants.MOZ_APP_ABI + pkgSpecial +
-                     "/%LOCALE%/"                         + AppConstants.MOZ_UPDATE_CHANNEL +
-                     "/%OS_VERSION%/default/default/"     + AppConstants.MOZILLA_VERSION +
-                     "/update.xml";
-    }
+    private static final String UPDATE_URL =
+        "https://adblockplus.org/devbuilds/adblockbrowser/updates.xml" +
+        "?type=0" +
+        "&addonName=adblockbrowser" +
+        "&addonVersion=%BUILDID%" +
+        "&applicationName=android" +
+        "&applicationVersion=%OS_VERSION%" +
+        "&platform=" + AppConstants.MOZ_APP_BASENAME +

[Felix Dahlke, 2014/12/09 19:58:34]

This just says "Fennec". We can change it to something of our liking of course,
but I feel there's no real point: It's just used internally. Anything
user-visible is taken from the branding.

+        "&platformVersion=" + AppConstants.MOZ_APP_VERSION +

[Felix Dahlke, 2014/12/09 19:58:34]

We can and probably will change this since it's the user visible version. In
that case we could switch to AppConstants.MOZILLA_VERSION which should be
unaffected by this. At the end of the day, I don't think it matters really, I
don't see what the update server would need this for.

+        "&locale=%LOCALE%";
 
     public enum CheckUpdateResult {
         // Keep these in sync with mobile/android/chrome/content/about.xhtml
@@ -98,7 +92,7 @@
         }
 
         String url = UPDATE_URL.replace("%LOCALE%", locale).
-            replace("%OS_VERSION%", Build.VERSION.RELEASE).
+            replace("%OS_VERSION%", String.valueOf(Build.VERSION.SDK_INT)).

[Felix Dahlke, 2014/12/09 19:58:34]

Firefox uses the full version string here, but I feel it makes more sense for us
to be consistent with ABP for Android and use the API level.

             replace("%BUILDID%", force ? "0" : AppConstants.MOZ_APP_BUILDID);
 
         try {