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.TargetApi;
 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")
+// You shouldn't change TargetApi, please see how Android M API was added.
+@TargetApi(Build.VERSION_CODES.LOLLIPOP)

[newt (away), 2015/10/13 21:50:57]

Why is it overkill to use @TargetApi on individual methods? If this class is
only used on Android L+, then this is fine. Otherwise, I don't see any version
checks in this class, so how are you ensuring that this doesn't crash on pre-L
devices?

[michaelbai, 2015/10/13 22:03:48]

Yes, Webview only update down to L.

 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();
         }
     }
 }