Usage of new API, cleanups (reduced)

src/org/adblockplus/android/AdblockPlus.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 314 matching lines @@
    */
   public String[] getSelectorsForDomain(final String domain)
   {
     /* We need to ignore element hiding rules here to work around two bugs:
      * 1. CSS is being injected even when there's an exception rule with $elemhide
      * 2. The injected CSS causes blank pages in Chrome for Android
      *
      * Starting with 1.1.2, we ignored element hiding rules after download anyway, to keep the
      * memory usage down. Doing this with libadblockplus is trickier, but would be the clean
      * solution. */
-
     return null;
 /*
     if (!filteringEnabled)
       return null;
 
     return abpEngine.getSelectorsForDomain(domain);
- */
+*/
   }
 
   /**
    * Checks if filters match request parameters.
    *
    * @param url
    *          Request URL
    * @param query
    *          Request query string
    * @param referrer
@@ skipping 71 matching lines @@
   /**
    * Enables or disables filtering.
    */
   public void setFilteringEnabled(final boolean enable)
   {
     filteringEnabled = enable;
     sendBroadcast(new Intent(BROADCAST_FILTERING_CHANGE).putExtra("enabled", filteringEnabled));
   }
 
   /**
-   * Starts ABP engine. It also initiates subscription refresh if it is enabled in user settings.

[Felix Dahlke, 2014/04/28 07:29:01]

This unrelated change seems fine. Looks like 100 columns, and you're touching
the function anyway.

[Felix Dahlke, 2014/04/28 10:09:41]

I actually meant it's OK, because it's still 100 columns :) But I don't mind
either way.

+   * Starts ABP engine. It also initiates subscription refresh if it is enabled
+   * in user settings.
    */
   public void startEngine()
   {
     if (abpEngine == null)
     {
       final File basePath = getFilesDir();
       abpEngine = ABPEngine.create(AdblockPlus.getApplication(), ABPEngine.generateAppInfo(this), basePath.getAbsolutePath());
     }
   }
 
@@ skipping 91 matching lines @@
       Log.e(TAG, e.getMessage(), e);
     }
 
     // Set crash handler
     Thread.setDefaultUncaughtExceptionHandler(new CrashHandler(this));
 
     // Initiate update check
     scheduleUpdater(0);
   }
 }