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

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

Issue 2897363003: Revert "Add metrics for search engine promo and search widget" (Closed)
Patch Set: Created 3 years, 7 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/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();
}
« no previous file with comments | « chrome/android/BUILD.gn ('k') | chrome/android/java/src/org/chromium/chrome/browser/locale/LocaleManager.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698