diff options
| author | newt <newt@chromium.org> | 2015-03-06 10:36:01 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-03-06 18:36:59 +0000 |
| commit | 04a35e18e9bc841122ce04d26a718cd5d5f8460c (patch) | |
| tree | f901adb36298e66c2e24df770a41c37e8dc145d0 /printing | |
| parent | 11c5341dc4b04486292b474518dcfd11e7b379f0 (diff) | |
| download | chromium_src-04a35e18e9bc841122ce04d26a718cd5d5f8460c.zip chromium_src-04a35e18e9bc841122ce04d26a718cd5d5f8460c.tar.gz chromium_src-04a35e18e9bc841122ce04d26a718cd5d5f8460c.tar.bz2 | |
Address NewApi lint warnings in src/printing.
These are all false positives. We just need to mark several classes
with @TargetApi(KitKat) since printing is only used on KitKat+.
BUG=411461
Review URL: https://codereview.chromium.org/980473002
Cr-Commit-Position: refs/heads/master@{#319467}
Diffstat (limited to 'printing')
3 files changed, 10 insertions, 1 deletions
diff --git a/printing/android/java/src/org/chromium/printing/PrintDocumentAdapterWrapper.java b/printing/android/java/src/org/chromium/printing/PrintDocumentAdapterWrapper.java index 9212508..676b8d0 100644 --- a/printing/android/java/src/org/chromium/printing/PrintDocumentAdapterWrapper.java +++ b/printing/android/java/src/org/chromium/printing/PrintDocumentAdapterWrapper.java @@ -4,6 +4,8 @@ package org.chromium.printing; +import android.annotation.TargetApi; +import android.os.Build; import android.os.Bundle; import android.os.CancellationSignal; import android.os.ParcelFileDescriptor; @@ -21,6 +23,7 @@ import android.print.PrintDocumentInfo; * {@link PrintDocumentAdapter#onLayout} and {@link PrintDocumentAdapter#onWrite}. This class helps * bypassing the limitation. */ +@TargetApi(Build.VERSION_CODES.KITKAT) public class PrintDocumentAdapterWrapper extends PrintDocumentAdapter { private PdfGenerator mPdfGenerator; diff --git a/printing/android/java/src/org/chromium/printing/PrintManagerDelegateImpl.java b/printing/android/java/src/org/chromium/printing/PrintManagerDelegateImpl.java index 5fd6406..0fd13ec 100644 --- a/printing/android/java/src/org/chromium/printing/PrintManagerDelegateImpl.java +++ b/printing/android/java/src/org/chromium/printing/PrintManagerDelegateImpl.java @@ -4,7 +4,9 @@ package org.chromium.printing; +import android.annotation.TargetApi; import android.content.Context; +import android.os.Build; import android.print.PrintAttributes; import android.print.PrintDocumentAdapter; import android.print.PrintManager; @@ -12,11 +14,12 @@ import android.print.PrintManager; /** * An implementation of {@link PrintManagerDelegate} using the Android framework print manager. */ +@TargetApi(Build.VERSION_CODES.KITKAT) public class PrintManagerDelegateImpl implements PrintManagerDelegate { private final PrintManager mPrintManager; public PrintManagerDelegateImpl(Context context) { - mPrintManager = (PrintManager) context.getSystemService(Context.PRINT_SERVICE); + mPrintManager = (PrintManager) context.getSystemService(Context.PRINT_SERVICE); } @Override diff --git a/printing/android/java/src/org/chromium/printing/PrintingControllerImpl.java b/printing/android/java/src/org/chromium/printing/PrintingControllerImpl.java index 55915e3..3d8bbb0 100644 --- a/printing/android/java/src/org/chromium/printing/PrintingControllerImpl.java +++ b/printing/android/java/src/org/chromium/printing/PrintingControllerImpl.java @@ -4,6 +4,8 @@ package org.chromium.printing; +import android.annotation.TargetApi; +import android.os.Build; import android.os.Bundle; import android.os.CancellationSignal; import android.os.ParcelFileDescriptor; @@ -27,6 +29,7 @@ import java.util.Iterator; * print button. The singleton object lives in UI thread. Interaction with the native side is * carried through PrintingContext class. */ +@TargetApi(Build.VERSION_CODES.KITKAT) public class PrintingControllerImpl implements PrintingController, PdfGenerator { private static final String LOG_TAG = "PrintingControllerImpl"; |