New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 762607 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 2017
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

AwVariationsConfigurationService.class Call requires API level 24 (current min is 21): `android.app.job.JobScheduler#getPendingJob`: NewApi

Project Member Reported by wnwen@chromium.org, Sep 6 2017

Issue description

org/chromium/android_webview/variations/AwVariationsConfigurationService.class Call requires API level 24 (current min is 21): `android.app.job.JobScheduler#getPendingJob`: NewApi [warning]

Introduced https://chromium.googlesource.com/chromium/src/+/48cabf586057327436191edba09869ec0393a8e8

 

Comment 1 by wnwen@chromium.org, Sep 6 2017

Owner: gsennton@chromium.org
How do I get that lint warning? I can't see it when just compiling WebView.

Comment 3 by wnwen@chromium.org, Sep 6 2017

It will be visible once this lands and you remove the suppression: https://chromium-review.googlesource.com/c/chromium/src/+/652964

For now removing it from suppressions.xml will cause a lot of breakages (as seen in that CL).
Project Member

Comment 4 by bugdroid1@chromium.org, Sep 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e641faa1afd8b63f8ea1b67bac2527cd38def3ed

commit e641faa1afd8b63f8ea1b67bac2527cd38def3ed
Author: Gustav Sennton <gsennton@google.com>
Date: Wed Sep 13 17:20:34 2017

[WebView] Replace use of JobScheduler.getPendingJob (API level 24).

AwVariationsConfigurationService uses JobScheduler.getPendingJob to
check whether a specific job is pending. This method was introduced in
Android version 24, but WebView supports version 21+, so replace the use
of JobScheduler.getPendingJob with JobScheduler.getAllPendingJobs instead.

Bug:  762607 
Change-Id: Ia05815e73690ec0deff78d8d78b04d63376ec255
Reviewed-on: https://chromium-review.googlesource.com/664722
Commit-Queue: Gustav Sennton <gsennton@chromium.org>
Reviewed-by: Tobias Sargeant <tobiasjs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501672}
[modify] https://crrev.com/e641faa1afd8b63f8ea1b67bac2527cd38def3ed/android_webview/java/src/org/chromium/android_webview/variations/AwVariationsConfigurationService.java

Status: Fixed (was: Assigned)

Sign in to add a comment