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

Issue 725356 link

Starred by 4 users

Issue metadata

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

Blocking:
issue 724622



Sign in to add a comment

Update incremental install to not use codename for Android O

Project Member Reported by nyquist@chromium.org, May 23 2017

Issue description

Chrome Version: c90c34417bb295752d1e7fa96ba5d41c106917fd
OS: Android

The incremental install does not use the correct checks for Android O, but still uses a codename.

It's in org.chromium.incrementalinstall.ClassLoaderPatcher, in the methods makeNativePathElements(...) and addDexElements(...).

See these two places:
-
 https://chromium.googlesource.com/chromium/src/+/c90c34417bb295752d1e7fa96ba5d41c106917fd/build/android/incremental_install/java/org/chromium/incrementalinstall/ClassLoaderPatcher.java#224
-
 https://chromium.googlesource.com/chromium/src/+/c90c34417bb295752d1e7fa96ba5d41c106917fd/build/android/incremental_install/java/org/chromium/incrementalinstall/ClassLoaderPatcher.java#256

For now, since this is in //build, and we don't have the latest SDK available, I suggest copy-pasting the implementation from BuildInfo#isAtLeastO() in base:
Build.VERSION.SDK_INT >= 26

See: https://chromium.googlesource.com/chromium/src/+/c90c34417bb295752d1e7fa96ba5d41c106917fd/base/android/java/src/org/chromium/base/BuildInfo.java#171

 

Comment 1 by torne@chromium.org, May 23 2017

Blocking: 724622
Cc: peconn@chromium.org

Comment 4 by zpeng@chromium.org, Jun 5 2017

Owner: zpeng@chromium.org
Status: Fixed (was: Untriaged)
Cc: jbudorick@chromium.org agrieve@chromium.org
 Issue 729541  has been merged into this issue.

Sign in to add a comment