Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(565)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/ItemChooserDialog.java

Issue 1842223003: Remove outdated devices from Android device chooser (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed unit test for ItemChooserDialog Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 fd9b72793e92be73ada4abba1b5e312403f35658..1069a92b55d3e8fd16d0a7652ba2ed64c61b5a45 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ItemChooserDialog.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ItemChooserDialog.java
@@ -392,6 +392,7 @@ public class ItemChooserDialog {
* @param id The id of the item to affect.
* @param enabled Whether the item should be enabled or not.
*/
+ @VisibleForTesting
scheib 2016/04/07 22:05:02 Is this needed for this patch?
perja 2016/04/08 07:47:35 I'm not 100% sure about this one. Seems that only
perja 2016/06/24 14:15:28 I have reverted the @VisibleForTesting line now.
public void setEnabled(String id, boolean enabled) {
mItemAdapter.setEnabled(id, enabled);
}

Powered by Google App Engine
This is Rietveld 408576698