Update to newer Android Lint and suppress new Lint errors

base/android/java/src/org/chromium/base/ApplicationStatus.java

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.base;
 
 import android.annotation.SuppressLint;
 import android.app.Activity;
 import android.app.Application;
 import android.app.Application.ActivityLifecycleCallbacks;
@@ skipping 43 matching lines @@
         public ObserverList<ActivityStateListener> getListeners() {
             return mListeners;
         }
     }
 
     private static Object sCachedApplicationStateLock = new Object();
     @ApplicationState
     private static Integer sCachedApplicationState;
 
     /** Last activity that was shown (or null if none or it was destroyed). */
+    // TODO(crbug.com/635567): Fix this properly.

[Ted C, 2017/04/27 19:02:47]

no fix needed, this is cleared when applicable already.

[F, 2017/04/28 18:25:14]

Done.

+    @SuppressLint("StaticFieldLeak")
     private static Activity sActivity;
 
     /** A lazily initialized listener that forwards application state changes to native. */
     private static ApplicationStateListener sNativeApplicationStateListener;
 
     /**
      * A map of which observers listen to state changes from which {@link Activity}.
      */
     private static final Map<Activity, ActivityInfo> sActivityInfo =
             new ConcurrentHashMap<Activity, ActivityInfo>();
@@ skipping 276 matching lines @@
     }
 
     /**
      * Registers the given listener to receive state changes for {@code activity}.  After a call to
      * {@link ActivityStateListener#onActivityStateChange(Activity, int)} with
      * {@link ActivityState#DESTROYED} all listeners associated with that particular
      * {@link Activity} are removed.
      * @param listener Listener to receive state changes.
      * @param activity Activity to track or {@code null} to track all activities.
      */
+    // TODO(crbug.com/635567): Fix this properly.

[Ted C, 2017/04/27 19:02:47]

fix what?  activity.isDestroyed() is available in O and guarded by that check,
so I don't think anything needs to be fixed here.

[F, 2017/04/28 18:25:14]

Done.

     @SuppressLint("NewApi")
     public static void registerStateListenerForActivity(ActivityStateListener listener,
             Activity activity) {
         assert activity != null;
 
         ActivityInfo info = sActivityInfo.get(activity);
         // TODO(tedchoc): crbug/691100.  The timing of application callback lifecycles were changed
         //                in O and the activity info may need to be lazily created if the onCreate
         //                event has not yet been received.
         if (BuildInfo.isAtLeastO() && info == null && !activity.isDestroyed()) {
@@ skipping 102 matching lines @@
 
         if (hasPausedActivity) return ApplicationState.HAS_PAUSED_ACTIVITIES;
         if (hasStoppedActivity) return ApplicationState.HAS_STOPPED_ACTIVITIES;
         return ApplicationState.HAS_DESTROYED_ACTIVITIES;
     }
 
     // Called to notify the native side of state changes.
     // IMPORTANT: This is always called on the main thread!
     private static native void nativeOnApplicationStateChange(@ApplicationState int newState);
 }