RESTRICT AUTOMERGE Fix unable to send file via OPP am: 5de62d1955 (e914289a) · Commits · e / os / android_packages_apps_Settings

src/com/android/settings/bluetooth/BluetoothPermissionActivity.java

+8 −11

Original line number	Original line	Diff line number	Diff line
	@@ -16,6 +16,8 @@

	package com.android.settings.bluetooth;		package com.android.settings.bluetooth;

			import static android.view.WindowManager.LayoutParams.SYSTEM_FLAG_HIDE_NON_SYSTEM_OVERLAY_WINDOWS;

	import android.bluetooth.BluetoothDevice;		import android.bluetooth.BluetoothDevice;
	import android.content.BroadcastReceiver;		import android.content.BroadcastReceiver;
	import android.content.Context;		import android.content.Context;
	@@ -30,12 +32,11 @@ import android.widget.TextView;

	import androidx.preference.Preference;		import androidx.preference.Preference;

			import com.android.internal.annotations.VisibleForTesting;
	import com.android.internal.app.AlertActivity;		import com.android.internal.app.AlertActivity;
	import com.android.internal.app.AlertController;		import com.android.internal.app.AlertController;
	import com.android.settings.R;		import com.android.settings.R;

	import static android.view.WindowManager.LayoutParams.SYSTEM_FLAG_HIDE_NON_SYSTEM_OVERLAY_WINDOWS;

	/**		/**
	* BluetoothPermissionActivity shows a dialog for accepting incoming		* BluetoothPermissionActivity shows a dialog for accepting incoming
	* profile connection request from untrusted devices.		* profile connection request from untrusted devices.
	@@ -51,8 +52,6 @@ public class BluetoothPermissionActivity extends AlertActivity implements
	private TextView messageView;		private TextView messageView;
	private Button mOkButton;		private Button mOkButton;
	private BluetoothDevice mDevice;		private BluetoothDevice mDevice;
	private String mReturnPackage = null;
	private String mReturnClass = null;

	private int mRequestType = 0;		private int mRequestType = 0;
	private BroadcastReceiver mReceiver = new BroadcastReceiver() {		private BroadcastReceiver mReceiver = new BroadcastReceiver() {
	@@ -89,8 +88,6 @@ public class BluetoothPermissionActivity extends AlertActivity implements
	}		}

	mDevice = i.getParcelableExtra(BluetoothDevice.EXTRA_DEVICE);		mDevice = i.getParcelableExtra(BluetoothDevice.EXTRA_DEVICE);
	mReturnPackage = i.getStringExtra(BluetoothDevice.EXTRA_PACKAGE_NAME);
	mReturnClass = i.getStringExtra(BluetoothDevice.EXTRA_CLASS_NAME);
	mRequestType = i.getIntExtra(BluetoothDevice.EXTRA_ACCESS_REQUEST_TYPE,		mRequestType = i.getIntExtra(BluetoothDevice.EXTRA_ACCESS_REQUEST_TYPE,
	BluetoothDevice.REQUEST_TYPE_PHONEBOOK_ACCESS);		BluetoothDevice.REQUEST_TYPE_PHONEBOOK_ACCESS);

	@@ -202,14 +199,14 @@ public class BluetoothPermissionActivity extends AlertActivity implements
	sendReplyIntentToReceiver(false, true);		sendReplyIntentToReceiver(false, true);
	}		}

	private void sendReplyIntentToReceiver(final boolean allowed, final boolean always) {		@VisibleForTesting
			void sendReplyIntentToReceiver(final boolean allowed, final boolean always) {
	Intent intent = new Intent(BluetoothDevice.ACTION_CONNECTION_ACCESS_REPLY);		Intent intent = new Intent(BluetoothDevice.ACTION_CONNECTION_ACCESS_REPLY);

	if (mReturnPackage != null && mReturnClass != null) {		if (DEBUG) {
	intent.setClassName(mReturnPackage, mReturnClass);		Log.i(TAG, "sendReplyIntentToReceiver() Request type: " + mRequestType
			+ " mReturnPackage");
	}		}
	if (DEBUG) Log.i(TAG, "sendReplyIntentToReceiver() Request type: " + mRequestType +
	" mReturnPackage" + mReturnPackage + " mReturnClass" + mReturnClass);

	intent.putExtra(BluetoothDevice.EXTRA_CONNECTION_ACCESS_RESULT,		intent.putExtra(BluetoothDevice.EXTRA_CONNECTION_ACCESS_RESULT,
	allowed ? BluetoothDevice.CONNECTION_ACCESS_YES		allowed ? BluetoothDevice.CONNECTION_ACCESS_YES

src/com/android/settings/bluetooth/BluetoothPermissionRequest.java

+6 −15

Original line number	Original line	Diff line number	Diff line
	@@ -56,8 +56,6 @@ public final class BluetoothPermissionRequest extends BroadcastReceiver {
	Context mContext;		Context mContext;
	int mRequestType;		int mRequestType;
	BluetoothDevice mDevice;		BluetoothDevice mDevice;
	String mReturnPackage = null;
	String mReturnClass = null;

	@Override		@Override
	public void onReceive(Context context, Intent intent) {		public void onReceive(Context context, Intent intent) {
	@@ -77,11 +75,10 @@ public final class BluetoothPermissionRequest extends BroadcastReceiver {
	mDevice = intent.getParcelableExtra(BluetoothDevice.EXTRA_DEVICE);		mDevice = intent.getParcelableExtra(BluetoothDevice.EXTRA_DEVICE);
	mRequestType = intent.getIntExtra(BluetoothDevice.EXTRA_ACCESS_REQUEST_TYPE,		mRequestType = intent.getIntExtra(BluetoothDevice.EXTRA_ACCESS_REQUEST_TYPE,
	BluetoothDevice.REQUEST_TYPE_PROFILE_CONNECTION);		BluetoothDevice.REQUEST_TYPE_PROFILE_CONNECTION);
	mReturnPackage = intent.getStringExtra(BluetoothDevice.EXTRA_PACKAGE_NAME);
	mReturnClass = intent.getStringExtra(BluetoothDevice.EXTRA_CLASS_NAME);

	if (DEBUG) Log.d(TAG, "onReceive request type: " + mRequestType + " return "		if (DEBUG) {
	+ mReturnPackage + "," + mReturnClass);		Log.d(TAG, "onReceive request type: " + mRequestType);
			}

	// Even if the user has already made the choice, Bluetooth still may not know that if		// Even if the user has already made the choice, Bluetooth still may not know that if
	// the user preference data have not been migrated from Settings app's shared		// the user preference data have not been migrated from Settings app's shared
	@@ -110,8 +107,6 @@ public final class BluetoothPermissionRequest extends BroadcastReceiver {
	connectionAccessIntent.putExtra(BluetoothDevice.EXTRA_ACCESS_REQUEST_TYPE,		connectionAccessIntent.putExtra(BluetoothDevice.EXTRA_ACCESS_REQUEST_TYPE,
	mRequestType);		mRequestType);
	connectionAccessIntent.putExtra(BluetoothDevice.EXTRA_DEVICE, mDevice);		connectionAccessIntent.putExtra(BluetoothDevice.EXTRA_DEVICE, mDevice);
	connectionAccessIntent.putExtra(BluetoothDevice.EXTRA_PACKAGE_NAME, mReturnPackage);
	connectionAccessIntent.putExtra(BluetoothDevice.EXTRA_CLASS_NAME, mReturnClass);

	String deviceAddress = mDevice != null ? mDevice.getAddress() : null;		String deviceAddress = mDevice != null ? mDevice.getAddress() : null;
	String deviceName = mDevice != null ? mDevice.getName() : null;		String deviceName = mDevice != null ? mDevice.getName() : null;
	@@ -288,10 +283,6 @@ public final class BluetoothPermissionRequest extends BroadcastReceiver {
	private void sendReplyIntentToReceiver(final boolean allowed) {		private void sendReplyIntentToReceiver(final boolean allowed) {
	Intent intent = new Intent(BluetoothDevice.ACTION_CONNECTION_ACCESS_REPLY);		Intent intent = new Intent(BluetoothDevice.ACTION_CONNECTION_ACCESS_REPLY);

	if (mReturnPackage != null && mReturnClass != null) {
	intent.setClassName(mReturnPackage, mReturnClass);
	}

	intent.putExtra(BluetoothDevice.EXTRA_CONNECTION_ACCESS_RESULT,		intent.putExtra(BluetoothDevice.EXTRA_CONNECTION_ACCESS_RESULT,
	allowed ? BluetoothDevice.CONNECTION_ACCESS_YES		allowed ? BluetoothDevice.CONNECTION_ACCESS_YES
	: BluetoothDevice.CONNECTION_ACCESS_NO);		: BluetoothDevice.CONNECTION_ACCESS_NO);

src/com/android/settings/bluetooth/DevicePickerFragment.java

+24 −4

Original line number	Original line	Diff line number	Diff line
	@@ -18,6 +18,7 @@ package com.android.settings.bluetooth;

	import static android.os.UserManager.DISALLOW_CONFIG_BLUETOOTH;		import static android.os.UserManager.DISALLOW_CONFIG_BLUETOOTH;

			import android.Manifest;
	import android.app.settings.SettingsEnums;		import android.app.settings.SettingsEnums;
	import android.bluetooth.BluetoothAdapter;		import android.bluetooth.BluetoothAdapter;
	import android.bluetooth.BluetoothDevice;		import android.bluetooth.BluetoothDevice;
	@@ -26,12 +27,15 @@ import android.content.Context;
	import android.content.Intent;		import android.content.Intent;
	import android.os.Bundle;		import android.os.Bundle;
	import android.os.UserManager;		import android.os.UserManager;
			import android.util.Log;
			import android.text.TextUtils;
	import android.view.Menu;		import android.view.Menu;
	import android.view.MenuInflater;		import android.view.MenuInflater;

	import androidx.annotation.VisibleForTesting;		import androidx.annotation.VisibleForTesting;

	import com.android.settings.R;		import com.android.settings.R;
			import com.android.settings.password.PasswordUtils;
	import com.android.settingslib.bluetooth.CachedBluetoothDevice;		import com.android.settingslib.bluetooth.CachedBluetoothDevice;
	import com.android.settingslib.core.AbstractPreferenceController;		import com.android.settingslib.core.AbstractPreferenceController;

	@@ -47,10 +51,16 @@ public final class DevicePickerFragment extends DeviceListPreferenceFragment {

	@VisibleForTesting		@VisibleForTesting
	BluetoothProgressCategory mAvailableDevicesCategory;		BluetoothProgressCategory mAvailableDevicesCategory;
			@VisibleForTesting
			Context mContext;
			@VisibleForTesting
			String mLaunchPackage;
			@VisibleForTesting
			String mLaunchClass;
			@VisibleForTesting
			String mCallingAppPackageName;

	private boolean mNeedAuth;		private boolean mNeedAuth;
	private String mLaunchPackage;
	private String mLaunchClass;
	private boolean mScanAllowed;		private boolean mScanAllowed;

	public DevicePickerFragment() {		public DevicePickerFragment() {
	@@ -84,6 +94,13 @@ public final class DevicePickerFragment extends DeviceListPreferenceFragment {
	getActivity().setTitle(getString(R.string.device_picker));		getActivity().setTitle(getString(R.string.device_picker));
	UserManager um = (UserManager) getSystemService(Context.USER_SERVICE);		UserManager um = (UserManager) getSystemService(Context.USER_SERVICE);
	mScanAllowed = !um.hasUserRestriction(DISALLOW_CONFIG_BLUETOOTH);		mScanAllowed = !um.hasUserRestriction(DISALLOW_CONFIG_BLUETOOTH);
			mCallingAppPackageName = PasswordUtils.getCallingAppPackageName(
			getActivity().getActivityToken());
			if (!TextUtils.equals(mCallingAppPackageName, mLaunchPackage)) {
			Log.w(TAG, "sendDevicePickedIntent() launch package name is not equivalent to"
			+ " calling package name!");
			}
			mContext = getContext();
	setHasOptionsMenu(true);		setHasOptionsMenu(true);
	}		}

	@@ -190,8 +207,11 @@ public final class DevicePickerFragment extends DeviceListPreferenceFragment {
	Intent intent = new Intent(BluetoothDevicePicker.ACTION_DEVICE_SELECTED);		Intent intent = new Intent(BluetoothDevicePicker.ACTION_DEVICE_SELECTED);
	intent.putExtra(BluetoothDevice.EXTRA_DEVICE, device);		intent.putExtra(BluetoothDevice.EXTRA_DEVICE, device);
	if (mLaunchPackage != null && mLaunchClass != null) {		if (mLaunchPackage != null && mLaunchClass != null) {
			if (TextUtils.equals(mCallingAppPackageName, mLaunchPackage)) {
	intent.setClassName(mLaunchPackage, mLaunchClass);		intent.setClassName(mLaunchPackage, mLaunchClass);
	}		}
	getActivity().sendBroadcast(intent);		}

			mContext.sendBroadcast(intent, Manifest.permission.BLUETOOTH_ADMIN);
	}		}
	}		}

tests/robotests/src/com/android/settings/bluetooth/BluetoothPermissionActivityTest.java

0 → 100644

+58 −0

Original line number	Original line	Diff line number	Diff line
			/*
			* Copyright (C) 2021 The Android Open Source Project
			*
			* Licensed under the Apache License, Version 2.0 (the "License");
			* you may not use this file except in compliance with the License.
			* You may obtain a copy of the License at
			*
			* http://www.apache.org/licenses/LICENSE-2.0
			*
			* Unless required by applicable law or agreed to in writing, software
			* distributed under the License is distributed on an "AS IS" BASIS,
			* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
			* See the License for the specific language governing permissions and
			* limitations under the License.
			*/

			package com.android.settings.bluetooth;

			import static org.mockito.ArgumentMatchers.eq;
			import static org.mockito.Mockito.spy;
			import static org.mockito.Mockito.verify;

			import android.content.Context;
			import android.content.Intent;

			import org.junit.Before;
			import org.junit.Test;
			import org.junit.runner.RunWith;
			import org.mockito.ArgumentCaptor;
			import org.mockito.MockitoAnnotations;
			import org.robolectric.RobolectricTestRunner;
			import org.robolectric.RuntimeEnvironment;
			import org.robolectric.util.ReflectionHelpers;

			@RunWith(RobolectricTestRunner.class)
			public class BluetoothPermissionActivityTest {

			private BluetoothPermissionActivity mActivity;
			private Context mContext;

			@Before
			public void setUp() {
			MockitoAnnotations.initMocks(this);
			mContext = spy(RuntimeEnvironment.application);
			mActivity = new BluetoothPermissionActivity();
			}

			@Test
			public void sendBroadcastWithPermission() {
			final ArgumentCaptor<Intent> intentCaptor = ArgumentCaptor.forClass(Intent.class);
			ReflectionHelpers.setField(mActivity, "mBase", mContext);

			mActivity.sendReplyIntentToReceiver(true, true);

			verify(mContext).sendBroadcast(intentCaptor.capture(),
			eq("android.permission.BLUETOOTH_ADMIN"));
			}
			}

tests/robotests/src/com/android/settings/bluetooth/DevicePickerFragmentTest.java

+54 −1

Original line number	Original line	Diff line number	Diff line
	@@ -16,28 +16,46 @@

	package com.android.settings.bluetooth;		package com.android.settings.bluetooth;

			import static com.google.common.truth.Truth.assertThat;

			import static org.mockito.ArgumentMatchers.eq;
			import static org.mockito.Mockito.mock;
			import static org.mockito.Mockito.never;
			import static org.mockito.Mockito.spy;
	import static org.mockito.Mockito.verify;		import static org.mockito.Mockito.verify;
			import static org.mockito.Mockito.when;

			import android.bluetooth.BluetoothDevice;
			import android.content.Context;
			import android.content.Intent;

			import com.android.settingslib.bluetooth.CachedBluetoothDevice;

	import org.junit.Before;		import org.junit.Before;
	import org.junit.Test;		import org.junit.Test;
	import org.junit.runner.RunWith;		import org.junit.runner.RunWith;
			import org.mockito.ArgumentCaptor;
	import org.mockito.Mock;		import org.mockito.Mock;
	import org.mockito.MockitoAnnotations;		import org.mockito.MockitoAnnotations;
	import org.robolectric.RobolectricTestRunner;		import org.robolectric.RobolectricTestRunner;
			import org.robolectric.RuntimeEnvironment;

	@RunWith(RobolectricTestRunner.class)		@RunWith(RobolectricTestRunner.class)
	public class DevicePickerFragmentTest {		public class DevicePickerFragmentTest {

	@Mock		@Mock
	private BluetoothProgressCategory mAvailableDevicesCategory;		private BluetoothProgressCategory mAvailableDevicesCategory;

	private DevicePickerFragment mFragment;		private DevicePickerFragment mFragment;
			private Context mContext;

	@Before		@Before
	public void setUp() {		public void setUp() {
	MockitoAnnotations.initMocks(this);		MockitoAnnotations.initMocks(this);

	mFragment = new DevicePickerFragment();		mFragment = new DevicePickerFragment();
			mContext = spy(RuntimeEnvironment.application);
			mFragment.mContext = mContext;
	mFragment.mAvailableDevicesCategory = mAvailableDevicesCategory;		mFragment.mAvailableDevicesCategory = mAvailableDevicesCategory;
	}		}

	@@ -49,4 +67,39 @@ public class DevicePickerFragmentTest {

	verify(mAvailableDevicesCategory).setProgress(true);		verify(mAvailableDevicesCategory).setProgress(true);
	}		}

			@Test
			public void callingPackageIsEqualToLaunchPackage_sendBroadcastToLaunchPackage() {
			final CachedBluetoothDevice cachedDevice = mock(CachedBluetoothDevice.class);
			final BluetoothDevice bluetoothDevice = mock(BluetoothDevice.class);
			final ArgumentCaptor<Intent> intentCaptor = ArgumentCaptor.forClass(Intent.class);
			when(cachedDevice.getDevice()).thenReturn(bluetoothDevice);
			mFragment.mSelectedDevice = bluetoothDevice;
			mFragment.mLaunchPackage = "com.android.settings";
			mFragment.mLaunchClass = "com.android.settings.bluetooth.BluetoothPermissionActivity";
			mFragment.mCallingAppPackageName = "com.android.settings";

			mFragment.onDeviceBondStateChanged(cachedDevice, BluetoothDevice.BOND_BONDED);

			verify(mContext).sendBroadcast(intentCaptor.capture(),
			eq("android.permission.BLUETOOTH_ADMIN"));
			assertThat(intentCaptor.getValue().getComponent().getPackageName())
			.isEqualTo(mFragment.mLaunchPackage);
			}

			@Test
			public void callingPackageIsNotEqualToLaunchPackage_broadcastNotSend() {
			final CachedBluetoothDevice cachedDevice = mock(CachedBluetoothDevice.class);
			final BluetoothDevice bluetoothDevice = mock(BluetoothDevice.class);
			final ArgumentCaptor<Intent> intentCaptor = ArgumentCaptor.forClass(Intent.class);
			when(cachedDevice.getDevice()).thenReturn(bluetoothDevice);
			mFragment.mSelectedDevice = bluetoothDevice;
			mFragment.mLaunchPackage = "com.fake.settings";
			mFragment.mLaunchClass = "com.android.settings.bluetooth.BluetoothPermissionActivity";
			mFragment.mCallingAppPackageName = "com.android.settings";

			mFragment.onDeviceBondStateChanged(cachedDevice, BluetoothDevice.BOND_BONDED);

			verify(mContext, never()).sendBroadcast(intentCaptor.capture());
			}
	}		}