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

Side by Side Diff: chrome/browser/autofill/android/personal_data_manager_android.cc

Issue 2120973002: [Payments] Show complete profiles first and limit to 4 suggestions. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments Created 4 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/autofill/android/personal_data_manager_android.h" 5 #include "chrome/browser/autofill/android/personal_data_manager_android.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <algorithm> 8 #include <algorithm>
9 #include <memory> 9 #include <memory>
10 #include <utility> 10 #include <utility>
(...skipping 489 matching lines...) Expand 10 before | Expand all | Expand 10 after
500 jint date) { 500 jint date) {
501 DCHECK(count >= 0 && date >= 0); 501 DCHECK(count >= 0 && date >= 0);
502 502
503 AutofillProfile* profile = personal_data_manager_->GetProfileByGUID( 503 AutofillProfile* profile = personal_data_manager_->GetProfileByGUID(
504 ConvertJavaStringToUTF8(env, jguid)); 504 ConvertJavaStringToUTF8(env, jguid));
505 if (!profile) 505 if (!profile)
506 return; 506 return;
507 507
508 profile->set_use_count(static_cast<size_t>(count)); 508 profile->set_use_count(static_cast<size_t>(count));
509 profile->set_use_date(base::Time::FromTimeT(date)); 509 profile->set_use_date(base::Time::FromTimeT(date));
510
511 personal_data_manager_->NotifyPersonalDataChangedForTests();
510 } 512 }
511 513
512 void PersonalDataManagerAndroid::SetCreditCardUseStatsForTesting( 514 void PersonalDataManagerAndroid::SetCreditCardUseStatsForTesting(
513 JNIEnv* env, 515 JNIEnv* env,
514 const JavaParamRef<jobject>& unused_obj, 516 const JavaParamRef<jobject>& unused_obj,
515 const JavaParamRef<jstring>& jguid, 517 const JavaParamRef<jstring>& jguid,
516 jint count, 518 jint count,
517 jint date) { 519 jint date) {
518 DCHECK(count >= 0 && date >= 0); 520 DCHECK(count >= 0 && date >= 0);
519 521
520 CreditCard* card = personal_data_manager_->GetCreditCardByGUID( 522 CreditCard* card = personal_data_manager_->GetCreditCardByGUID(
521 ConvertJavaStringToUTF8(env, jguid)); 523 ConvertJavaStringToUTF8(env, jguid));
522 if (!card) 524 if (!card)
523 return; 525 return;
524 526
525 card->set_use_count(static_cast<size_t>(count)); 527 card->set_use_count(static_cast<size_t>(count));
526 card->set_use_date(base::Time::FromTimeT(date)); 528 card->set_use_date(base::Time::FromTimeT(date));
529
530 personal_data_manager_->NotifyPersonalDataChangedForTests();
527 } 531 }
528 532
529 ScopedJavaLocalRef<jobjectArray> PersonalDataManagerAndroid::GetProfileGUIDs( 533 ScopedJavaLocalRef<jobjectArray> PersonalDataManagerAndroid::GetProfileGUIDs(
530 JNIEnv* env, 534 JNIEnv* env,
531 const std::vector<AutofillProfile*>& profiles) { 535 const std::vector<AutofillProfile*>& profiles) {
532 std::vector<base::string16> guids; 536 std::vector<base::string16> guids;
533 for (AutofillProfile* profile : profiles) 537 for (AutofillProfile* profile : profiles)
534 guids.push_back(base::UTF8ToUTF16(profile->guid())); 538 guids.push_back(base::UTF8ToUTF16(profile->guid()));
535 539
536 return base::android::ToJavaArrayOfStrings(env, guids); 540 return base::android::ToJavaArrayOfStrings(env, guids);
(...skipping 82 matching lines...) Expand 10 before | Expand all | Expand 10 after
619 base::android::ConvertJavaStringToUTF16(env, jcountry_name))); 623 base::android::ConvertJavaStringToUTF16(env, jcountry_name)));
620 } 624 }
621 625
622 static jlong Init(JNIEnv* env, const JavaParamRef<jobject>& obj) { 626 static jlong Init(JNIEnv* env, const JavaParamRef<jobject>& obj) {
623 PersonalDataManagerAndroid* personal_data_manager_android = 627 PersonalDataManagerAndroid* personal_data_manager_android =
624 new PersonalDataManagerAndroid(env, obj); 628 new PersonalDataManagerAndroid(env, obj);
625 return reinterpret_cast<intptr_t>(personal_data_manager_android); 629 return reinterpret_cast<intptr_t>(personal_data_manager_android);
626 } 630 }
627 631
628 } // namespace autofill 632 } // namespace autofill
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698