[Payments] Record use of profiles and credit cards in Payment Request.

chrome/browser/autofill/android/personal_data_manager_android.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_AUTOFILL_ANDROID_PERSONAL_DATA_MANAGER_ANDROID_H_
 #define CHROME_BROWSER_AUTOFILL_ANDROID_PERSONAL_DATA_MANAGER_ANDROID_H_
 
 #include "base/android/jni_weak_ref.h"
 #include "base/android/scoped_java_ref.h"
 #include "base/macros.h"
@@ skipping 123 matching lines @@
       const base::android::JavaParamRef<jobject>& jweb_contents,
       const base::android::JavaParamRef<jstring>& jguid,
       const base::android::JavaParamRef<jobject>& jdelegate);
 
   // PersonalDataManagerObserver:
   void OnPersonalDataChanged() override;
 
   // Registers the JNI bindings for this class.
   static bool Register(JNIEnv* env);
 
+  // These functions act on the usage stats of local profiles and credit cards.
+  // --------------------
+
+  // Records the use and log usage metrics for the profile associated with the
+  // |jguid|. Increments the use count of the profile and sets its use date to
+  // the current time.
+  void RecordAndLogProfileUse(
+      JNIEnv* env,
+      const base::android::JavaParamRef<jobject>& unused_obj,
+      const base::android::JavaParamRef<jstring>& jguid);
+
   // Sets the use count and use date of the profile associated to the |jguid|.
   // Both |count| and |date| should be non-negative. |date| 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.
   void SetProfileUseStatsForTesting(
       JNIEnv* env,
       const base::android::JavaParamRef<jobject>& unused_obj,
       const base::android::JavaParamRef<jstring>& jguid,
       jint count,
       jint date);
 
+  // Returns the use count of the profile associated to the |jguid|.
+  jint GetProfileUseCountForTesting(
+    JNIEnv* env,

[Bernhard Bauer, 2016/07/14 10:14:19]

Indent four spaces.

[sebsg, 2016/07/20 14:56:47]

Done.

+    const base::android::JavaParamRef<jobject>& unused_obj,
+    const base::android::JavaParamRef<jstring>& jguid);
+
+  // Returns the use date of the profile associated to the |jguid|. 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.
+  jlong GetProfileUseDateForTesting(
+    JNIEnv* env,
+    const base::android::JavaParamRef<jobject>& unused_obj,
+    const base::android::JavaParamRef<jstring>& jguid);
+
+  // Records the use and log usage metrics for the credit card associated with
+  // the |jguid|. Increments the use count of the credit card and sets its use
+  // date to the current time.
+  void RecordAndLogCreditCardUse(
+      JNIEnv* env,
+      const base::android::JavaParamRef<jobject>& unused_obj,
+      const base::android::JavaParamRef<jstring>& jguid);
+
   // Sets the use count and use date of the credit card associated to the
   // |jguid|. Both |count| and |date| should be non-negative. |date| 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.
   void SetCreditCardUseStatsForTesting(
       JNIEnv* env,
       const base::android::JavaParamRef<jobject>& unused_obj,
       const base::android::JavaParamRef<jstring>& jguid,
       jint count,
       jint date);
 
+  // Returns the use count of the credit card associated to the |jguid|.
+  jint GetCreditCardUseCountForTesting(
+    JNIEnv* env,
+    const base::android::JavaParamRef<jobject>& unused_obj,
+    const base::android::JavaParamRef<jstring>& jguid);
+
+  // Returns the use date of the credit card associated to the |jguid|. 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.
+  jlong GetCreditCardUseDateForTesting(
+    JNIEnv* env,
+    const base::android::JavaParamRef<jobject>& unused_obj,
+    const base::android::JavaParamRef<jstring>& jguid);
+
+  // Returns the current date represented as an absolute point in coordinated
+  // universal time (UTC) represented as microseconds since the Windows epoch.

[Bernhard Bauer, 2016/07/14 10:14:19]

Uh, this is how the time is stored internally, but you are calling ToTimeT(),
which is based on the Unix epoch. Why not use ToJavaTime()?

[sebsg, 2016/07/20 14:56:47]

I did not use it because it does not seem to be paired with a FromJavaTime() to
be used when setting the use stats. Also because TimeT is used in the autofill
code so I wanted to be consistent with that. 

This method will be removed once the mock clock is implemented. I can probably
make it work with JavaTime in the meantime if you prefer.

[Bernhard Bauer, 2016/07/20 15:09:44]

Hm, if you want to use time_t, can you at least update the comment to be
correct?

[sebsg, 2016/07/20 20:30:50]

Done.

+  // For more details see the comment header in time.h
+  jlong GetCurrentDateForTesting(
+    JNIEnv* env,
+    const base::android::JavaParamRef<jobject>& unused_obj);
+
  private:
   ~PersonalDataManagerAndroid() override;
 
   // Returns the GUIDs of the |profiles| passed as parameter.
   base::android::ScopedJavaLocalRef<jobjectArray> GetProfileGUIDs(
       JNIEnv* env,
       const std::vector<AutofillProfile*>& profiles);
 
   // Returns the GUIDs of the |credit_cards| passed as parameter.
   base::android::ScopedJavaLocalRef<jobjectArray> GetCreditCardGUIDs(
@@ skipping 17 matching lines @@
 
   // Pointer to the PersonalDataManager for the main profile.
   PersonalDataManager* personal_data_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(PersonalDataManagerAndroid);
 };
 
 }  // namespace autofill
 
 #endif  // CHROME_BROWSER_AUTOFILL_ANDROID_PERSONAL_DATA_MANAGER_ANDROID_H_