Commit 57defc4b authored by Nate Fischer's avatar Nate Fischer Committed by Chromium LUCI CQ

Android: inline deprecated BuildInfo methods (/base/android/java/src/org/chromium/base)

No change to logic. This inlines BuildInfo.isAtLeast{Q,R} and
BuildInfo.targetsAtLeast{Q,R} (and their C++ equivalents) in
/base/android/java/src/org/chromium/base.

The methods themselves will be removed in a follow-up CL once all call
sites have been removed.

This CL was uploaded by git cl split.

R=torne@chromium.org

Bug: 1106920
Test: autoninja -C out/Default monochrome_public_apk
Change-Id: I97b895b6d9065d2d040e07528d5e2f84b4fba5b0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2575371
Auto-Submit: Nate Fischer <ntfschr@chromium.org>
Reviewed-by: default avatarRichard Coles <torne@chromium.org>
Commit-Queue: Richard Coles <torne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#834261}
parent fa284316
......@@ -206,7 +206,7 @@ public abstract class PathUtils {
public static @NonNull String getDownloadsDirectory() {
// TODO(crbug.com/508615): Move calls to getDownloadsDirectory() to background thread.
try (StrictModeContext ignored = StrictModeContext.allowDiskReads()) {
if (BuildInfo.isAtLeastQ()) {
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
// https://developer.android.com/preview/privacy/scoped-storage
// In Q+, Android has begun sandboxing external storage. Chrome may not have
// permission to write to Environment.getExternalStoragePublicDirectory(). Instead
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment