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

Unified Diff: components/payments/content/android/currency_formatter_android.cc

Issue 2750103005: [Payments] Move journey logger to native. (Closed)
Patch Set: Add the component_jni_registrar files Created 3 years, 9 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: components/payments/content/android/currency_formatter_android.cc
diff --git a/components/payments/content/android/currency_formatter_android.cc b/components/payments/content/android/currency_formatter_android.cc
index 0ea0f91203346dfe2d267d5b63aadfb12459b640..3b3694e5c3da394b1efdd0de26ecd04f4d65040e 100644
--- a/components/payments/content/android/currency_formatter_android.cc
+++ b/components/payments/content/android/currency_formatter_android.cc
@@ -19,7 +19,7 @@ using ::base::android::ConvertJavaStringToUTF8;
CurrencyFormatterAndroid::CurrencyFormatterAndroid(
JNIEnv* env,
- jobject unused_obj,
+ jobject jcaller,
const JavaParamRef<jstring>& currency_code,
const JavaParamRef<jstring>& currency_system,
const JavaParamRef<jstring>& locale_name) {
@@ -34,13 +34,13 @@ CurrencyFormatterAndroid::CurrencyFormatterAndroid(
CurrencyFormatterAndroid::~CurrencyFormatterAndroid() {}
void CurrencyFormatterAndroid::Destroy(JNIEnv* env,
- const JavaParamRef<jobject>& obj) {
+ const JavaParamRef<jobject>& jcaller) {
delete this;
}
base::android::ScopedJavaLocalRef<jstring> CurrencyFormatterAndroid::Format(
JNIEnv* env,
- const JavaParamRef<jobject>& unused_obj,
+ const JavaParamRef<jobject>& jcaller,
const JavaParamRef<jstring>& amount) {
base::string16 result =
currency_formatter_->Format(ConvertJavaStringToUTF8(env, amount));

Powered by Google App Engine
This is Rietveld 408576698