Commit 8856c278 authored by shyam's avatar shyam Committed by Commit Bot

Exposing set mutator in ChromecastConfigAndroid

Adding set method in ChromecastConfigAndroid which will be called by
native method.
This method will be called when prefs value is updated.

called when prefs value changes.

Test: Verified that the ChromecastConfigAndroid mutator function is
Bug: Internal b/77485825
Change-Id: Ie6021da906a7a87ec07a2d27e56c98ef75925bc4
Reviewed-on: https://chromium-review.googlesource.com/1045314Reviewed-by: default avatarLuke Halliwell <halliwell@chromium.org>
Commit-Queue: Shyamsundar Parthasarathy <shyamsundarp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#556582}
parent 4dfc4d7f
...@@ -32,12 +32,17 @@ ChromecastConfigAndroid::~ChromecastConfigAndroid() { ...@@ -32,12 +32,17 @@ ChromecastConfigAndroid::~ChromecastConfigAndroid() {
} }
bool ChromecastConfigAndroid::CanSendUsageStats() { bool ChromecastConfigAndroid::CanSendUsageStats() {
// TODO(gunsch): make opt-in.stats pref the source of truth for this data, // TODO(sanfin): make opt-in.stats pref the source of truth for this data,
// instead of Android prefs, then delete ChromecastConfigAndroid. // instead of Android prefs, then delete ChromecastConfigAndroid.
JNIEnv* env = base::android::AttachCurrentThread(); JNIEnv* env = base::android::AttachCurrentThread();
return Java_ChromecastConfigAndroid_canSendUsageStats(env); return Java_ChromecastConfigAndroid_canSendUsageStats(env);
} }
void ChromecastConfigAndroid::SetSendUsageStats(bool enabled) {
JNIEnv* env = base::android::AttachCurrentThread();
Java_ChromecastConfigAndroid_setSendUsageStats(env, enabled);
}
// Registers a handler to be notified when SendUsageStats is changed. // Registers a handler to be notified when SendUsageStats is changed.
void ChromecastConfigAndroid::SetSendUsageStatsChangedCallback( void ChromecastConfigAndroid::SetSendUsageStatsChangedCallback(
base::RepeatingCallback<void(bool)> callback) { base::RepeatingCallback<void(bool)> callback) {
......
...@@ -22,6 +22,8 @@ class ChromecastConfigAndroid { ...@@ -22,6 +22,8 @@ class ChromecastConfigAndroid {
// crash reports. // crash reports.
bool CanSendUsageStats(); bool CanSendUsageStats();
void SetSendUsageStats(bool enabled);
// Registers a handler to be notified when SendUsageStats is changed. // Registers a handler to be notified when SendUsageStats is changed.
void SetSendUsageStatsChangedCallback( void SetSendUsageStatsChangedCallback(
base::RepeatingCallback<void(bool)> callback); base::RepeatingCallback<void(bool)> callback);
......
...@@ -32,5 +32,10 @@ public final class ChromecastConfigAndroid { ...@@ -32,5 +32,10 @@ public final class ChromecastConfigAndroid {
return sSettingsManager.isSendUsageStatsEnabled(); return sSettingsManager.isSendUsageStatsEnabled();
} }
@CalledByNative
public static void setSendUsageStats(boolean enabled) {
sSettingsManager.setSendUsageStatsEnabled(enabled);
}
private static native void nativeSetSendUsageStatsEnabled(boolean enabled); private static native void nativeSetSendUsageStatsEnabled(boolean enabled);
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment