New issue
Advanced search Search tips

Issue 635179 link

Starred by 2 users
Status: Fixed
Owner:
estevenson@chromium.org
Closed: Aug 2016
Cc:
agrieve@chromium.org
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug

Blocking:
issue 607761



Sign in to add a comment

Clean up UseSparseArrays lint warning

Project Member Reported by estevenson@chromium.org, Aug 5 2016 
A UseSparseArrays lint warning is generated due to the use of SparseArray<Boolean> when SparseBooleanArray is available.

/tmp/tmprtJeEe/SRC_ROOT1/chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabPersistentStore.java:1270 Use `new SparseBooleanArray(...)` instead for better performance: UseSparseArrays [warning]
            mOtherTabIds = new SparseArray<Boolean>();
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 8 2016 

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

commit 5cefdd51a8a172292fbae98ef5d9efbe4563d4d5
Author: estevenson <estevenson@chromium.org>
Date: Mon Aug 08 14:14:50 2016

Use SparseBooleanArray to get rid of lint warning.

The use of SparseArray causes a UseSparseArrays lint
warning.

BUG= 635179 

Review-Url: https://codereview.chromium.org/2222683002
Cr-Commit-Position: refs/heads/master@{#410347}

[modify] https://crrev.com/5cefdd51a8a172292fbae98ef5d9efbe4563d4d5/chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabPersistentStore.java

Comment 2 by estevenson@chromium.org, Aug 8 2016

Owner: estevenson@chromium.org
Status: Fixed (was: Unconfirmed)
►

Sign in to add a comment