Commit b39fc4ac authored by Mia Glaese's avatar Mia Glaese Committed by Commit Bot

Change the order of added languages in LanguageAskPrompt

Bug: 895542
Change-Id: I892dc63f438e77d865813758303d7f249600f9b2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1695968Reviewed-by: default avatarYusuf Ozuysal <yusufo@chromium.org>
Reviewed-by: default avataranthonyvd <anthonyvd@chromium.org>
Commit-Queue: Mia Glaese <glamia@chromium.org>
Cr-Commit-Position: refs/heads/master@{#684018}
parent 513f26cd
...@@ -37,6 +37,7 @@ import java.util.Comparator; ...@@ -37,6 +37,7 @@ import java.util.Comparator;
import java.util.HashSet; import java.util.HashSet;
import java.util.LinkedHashSet; import java.util.LinkedHashSet;
import java.util.List; import java.util.List;
import java.util.TreeSet;
/** /**
* Implements a modal dialog that prompts the user about the languages they can read. Displayed * Implements a modal dialog that prompts the user about the languages they can read. Displayed
...@@ -264,7 +265,7 @@ public class LanguageAskPrompt implements ModalDialogProperties.Controller { ...@@ -264,7 +265,7 @@ public class LanguageAskPrompt implements ModalDialogProperties.Controller {
* mLanguagesUpdate, and by extension which languages were checked by the user in the prompt. * mLanguagesUpdate, and by extension which languages were checked by the user in the prompt.
*/ */
private void saveLanguages() { private void saveLanguages() {
HashSet<String> languagesToAdd = new HashSet<String>(mLanguagesUpdate); TreeSet<String> languagesToAdd = new TreeSet<String>(mLanguagesUpdate);
languagesToAdd.removeAll(mInitialLanguages); languagesToAdd.removeAll(mInitialLanguages);
for (String language : languagesToAdd) { for (String language : languagesToAdd) {
......
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