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

Unified Diff: chrome/browser/payments/android/journey_logger_android.cc

Issue 2808513002: [Payments] Add PaymentRequest checkout funnel UKMs. (Closed)
Patch Set: Rebase Created 3 years, 8 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
« no previous file with comments | « chrome/browser/payments/android/journey_logger_android.h ('k') | components/autofill/core/browser/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/payments/android/journey_logger_android.cc
diff --git a/components/payments/content/android/journey_logger_android.cc b/chrome/browser/payments/android/journey_logger_android.cc
similarity index 74%
rename from components/payments/content/android/journey_logger_android.cc
rename to chrome/browser/payments/android/journey_logger_android.cc
index e373ae8253dc6848196c63cac74a59ec9d3df27b..c13d9e29932330c7a8edee58a99b11c3b89aaaf1 100644
--- a/components/payments/content/android/journey_logger_android.cc
+++ b/chrome/browser/payments/android/journey_logger_android.cc
@@ -2,10 +2,12 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "components/payments/content/android/journey_logger_android.h"
+#include "chrome/browser/payments/android/journey_logger_android.h"
#include "base/android/jni_string.h"
+#include "chrome/browser/browser_process.h"
#include "jni/JourneyLogger_jni.h"
+#include "url/gurl.h"
namespace payments {
namespace {
@@ -20,8 +22,11 @@ bool JourneyLoggerAndroid::Register(JNIEnv* env) {
return RegisterNativesImpl(env);
}
-JourneyLoggerAndroid::JourneyLoggerAndroid(bool is_incognito)
- : journey_logger_(is_incognito) {}
+JourneyLoggerAndroid::JourneyLoggerAndroid(bool is_incognito,
+ const std::string& url)
+ : journey_logger_(is_incognito,
+ GURL(url),
+ g_browser_process->ukm_service()) {}
JourneyLoggerAndroid::~JourneyLoggerAndroid() {}
@@ -84,6 +89,15 @@ void JourneyLoggerAndroid::SetShowCalled(
journey_logger_.SetShowCalled();
}
+void JourneyLoggerAndroid::SetEventOccurred(
+ JNIEnv* env,
+ const base::android::JavaParamRef<jobject>& jcaller,
+ jint jevent) {
+ DCHECK_GE(jevent, 0);
+ DCHECK_LT(jevent, JourneyLogger::Event::EVENT_MAX);
+ journey_logger_.SetEventOccurred(static_cast<JourneyLogger::Event>(jevent));
+}
+
void JourneyLoggerAndroid::RecordJourneyStatsHistograms(
JNIEnv* env,
const base::android::JavaParamRef<jobject>& jcaller,
@@ -95,10 +109,13 @@ void JourneyLoggerAndroid::RecordJourneyStatsHistograms(
static_cast<JourneyLogger::CompletionStatus>(jcompletion_status));
}
-static jlong InitJourneyLoggerAndroid(JNIEnv* env,
- const JavaParamRef<jobject>& jcaller,
- jboolean jis_incognito) {
- return reinterpret_cast<jlong>(new JourneyLoggerAndroid(jis_incognito));
+static jlong InitJourneyLoggerAndroid(
+ JNIEnv* env,
+ const JavaParamRef<jobject>& jcaller,
+ jboolean jis_incognito,
+ const base::android::JavaParamRef<jstring>& jurl) {
+ return reinterpret_cast<jlong>(new JourneyLoggerAndroid(
+ jis_incognito, ConvertJavaStringToUTF8(env, jurl)));
}
} // namespace payments
« no previous file with comments | « chrome/browser/payments/android/journey_logger_android.h ('k') | components/autofill/core/browser/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698