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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/autofill/AutofillTestHelper.java

Issue 2033323003: Revert of [Autofill] Sort profiles and credit cards by frecency in PaymentRequest. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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/javatests/src/org/chromium/chrome/browser/autofill/AutofillTestHelper.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/autofill/AutofillTestHelper.java b/chrome/android/javatests/src/org/chromium/chrome/browser/autofill/AutofillTestHelper.java
index 173fb565e50a8b01a6d0da5afe1fc824e4dc335a..20467b77401de6cd607e7c92c81633012ed8abfd 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/autofill/AutofillTestHelper.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/autofill/AutofillTestHelper.java
@@ -34,30 +34,22 @@
});
}
- List<AutofillProfile> getProfilesToSuggest() throws ExecutionException {
+ List<AutofillProfile> getProfiles() throws ExecutionException {
return ThreadUtils.runOnUiThreadBlocking(new Callable<List<AutofillProfile>>() {
@Override
public List<AutofillProfile> call() {
- return PersonalDataManager.getInstance().getProfilesToSuggest();
+ return PersonalDataManager.getInstance().getProfiles();
}
});
}
- List<AutofillProfile> getProfilesForSettings() throws ExecutionException {
- return ThreadUtils.runOnUiThreadBlocking(new Callable<List<AutofillProfile>>() {
+ int getNumberOfProfiles() throws ExecutionException {
+ return ThreadUtils.runOnUiThreadBlocking(new Callable<Integer>() {
@Override
- public List<AutofillProfile> call() {
- return PersonalDataManager.getInstance().getProfilesForSettings();
+ public Integer call() {
+ return PersonalDataManager.getInstance().getProfiles().size();
}
- });
- }
-
- int getNumberOfProfilesToSuggest() throws ExecutionException {
- return getProfilesToSuggest().size();
- }
-
- int getNumberOfProfilesForSettings() throws ExecutionException {
- return getProfilesForSettings().size();
+ }).intValue();
}
public String setProfile(final AutofillProfile profile) throws InterruptedException,
@@ -91,30 +83,13 @@
});
}
- List<CreditCard> getCreditCardsToSuggest() throws ExecutionException {
- return ThreadUtils.runOnUiThreadBlocking(new Callable<List<CreditCard>>() {
+ int getNumberOfCreditCards() throws ExecutionException {
+ return ThreadUtils.runOnUiThreadBlocking(new Callable<Integer>() {
@Override
- public List<CreditCard> call() {
- return PersonalDataManager.getInstance().getCreditCardsToSuggest();
+ public Integer call() {
+ return PersonalDataManager.getInstance().getCreditCards().size();
}
- });
- }
-
- List<CreditCard> getCreditCardsForSettings() throws ExecutionException {
- return ThreadUtils.runOnUiThreadBlocking(new Callable<List<CreditCard>>() {
- @Override
- public List<CreditCard> call() {
- return PersonalDataManager.getInstance().getCreditCardsForSettings();
- }
- });
- }
-
- int getNumberOfCreditCardsToSuggest() throws ExecutionException {
- return getCreditCardsToSuggest().size();
- }
-
- int getNumberOfCreditCardsForSettings() throws ExecutionException {
- return getCreditCardsForSettings().size();
+ }).intValue();
}
public String setCreditCard(final CreditCard card) throws InterruptedException,
@@ -129,63 +104,11 @@
return guid;
}
- public void addServerCreditCard(final CreditCard card)
- throws InterruptedException, ExecutionException {
- ThreadUtils.runOnUiThreadBlocking(new Runnable() {
- @Override
- public void run() {
- PersonalDataManager.getInstance().addServerCreditCardForTest(card);
- }
- });
- waitForDataChanged();
- }
-
void deleteCreditCard(final String guid) throws InterruptedException {
ThreadUtils.runOnUiThreadBlocking(new Runnable() {
@Override
public void run() {
PersonalDataManager.getInstance().deleteCreditCard(guid);
- }
- });
- waitForDataChanged();
- }
-
- /**
- * Sets the use |count| and use |date| of the test profile associated with the |guid|.
- * @param guid The GUID of the profile to modify.
- * @param count The use count to assign to the profile. It should be non-negative.
- * @param date The use date to assign to the profile. It represents an absolute point in
- * coordinated universal time (UTC) represented as microseconds since the Windows
- * epoch. For more details see the comment header in time.h. It should always be a
- * positive number.
- */
- void setProfileUseStatsForTesting(final String guid, final int count, final long date)
- throws InterruptedException {
- ThreadUtils.runOnUiThreadBlocking(new Runnable() {
- @Override
- public void run() {
- PersonalDataManager.getInstance().setProfileUseStatsForTesting(guid, count, date);
- }
- });
- waitForDataChanged();
- }
-
- /**
- * Sets the use |count| and use |date| of the test credit card associated with the |guid|.
- * @param guid The GUID of the credit card to modify.
- * @param count The use count to assign to the credit card. It should be non-negative.
- * @param date The use date to assign to the credit card. It represents an absolute point in
- * coordinated universal time (UTC) represented as microseconds since the Windows
- * epoch. For more details see the comment header in time.h. It should always be a
- * positive number.
- */
- void setCreditCardUseStatsForTesting(final String guid, final int count, final long date)
- throws InterruptedException {
- ThreadUtils.runOnUiThreadBlocking(new Runnable() {
- @Override
- public void run() {
- PersonalDataManager.getInstance().setCreditCardUseStatsForTesting(
- guid, count, date);
}
});
waitForDataChanged();

Powered by Google App Engine
This is Rietveld 408576698