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

Unified Diff: device/bluetooth/test/android/java/src/org/chromium/device/bluetooth/Fakes.java

Issue 2038753004: Add a LocationUtils class to give all Chromium Android code access to location helpers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@lkcr
Patch Set: Adjust DEPS. Created 4 years, 6 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: device/bluetooth/test/android/java/src/org/chromium/device/bluetooth/Fakes.java
diff --git a/device/bluetooth/test/android/java/src/org/chromium/device/bluetooth/Fakes.java b/device/bluetooth/test/android/java/src/org/chromium/device/bluetooth/Fakes.java
index 35d2dbcaf67247147ba9b0ac1642c6962269b0c7..ae1873581adce554331f4e26fee34bd6a2b9e362 100644
--- a/device/bluetooth/test/android/java/src/org/chromium/device/bluetooth/Fakes.java
+++ b/device/bluetooth/test/android/java/src/org/chromium/device/bluetooth/Fakes.java
@@ -4,7 +4,6 @@
package org.chromium.device.bluetooth;
-import android.Manifest;
import android.annotation.TargetApi;
import android.bluetooth.BluetoothDevice;
import android.bluetooth.le.ScanFilter;
@@ -23,9 +22,7 @@ import org.chromium.base.annotations.JNINamespace;
import java.util.ArrayList;
import java.util.HashMap;
-import java.util.HashSet;
import java.util.List;
-import java.util.Set;
import java.util.UUID;
/**
@@ -68,7 +65,7 @@ class Fakes {
@CalledByNative("FakeBluetoothAdapter")
public void denyPermission() {
- mFakeContext.mPermissions.clear();
+ mFakeContext.mHasLocation = false;
}
/**
@@ -194,16 +191,15 @@ class Fakes {
* Fakes android.content.Context.
*/
static class FakeContext extends Wrappers.ContextWrapper {
- public final Set<String> mPermissions = new HashSet<String>();
+ public boolean mHasLocation = true;
public FakeContext() {
super(null);
- mPermissions.add(Manifest.permission.ACCESS_COARSE_LOCATION);
}
@Override
- public boolean checkPermission(String permission) {
- return mPermissions.contains(permission);
+ public boolean hasAndroidLocationPermission() {
+ return mHasLocation;
}
@Override

Powered by Google App Engine
This is Rietveld 408576698