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

Unified Diff: components/variations/android/variations_seed_bridge.cc

Issue 1437833004: Implemented clearing Java prefs after pulling variations first run seed from Java to C++ side (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merged with CL 1437833002 + fixed some code review comments Created 5 years, 1 month 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/variations/android/variations_seed_bridge.cc
diff --git a/components/variations/android/variations_seed_bridge.cc b/components/variations/android/variations_seed_bridge.cc
index cf5cbe41db3ac65ff1fe91a503f42eb94c976575..257e18203f05d214257c250ad40592d508c652c8 100644
--- a/components/variations/android/variations_seed_bridge.cc
+++ b/components/variations/android/variations_seed_bridge.cc
@@ -55,5 +55,16 @@ void GetVariationsFirstRunSeed(std::string* seed_data,
*seed_country = ConvertJavaStringToUTF8(j_seed_country);
}
+void ClearJavaFirstRunPrefs() {
+ JNIEnv* env = AttachCurrentThread();
+ Java_VariationsSeedBridge_clearFirstRunPrefs(env, GetApplicationContext());
+}
+
+void markVariationsSeedAsStored() {
Alexei Svitkine (slow) 2015/11/12 19:03:22 Cap the first letter.
Alexander Agulenko 2015/11/12 19:50:09 Done.
+ JNIEnv* env = AttachCurrentThread();
+ Java_VariationsSeedBridge_markVariationsSeedAsStored(env,
+ GetApplicationContext());
+}
+
} // namespace android
} // namespace variations

Powered by Google App Engine
This is Rietveld 408576698