Remove never_lint from WebView

android_webview/glue/java/src/com/android/webview/chromium/WebViewContentsClientAdapter.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 com.android.webview.chromium;
 
+import android.annotation.SuppressLint;
 import android.content.Context;
 import android.graphics.Bitmap;
 import android.graphics.BitmapFactory;
 import android.graphics.Canvas;
 import android.graphics.Color;
 import android.graphics.Picture;
 import android.net.Uri;
 import android.net.http.SslError;
 import android.os.Build;
 import android.os.Handler;
@@ skipping 55 matching lines @@
  *
  * A {@link ContentViewClient} may be shared between multiple {@link ContentView}s,
  * and hence multiple WebViews. Many WebViewClient methods pass the source
  * WebView as an argument. This means that we either need to pass the
  * corresponding ContentView to the corresponding ContentViewClient methods,
  * or use an instance of ContentViewClientAdapter per WebViewChromium, to
  * allow the source WebView to be injected by ContentViewClientAdapter. We
  * choose the latter, because it makes for a cleaner design.
  */
 @SuppressWarnings("deprecation")
+@SuppressLint("NewApi")

[newt (away), 2015/10/13 20:41:38]

same deal here

[michaelbai, 2015/10/13 21:40:15]

Changed to use TargetApi, but it is a little bit over-skill to make it more
specific, furthermore, we already check the build version before using M APIs.

 public class WebViewContentsClientAdapter extends AwContentsClient {
     // TAG is chosen for consistency with classic webview tracing.
     private static final String TAG = "WebViewCallback";
     // Enables API callback tracing
     private static final boolean TRACE = false;
     // Default WebViewClient used to avoid null checks.
     private static WebViewClient sNullWebViewClient = new WebViewClient();
     // The WebView instance that this adapter is serving.
     private final WebView mWebView;
     // The Context to use. This is different from mWebView.getContext(), which should not be used.
@@ skipping 1181 matching lines @@
                 mAwPermissionRequest.deny();
             }
         }
 
         @Override
         public void deny() {
             mAwPermissionRequest.deny();
         }
     }
 }