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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/preferences/autofill/AutofillProfileBridge.java

Issue 2800013004: Revert of Use dropdown list for admin areas in pr form. (Closed)
Patch Set: Created 3 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/preferences/autofill/AutofillProfileBridge.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/preferences/autofill/AutofillProfileBridge.java b/chrome/android/java/src/org/chromium/chrome/browser/preferences/autofill/AutofillProfileBridge.java
index a5bc8eb534f070ba275cd5e72b3c0951e2d54524..7f0f63db0c8742eaadca5814a47eae45fbf82941 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/preferences/autofill/AutofillProfileBridge.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/preferences/autofill/AutofillProfileBridge.java
@@ -97,29 +97,8 @@
return result;
}
});
+
return countries;
- }
-
- /** @return The list of admin areas sorted by their localized display names. */
- public static List<DropdownKeyValue> getAdminAreaDropdownList(String[] keys) {
- List<DropdownKeyValue> adminAreas = new ArrayList<>();
-
- for (int i = 0; i < keys.length; ++i) {
- // TODO (parastoog): show names, save keys. @crbug.com/691643
- adminAreas.add(new DropdownKeyValue(keys[i], keys[i]));
- }
-
- final Collator collator = Collator.getInstance(Locale.getDefault());
- collator.setStrength(Collator.PRIMARY);
- Collections.sort(adminAreas, new Comparator<DropdownKeyValue>() {
- @Override
- public int compare(DropdownKeyValue lhs, DropdownKeyValue rhs) {
- // Sorted according to the admin area values, such as Quebec,
- // rather than the admin area keys, such as QC.
- return collator.compare(lhs.getValue(), rhs.getValue());
- }
- });
- return adminAreas;
}
/** @return The list of required fields. COUNTRY is always included. RECIPIENT often omitted. */

Powered by Google App Engine
This is Rietveld 408576698