Usage of new API, cleanups (reduced)

src/org/adblockplus/android/ProxyService.java

 /*
  * This file is part of Adblock Plus <http://adblockplus.org/>,
  * Copyright (C) 2006-2014 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 41 matching lines @@
 import android.os.StrictMode;
 import android.preference.PreferenceManager;
 import android.support.v4.app.NotificationCompat;
 import android.util.Log;
 
 import com.stericson.RootTools.RootTools;
 import com.stericson.RootTools.RootToolsException;
 
 public class ProxyService extends Service implements OnSharedPreferenceChangeListener
 {
+  private static final String LOCALHOST = "127.0.0.1";
+  /**
+   * Indicates that system supports native proxy configuration.
+   */
+  public static final boolean NATIVE_PROXY_SUPPORTED = Build.VERSION.SDK_INT >= 12; // Honeycomb 3.1
+
   static
   {
     RootTools.debugMode = false;
   }
 
   private static final String TAG = Utils.getTag(ProxyService.class);
-
-  private static final String LOCALHOST = "127.0.0.1";
-  /**
-   * Indicates that system supports native proxy configuration.
-   */
   private static final boolean logRequests = false;
 
-  public static final boolean NATIVE_PROXY_SUPPORTED = Build.VERSION.SDK_INT >= 12; // Honeycomb
-  private static final int NOTRAFFIC_NOTIFICATION_ID = R.string.app_name + 3;
-  static final int ONGOING_NOTIFICATION_ID = R.string.app_name;
-
-  private static final long POSITION_RIGHT = Build.VERSION.SDK_INT >= Build.VERSION_CODES.GINGERBREAD ? Long.MIN_VALUE : Long.MAX_VALUE;
+  // Do not use 8080 because it is a "dirty" port, Android uses it if something goes wrong
+  // first element is reserved for previously used port
+  private static final int[] portVariants = new int[] {-1, 2020, 3030, 4040, 5050, 6060, 7070, 9090, 1234, 12345, 4321, 0};
 
   private static final int DEFAULT_TIMEOUT = 3000;
   private static final int NO_TRAFFIC_TIMEOUT = 5 * 60 * 1000; // 5 minutes
 
-  // Do not use 8080 because it is a "dirty" port, Android uses it if something
-  // goes wrong
-  // first element is reserved for previously used port
-  private static final int[] portVariants = new int[] { -1, 2020, 3030, 4040, 5050, 6060, 7070, 9090, 1234, 12345, 4321, 0 };
+  static final int ONGOING_NOTIFICATION_ID = R.string.app_name;
+  private static final long POSITION_RIGHT = Build.VERSION.SDK_INT >= Build.VERSION_CODES.GINGERBREAD ? Long.MIN_VALUE : Long.MAX_VALUE;
+  private static final int NOTRAFFIC_NOTIFICATION_ID = R.string.app_name + 3;
 
   /**
    * Broadcasted when service starts or stops.
    */
   public static final String BROADCAST_STATE_CHANGED = "org.adblockplus.android.service.state";
   /**
    * Broadcasted if proxy fails to start.
    */
   public static final String BROADCAST_PROXY_FAILED = "org.adblockplus.android.proxy.failure";
 
@@ skipping 23 matching lines @@
   private String iptables = null;
 
   @SuppressLint("NewApi")
   @Override
   public void onCreate()
   {
     super.onCreate();
 
     if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.GINGERBREAD)
     {
-      // Proxy is running in separate thread, it's just some resolution request
-      // during initialization.
-      // Not worth spawning a separate thread for
+      // Proxy is running in separate thread, it's just some resolution request during initialization.
+      // Not worth spawning a separate thread for this.
       final StrictMode.ThreadPolicy policy = new StrictMode.ThreadPolicy.Builder().permitNetwork().build();
       StrictMode.setThreadPolicy(policy);
     }
 
     // Get port for local proxy
     final SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(this);
     final Resources resources = getResources();
 
     // Try to read user proxy settings
     String proxyHost = null;
@@ skipping 66 matching lines @@
       // Try to set native proxy
       nativeProxyAutoConfigured = ProxySettings.setConnectionProxy(getApplicationContext(), LOCALHOST, port, "");
 
       if (NATIVE_PROXY_SUPPORTED)
       {
         registerReceiver(connectionReceiver, new IntentFilter(ConnectivityManager.CONNECTIVITY_ACTION));
         registerReceiver(connectionReceiver, new IntentFilter(Proxy.PROXY_CHANGE_ACTION));
       }
     }
 
-    // Save current native proxy situation. The service is always started on the
-    // first run so
+    // Save current native proxy situation. The service is always started on the first run so
     // we will always have a correct value from the box
     final SharedPreferences.Editor editor = prefs.edit();
     editor.putBoolean(getString(R.string.pref_proxyautoconfigured), transparent || nativeProxyAutoConfigured);
     editor.commit();
 
     registerReceiver(proxyReceiver, new IntentFilter(ProxyService.BROADCAST_PROXY_FAILED));
     registerReceiver(filterReceiver, new IntentFilter(AdblockPlus.BROADCAST_FILTERING_CHANGE));
     registerReceiver(filterReceiver, new IntentFilter(AdblockPlus.BROADCAST_FILTER_MATCHES));
 
     // Start proxy
     if (proxy == null)
     {
-      // Select available port and bind to it, use previously selected port by
-      // default
+      // Select available port and bind to it, use previously selected port by default
       portVariants[0] = prefs.getInt(getString(R.string.pref_lastport), -1);
       ServerSocket listen = null;
       String msg = null;
       for (final int p : portVariants)
       {
         if (p < 0)
           continue;
         try
         {
           // Fix for #232, bind proxy socket to loopback only
@@ skipping 71 matching lines @@
         Log.e(TAG, "Failed to initialize iptables", e);
       }
     }
 
     prefs.registerOnSharedPreferenceChangeListener(this);
 
     // Lock service
     hideIcon = prefs.getBoolean(getString(R.string.pref_hideicon), resources.getBoolean(R.bool.def_hideicon));
     startForeground(ONGOING_NOTIFICATION_ID, getNotification());
 
-    // If automatic setting of proxy was blocked, check if user has set it
-    // manually
+    // If automatic setting of proxy was blocked, check if user has set it manually
     final boolean manual = isManual();
     if (manual && NATIVE_PROXY_SUPPORTED)
     {
       final ConnectivityManager connectivityManager = (ConnectivityManager) getSystemService(Context.CONNECTIVITY_SERVICE);
       updateNoTrafficCheck(connectivityManager);
     }
 
     sendStateChangedBroadcast();
     Log.i(TAG, "Service started");
   }
@@ skipping 49 matching lines @@
     if (proxy != null)
       proxy.close();
 
     // Release service lock
     stopForeground(true);
 
     Log.i(TAG, "Service stopped");
   }
 
   /**
-   * Restores system proxy settings via native call on Android 3.1+ devices using Java reflection.
+   * Restores system proxy settings via native call on Android 3.1+ devices
+   * using Java reflection.
    */
   private void clearConnectionProxy()
   {
     final String proxyHost = proxyConfiguration.getProperty("adblock.proxyHost");
     final String proxyPort = proxyConfiguration.getProperty("adblock.proxyPort");
     final String proxyExcl = proxyConfiguration.getProperty("adblock.proxyExcl");
     int port = 0;
     try
     {
       if (proxyHost != null)
@@ skipping 136 matching lines @@
   {
     return notrafficHandler != null;
   }
 
   /**
    * Checks if specified host is local.
    */
   private static final boolean isLocalHost(final String host)
   {
     if (host == null)
-    {
       return false;
-    }
 
     try
     {
       if (host.equalsIgnoreCase("localhost"))
         return true;
 
       final String className = "android.net.NetworkUtils";
       final Class<?> c = Class.forName(className);
       /*
        * InetAddress address = NetworkUtils.numericToInetAddress(host);
@@ skipping 68 matching lines @@
       return RootTools.sendShell(command, DEFAULT_TIMEOUT);
     }
     catch (final Exception e)
     {
       Log.e(TAG, "Failed to get iptables configuration", e);
       return null;
     }
   }
 
   /**
-   * Raises or removes no traffic notification based on current link proxy settings
+   * Raises or removes no traffic notification based on current link proxy
+   * settings
    */
   private void updateNoTrafficCheck(final ConnectivityManager connectivityManager)
   {
     try
     {
       final Object pp = ProxySettings.getActiveLinkProxy(connectivityManager);
       final String[] userProxy = ProxySettings.getUserProxy(pp);
       if (userProxy != null)
         Log.i(TAG, "Proxy settings: " + userProxy[0] + ":" + userProxy[1] + "(" + userProxy[2] + ")");
       updateNoTrafficCheck(userProxy);
@@ skipping 55 matching lines @@
     if (nativeProxyAutoConfigured || proxyManualyConfigured)
       msgId = filtering ? R.string.notif_wifi : R.string.notif_wifi_nofiltering;
     if (transparent)
       msgId = R.string.notif_all;
 
     final NotificationCompat.Builder builder = new NotificationCompat.Builder(this);
     if (hideIcon && msgId != R.string.notif_waiting)
     {
       builder.setWhen(POSITION_RIGHT);
       builder.setSmallIcon(R.drawable.transparent);
-      // builder.setContent(new RemoteViews(getPackageName(),
-      // R.layout.notif_hidden));
+      // builder.setContent(new RemoteViews(getPackageName(), R.layout.notif_hidden));
     }
     else
     {
       builder.setWhen(0);
       builder.setSmallIcon(R.drawable.ic_stat_blocking);
     }
     final PendingIntent contentIntent = PendingIntent.getActivity(this, 0, new Intent(this, Preferences.class).addFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP | Intent.FLAG_ACTIVITY_NEW_TASK), 0);
     builder.setContentIntent(contentIntent);
     builder.setContentTitle(getText(R.string.app_name));
     builder.setContentText(getString(msgId, port));
@@ skipping 30 matching lines @@
     }
   }
 
   @Override
   public IBinder onBind(final Intent intent)
   {
     return binder;
   }
 
   /**
-   * Executed if no traffic is detected after a period of time. Notifies user about possible
-   * configuration problems.
+   * Executed if no traffic is detected after a period of time. Notifies user
+   * about possible configuration problems.
    */
   private final Runnable noTraffic = new Runnable()
   {
     @Override
     public void run()
     {
-      // It's weird but notrafficHandler.removeCallbacks(noTraffic) does not
-      // remove this callback
+      // It's weird but notrafficHandler.removeCallbacks(noTraffic) does not remove this callback
       if (notrafficHandler == null)
-      {
         return;
-      }
       // Show warning notification
       final NotificationCompat.Builder builder = new NotificationCompat.Builder(ProxyService.this);
       builder.setSmallIcon(R.drawable.ic_stat_warning);
       builder.setWhen(System.currentTimeMillis());
       builder.setAutoCancel(true);
       final Intent intent = new Intent(ProxyService.this, ConfigurationActivity.class).addFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
       intent.putExtra("port", port);
       final PendingIntent contentIntent = PendingIntent.getActivity(ProxyService.this, 0, intent, PendingIntent.FLAG_UPDATE_CURRENT);
       builder.setContentIntent(contentIntent);
       builder.setContentTitle(getText(R.string.app_name));
@@ skipping 40 matching lines @@
     public void onReceive(final Context context, final Intent intent)
     {
       if (intent.getAction().equals(ProxyService.BROADCAST_PROXY_FAILED))
       {
         stopSelf();
       }
     }
   };
 
   /**
-   * Monitors system network connection settings changes and updates proxy settings accordingly.
+   * Monitors system network connection settings changes and updates proxy
+   * settings accordingly.
    */
   private final BroadcastReceiver connectionReceiver = new BroadcastReceiver()
   {
     @Override
     public void onReceive(final Context ctx, final Intent intent)
     {
       final String action = intent.getAction();
       Log.i(TAG, "Action: " + action);
       // Connectivity change
       if (ConnectivityManager.CONNECTIVITY_ACTION.equals(action))
@@ skipping 75 matching lines @@
     @Override
     public void log(final int level, final Object obj, final String message)
     {
       if (level <= logLevel)
       {
         Log.println(7 - level, obj != null ? obj.toString() : TAG, message);
       }
     }
   }
 }