Bug 1131257 - Part 0: Cleanup in LocalReadingListDB.

mobile/android/base/db/LocalBrowserDB.java

@@ -63,16 +63,10 @@ import static org.mozilla.gecko.util.IOUtils.ConsumedInputStream;
 import static org.mozilla.gecko.favicons.LoadFaviconTask.DEFAULT_FAVICON_BUFFER_SIZE;
 
 public class LocalBrowserDB implements BrowserDB {
-    // Calculate these once, at initialization. isLoggable is too expensive to
-    // have in-line in each log call.
     private static final String LOGTAG = "GeckoLocalBrowserDB";
 
-    // Sentinel value used to indicate a failure to locate an ID for a default favicon.
-    private static final int FAVICON_ID_NOT_FOUND = Integer.MIN_VALUE;
-
-    // Constant used to indicate that no folder was found for particular GUID.
-    private static final long FOLDER_NOT_FOUND = -1L;
-
+    // Calculate this once, at initialization. isLoggable is too expensive to
+    // have in-line in each log call.
     private static final boolean logDebug = Log.isLoggable(LOGTAG, Log.DEBUG);
     protected static void debug(String message) {
         if (logDebug) {
@@ -80,6 +74,12 @@ public class LocalBrowserDB implements BrowserDB {
         }
     }
 
+    // Sentinel value used to indicate a failure to locate an ID for a default favicon.
+    private static final int FAVICON_ID_NOT_FOUND = Integer.MIN_VALUE;
+
+    // Constant used to indicate that no folder was found for particular GUID.
+    private static final long FOLDER_NOT_FOUND = -1L;
+
     private final String mProfile;
 
     // Map of folder GUIDs to IDs. Used for caching.