Remove never_lint from WebView

android_webview/glue/java/src/com/android/webview/chromium/WebViewChromium.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.app.assist.AssistStructure.ViewNode;
 import android.content.Context;
 import android.content.res.Configuration;
 import android.graphics.Bitmap;
 import android.graphics.Canvas;
 import android.graphics.Paint;
 import android.graphics.Picture;
 import android.graphics.Rect;
 import android.graphics.drawable.Drawable;
 import android.net.Uri;
@@ skipping 53 matching lines @@
 
 /**
  * This class is the delegate to which WebViewProxy forwards all API calls.
  *
  * Most of the actual functionality is implemented by AwContents (or ContentViewCore within
  * it). This class also contains WebView-specific APIs that require the creation of other
  * adapters (otherwise org.chromium.content would depend on the webview.chromium package)
  * and a small set of no-op deprecated APIs.
  */
 @SuppressWarnings("deprecation")
+@SuppressLint("NewApi")

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

What NewApi warnings are you suppressing? NewApi is one of the most helpful lint
warnings, so a blanket suppression of all NewApi warnings in a huge class is far
from ideal. Here are several better approaches:
 1. Use @TargetApi as explained in my comment in WebViewDelegateFactory.java,
and/or
 2. Apply @TargetApi locally to the methods that call the new APIs instead of to
the entire class, and/or
 3. Extract the code that uses the new APIs into an inner class and add
@TargetApi to that inner class.

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

Done.

 class WebViewChromium implements WebViewProvider, WebViewProvider.ScrollDelegate,
                                  WebViewProvider.ViewDelegate, SmartClipProvider {
     private class WebViewChromiumRunQueue {
         public WebViewChromiumRunQueue() {
             mQueue = new ConcurrentLinkedQueue<Runnable>();
         }
 
         public void addTask(Runnable task) {
             mQueue.add(task);
             if (mFactory.hasStarted()) {
@@ skipping 1615 matching lines @@
                 public void run() {
                     onWindowVisibilityChanged(visibility);
                 }
             });
             return;
         }
         mAwContents.onWindowVisibilityChanged(visibility);
     }
 
     @Override
+    @SuppressLint("DrawAllocation")
     public void onDraw(final Canvas canvas) {
         mFactory.startYourEngines(true);
         if (checkNeedsPost()) {
             runVoidTaskOnUiThreadBlocking(new Runnable() {
                 @Override
                 public void run() {
                     onDraw(canvas);
                 }
             });
             return;
@@ skipping 276 matching lines @@
                     return requestFocus(direction, previouslyFocusedRect);
                 }
             });
             return ret;
         }
         mAwContents.requestFocus();
         return mWebViewPrivate.super_requestFocus(direction, previouslyFocusedRect);
     }
 
     @Override
+    @SuppressLint("DrawAllocation")
     public void onMeasure(final int widthMeasureSpec, final int heightMeasureSpec) {
         mFactory.startYourEngines(false);
         if (checkNeedsPost()) {
             runVoidTaskOnUiThreadBlocking(new Runnable() {
                 @Override
                 public void run() {
                     onMeasure(widthMeasureSpec, heightMeasureSpec);
                 }
             });
             return;
@@ skipping 267 matching lines @@
         mAwContents.extractSmartClipData(x, y, width, height);
     }
 
     // Implements SmartClipProvider
     @Override
     public void setSmartClipResultHandler(final Handler resultHandler) {
         checkThread();
         mAwContents.setSmartClipResultHandler(resultHandler);
     }
 }