Index: chrome/android/java/src/org/chromium/chrome/browser/locale/DefaultSearchEngineDialogHelper.java |
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/locale/DefaultSearchEngineDialogHelper.java b/chrome/android/java/src/org/chromium/chrome/browser/locale/DefaultSearchEngineDialogHelper.java |
index fe0aab8df5b3b3f7b2d24f1e97666ec30b0de7e0..31c69f09aacfd594db0d6703ea38f424c9ff8307 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/locale/DefaultSearchEngineDialogHelper.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/locale/DefaultSearchEngineDialogHelper.java |
@@ -39,9 +39,9 @@ public class DefaultSearchEngineDialogHelper implements OnCheckedChangeListener, |
} |
/** Called when the search engine the user selected is confirmed to be the one they want. */ |
- protected void onUserSeachEngineChoice(List<String> keywords, String keyword) { |
+ protected void onUserSeachEngineChoice(String keyword) { |
LocaleManager.getInstance().onUserSearchEngineChoiceFromPromoDialog( |
- mDialogType, keywords, keyword); |
+ mDialogType, keyword); |
} |
} |
@@ -50,11 +50,6 @@ public class DefaultSearchEngineDialogHelper implements OnCheckedChangeListener, |
private final Button mConfirmButton; |
/** |
- * List of search engine keywords in the order shown to the user. |
- */ |
- private final List<String> mSearchEngineKeywords; |
- |
- /** |
* Keyword for the search engine that is selected in the RadioButtonLayout. |
* This value is not locked into the TemplateUrlService until the user confirms it by clicking |
* on {@link #mConfirmButton}. |
@@ -79,16 +74,16 @@ public class DefaultSearchEngineDialogHelper implements OnCheckedChangeListener, |
// Shuffle up the engines. |
List<TemplateUrl> engines = mDelegate.getSearchEngines(); |
List<CharSequence> engineNames = new ArrayList<>(); |
- mSearchEngineKeywords = new ArrayList<>(); |
+ List<String> engineKeywords = new ArrayList<>(); |
Collections.shuffle(engines); |
for (int i = 0; i < engines.size(); i++) { |
TemplateUrl engine = engines.get(i); |
engineNames.add(engine.getShortName()); |
- mSearchEngineKeywords.add(engine.getKeyword()); |
+ engineKeywords.add(engine.getKeyword()); |
} |
// Add the search engines to the dialog without any of them being selected by default. |
- controls.addOptions(engineNames, mSearchEngineKeywords); |
+ controls.addOptions(engineNames, engineKeywords); |
controls.selectChildAtIndex(RadioButtonLayout.INVALID_INDEX); |
controls.setOnCheckedChangeListener(this); |
@@ -122,8 +117,7 @@ public class DefaultSearchEngineDialogHelper implements OnCheckedChangeListener, |
return; |
} |
- mDelegate.onUserSeachEngineChoice( |
- mSearchEngineKeywords, mCurrentlySelectedKeyword.toString()); |
+ mDelegate.onUserSeachEngineChoice(mCurrentlySelectedKeyword.toString()); |
mFinishRunnable.run(); |
} |