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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/PhysicalWebPreferenceFragment.java

Issue 2600283002: Remove context param from PhysicalWebUma methods (Closed)
Patch Set: Created 4 years 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: chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/PhysicalWebPreferenceFragment.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/PhysicalWebPreferenceFragment.java b/chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/PhysicalWebPreferenceFragment.java
index 083aab5d734b0b30a021f31175b334643d2c1899..f6851a379b3bad10e9165e3e7a51a3df9deef1df 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/PhysicalWebPreferenceFragment.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/PhysicalWebPreferenceFragment.java
@@ -62,11 +62,11 @@ public class PhysicalWebPreferenceFragment extends PreferenceFragment {
case REQUEST_ID:
if (grantResults.length > 0
&& grantResults[0] == PackageManager.PERMISSION_GRANTED) {
- PhysicalWebUma.onPrefsLocationGranted(getActivity());
+ PhysicalWebUma.onPrefsLocationGranted();
Log.d(TAG, "Location permission granted");
PhysicalWeb.startPhysicalWeb();
} else {
- PhysicalWebUma.onPrefsLocationDenied(getActivity());
+ PhysicalWebUma.onPrefsLocationDenied();
Log.d(TAG, "Location permission denied");
}
break;
@@ -86,10 +86,10 @@ public class PhysicalWebPreferenceFragment extends PreferenceFragment {
public boolean onPreferenceChange(Preference preference, Object newValue) {
boolean enabled = (boolean) newValue;
if (enabled) {
- PhysicalWebUma.onPrefsFeatureEnabled(getActivity());
+ PhysicalWebUma.onPrefsFeatureEnabled();
ensureLocationPermission();
} else {
- PhysicalWebUma.onPrefsFeatureDisabled(getActivity());
+ PhysicalWebUma.onPrefsFeatureDisabled();
}
PrivacyPreferencesManager.getInstance().setPhysicalWebEnabled(enabled);
return true;

Powered by Google App Engine
This is Rietveld 408576698