DO NO SUMMIT: bluetooth: Use a LevelListDrawable instead of an array of drawables

chrome/android/java/src/org/chromium/chrome/browser/ItemChooserDialog.java

Index: chrome/android/java/src/org/chromium/chrome/browser/ItemChooserDialog.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ItemChooserDialog.java b/chrome/android/java/src/org/chromium/chrome/browser/ItemChooserDialog.java
index 5318960d85db9ea3ce46fba5c58895868eea2bd8..1629d2b2de5c632e5baa38dbf2c723dd73adf9ca 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ItemChooserDialog.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ItemChooserDialog.java
@@ -354,7 +354,14 @@ public class ItemChooserDialog {
                 ItemChooserRow item = getItem(position);
                 if (item.mIcon != null) {
                     row.mImageView.setContentDescription(item.mIconDescription);
+
+                    // Save the icon's level since setImageLevel could
+                    // replace the drawable's level.
+                    int drawableLevel = item.mIcon.getLevel();

[ortuno, 2017/04/18 06:57:07]

This seems pretty leaky.

[David Trainor- moved to gerrit, 2017/04/19 21:32:07]

Should mIconLevel or "mSignalLevel" or something be a top level item concept? 
That way if it changes you just know to refresh and can always push here instead
of pulling the level first?

[ortuno, 2017/04/19 23:48:02]

Oh right, we can save it as part of ItemChooserRow and then set it here. Done.

+
                     row.mImageView.setImageDrawable(item.mIcon);
+
+                    row.mImageView.setImageLevel(drawableLevel);
                     row.mImageView.setVisibility(View.VISIBLE);
                 } else {
                     row.mImageView.setVisibility(View.INVISIBLE);