Issue 6187 - Adjust query string parameters for Yandex

adblockplussbrowser/src/main/java/org/adblockplus/sbrowser/contentblocker/engine/AppInfo.java

Index: adblockplussbrowser/src/main/java/org/adblockplus/sbrowser/contentblocker/engine/AppInfo.java
===================================================================
--- a/adblockplussbrowser/src/main/java/org/adblockplus/sbrowser/contentblocker/engine/AppInfo.java
+++ b/adblockplussbrowser/src/main/java/org/adblockplus/sbrowser/contentblocker/engine/AppInfo.java
@@ -15,16 +15,17 @@
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
 package org.adblockplus.sbrowser.contentblocker.engine;
 
 import java.util.Locale;
 import android.annotation.SuppressLint;
 import android.content.Context;
+import android.content.pm.PackageManager;
 import android.os.Build;
 
 @SuppressLint("DefaultLocale")
 public class AppInfo
 {
   public final String addonName;
   public final String addonVersion;
   public final String application;
@@ -78,20 +79,23 @@ public class AppInfo
   public static AppInfo create(final Context context)
   {
     return new Builder().autoFill(context).build();
   }
 
   public static class Builder
   {
     private static final String SBROWSER_PACKAGE_NAME = "com.sec.android.app.sbrowser";
+    private static final String SBROWSER_APP_NAME = "sbrowser";
+    private static final String YANDEX_PACKAGE_NAME = "com.yandex.browser";
+    private static final String YANDEX_APP_NAME = "yandex";
 
     private String addonName = "adblockplussbrowser";
     private String addonVersion = "0";
-    private String application = "sbrowser";
+    private String application = "";
     private String applicationVersion = "0";
     private String platform = "android";
     private String platformVersion = Integer.toString(Build.VERSION.SDK_INT);
     private String locale = "en-US";
 
     public Builder autoFill(Context context)
     {
       try
@@ -111,18 +115,44 @@ public class AppInfo
       }
       catch (Throwable t)
       {
         // ignored
       }
 
       this.locale = Locale.getDefault().toString().replace('_', '-');
 
+      this.application = checkForCompatibleInstalledBrowser(context.getPackageManager());
+
       return this;
     }
 
     public AppInfo build()
     {
       return new AppInfo(this.addonName, this.addonVersion, this.application,
           this.applicationVersion, this.platform, this.platformVersion, this.locale);
     }
+
+    private String checkForCompatibleInstalledBrowser(final PackageManager packageManager)

[diegocarloslima, 2017/12/15 16:13:43]

I think that we should also consider all the alpha/beta variants packages:
com.sec.android.app.sbrowser.beta
com.yandex.browser.alpha
com.yandex.browser.beta

[jens, 2017/12/18 08:41:34]

Thats actually a good idea :)

+    {
+      StringBuilder installedCompatibleBrowser = new StringBuilder();
+      try
+      {
+        packageManager.getPackageInfo(SBROWSER_PACKAGE_NAME, 0);
+        installedCompatibleBrowser.append(SBROWSER_APP_NAME);
+      }
+      catch (PackageManager.NameNotFoundException e)
+      {
+        // nothing
+      }
+      try
+      {
+        packageManager.getPackageInfo(YANDEX_PACKAGE_NAME, 0);
+        installedCompatibleBrowser.append(YANDEX_APP_NAME);
+      }
+      catch (PackageManager.NameNotFoundException e)
+      {
+        // nothing
+      }
+      return installedCompatibleBrowser.toString();
+    }
   }
 }