summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Babak Bostan <babakbo@google.com> 2020-03-03 22:54:50 +0000
committer Android (Google) Code Review <android-gerrit@google.com> 2020-03-03 22:54:50 +0000
commit4a3610675666566125a3e6202bdb5ac03bf9af3e (patch)
treedb78380fbda9e1a1b247b1245b74a65da6d09b9e
parent4431668e6b2a5356e3988f071c71db0db849b12c (diff)
parenta8600124133be26c543ba3ca181f96495fcf0ad5 (diff)
Merge "Add toast message when bluetooth connects to voice recognition." into rvc-dev
-rw-r--r--packages/CarSystemUI/res/values/config.xml1
-rw-r--r--packages/CarSystemUI/res/values/strings.xml2
-rw-r--r--packages/CarSystemUI/src/com/android/systemui/CarSystemUIBinder.java8
-rw-r--r--packages/CarSystemUI/src/com/android/systemui/voicerecognition/car/ConnectedDeviceVoiceRecognitionNotifier.java95
-rw-r--r--packages/CarSystemUI/tests/src/com/android/systemui/voicerecognition/car/ConnectedDeviceVoiceRecognitionNotifierTest.java97
5 files changed, 203 insertions, 0 deletions
diff --git a/packages/CarSystemUI/res/values/config.xml b/packages/CarSystemUI/res/values/config.xml
index d2f514c6c0ca..825b2813d47d 100644
--- a/packages/CarSystemUI/res/values/config.xml
+++ b/packages/CarSystemUI/res/values/config.xml
@@ -84,5 +84,6 @@
<item>com.android.systemui.theme.ThemeOverlayController</item>
<item>com.android.systemui.navigationbar.car.CarNavigationBar</item>
<item>com.android.systemui.toast.ToastUI</item>
+ <item>com.android.systemui.voicerecognition.car.ConnectedDeviceVoiceRecognitionNotifier</item>
</string-array>
</resources>
diff --git a/packages/CarSystemUI/res/values/strings.xml b/packages/CarSystemUI/res/values/strings.xml
index 0368e61978fd..9ea7ed027d34 100644
--- a/packages/CarSystemUI/res/values/strings.xml
+++ b/packages/CarSystemUI/res/values/strings.xml
@@ -20,4 +20,6 @@
<string name="hvac_min_text">Min</string>
<!-- String to represent largest setting of an HVAC system [CHAR LIMIT=5]-->
<string name="hvac_max_text">Max</string>
+ <!-- Text for voice recognition toast. [CHAR LIMIT=60] -->
+ <string name="voice_recognition_toast">Voice recognition now handled by connected Bluetooth device</string>
</resources>
diff --git a/packages/CarSystemUI/src/com/android/systemui/CarSystemUIBinder.java b/packages/CarSystemUI/src/com/android/systemui/CarSystemUIBinder.java
index 8eeaefda0920..8f9d7ed8c9b7 100644
--- a/packages/CarSystemUI/src/com/android/systemui/CarSystemUIBinder.java
+++ b/packages/CarSystemUI/src/com/android/systemui/CarSystemUIBinder.java
@@ -37,6 +37,7 @@ import com.android.systemui.statusbar.tv.TvStatusBar;
import com.android.systemui.theme.ThemeOverlayController;
import com.android.systemui.toast.ToastUI;
import com.android.systemui.util.leak.GarbageMonitor;
+import com.android.systemui.voicerecognition.car.ConnectedDeviceVoiceRecognitionNotifier;
import com.android.systemui.volume.VolumeUI;
import dagger.Binds;
@@ -174,4 +175,11 @@ public abstract class CarSystemUIBinder {
@IntoMap
@ClassKey(ToastUI.class)
public abstract SystemUI bindToastUI(ToastUI service);
+
+ /** Inject into ConnectedDeviceVoiceRecognitionNotifier. */
+ @Binds
+ @IntoMap
+ @ClassKey(ConnectedDeviceVoiceRecognitionNotifier.class)
+ public abstract SystemUI bindConnectedDeviceVoiceRecognitionNotifier(
+ ConnectedDeviceVoiceRecognitionNotifier sysui);
}
diff --git a/packages/CarSystemUI/src/com/android/systemui/voicerecognition/car/ConnectedDeviceVoiceRecognitionNotifier.java b/packages/CarSystemUI/src/com/android/systemui/voicerecognition/car/ConnectedDeviceVoiceRecognitionNotifier.java
new file mode 100644
index 000000000000..2f79f960f951
--- /dev/null
+++ b/packages/CarSystemUI/src/com/android/systemui/voicerecognition/car/ConnectedDeviceVoiceRecognitionNotifier.java
@@ -0,0 +1,95 @@
+/*
+ * Copyright (C) 2020 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.systemui.voicerecognition.car;
+
+import android.bluetooth.BluetoothHeadsetClient;
+import android.content.BroadcastReceiver;
+import android.content.Context;
+import android.content.Intent;
+import android.content.IntentFilter;
+import android.os.Handler;
+import android.os.UserHandle;
+import android.util.Log;
+import android.widget.Toast;
+
+import com.android.internal.annotations.VisibleForTesting;
+import com.android.systemui.R;
+import com.android.systemui.SysUIToast;
+import com.android.systemui.SystemUI;
+import com.android.systemui.dagger.qualifiers.Main;
+
+import javax.inject.Inject;
+
+/**
+ * Controller responsible for showing toast message when voice recognition over bluetooth device
+ * getting activated.
+ */
+public class ConnectedDeviceVoiceRecognitionNotifier extends SystemUI {
+
+ private static final String TAG = "CarVoiceRecognition";
+ @VisibleForTesting
+ static final int INVALID_VALUE = -1;
+ @VisibleForTesting
+ static final int VOICE_RECOGNITION_STARTED = 1;
+
+ private Handler mHandler;
+
+ private final BroadcastReceiver mVoiceRecognitionReceiver = new BroadcastReceiver() {
+ @Override
+ public void onReceive(Context context, Intent intent) {
+ if (Log.isLoggable(TAG, Log.DEBUG)) {
+ Log.d(TAG, "Voice recognition received an intent!");
+ }
+ if (intent == null
+ || intent.getAction() == null
+ || !BluetoothHeadsetClient.ACTION_AG_EVENT.equals(intent.getAction())
+ || !intent.hasExtra(BluetoothHeadsetClient.EXTRA_VOICE_RECOGNITION)) {
+ return;
+ }
+
+ int voiceRecognitionState = intent.getIntExtra(
+ BluetoothHeadsetClient.EXTRA_VOICE_RECOGNITION, INVALID_VALUE);
+
+ if (voiceRecognitionState == VOICE_RECOGNITION_STARTED) {
+ showToastMessage();
+ }
+ }
+ };
+
+ private void showToastMessage() {
+ mHandler.post(() -> SysUIToast.makeText(mContext, R.string.voice_recognition_toast,
+ Toast.LENGTH_LONG).show());
+ }
+
+ @Inject
+ public ConnectedDeviceVoiceRecognitionNotifier(Context context, @Main Handler handler) {
+ super(context);
+ mHandler = handler;
+ }
+
+ @Override
+ public void start() {
+ }
+
+ @Override
+ protected void onBootCompleted() {
+ IntentFilter filter = new IntentFilter();
+ filter.addAction(BluetoothHeadsetClient.ACTION_AG_EVENT);
+ mContext.registerReceiverAsUser(mVoiceRecognitionReceiver, UserHandle.ALL, filter,
+ /* broadcastPermission= */ null, /* scheduler= */ null);
+ }
+}
diff --git a/packages/CarSystemUI/tests/src/com/android/systemui/voicerecognition/car/ConnectedDeviceVoiceRecognitionNotifierTest.java b/packages/CarSystemUI/tests/src/com/android/systemui/voicerecognition/car/ConnectedDeviceVoiceRecognitionNotifierTest.java
new file mode 100644
index 000000000000..38b47d0aea5d
--- /dev/null
+++ b/packages/CarSystemUI/tests/src/com/android/systemui/voicerecognition/car/ConnectedDeviceVoiceRecognitionNotifierTest.java
@@ -0,0 +1,97 @@
+/*
+ * Copyright (C) 2019 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.systemui.voicerecognition.car;
+
+import static com.android.systemui.voicerecognition.car.ConnectedDeviceVoiceRecognitionNotifier.INVALID_VALUE;
+import static com.android.systemui.voicerecognition.car.ConnectedDeviceVoiceRecognitionNotifier.VOICE_RECOGNITION_STARTED;
+
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.never;
+import static org.mockito.Mockito.spy;
+import static org.mockito.Mockito.verify;
+
+import android.bluetooth.BluetoothHeadsetClient;
+import android.content.Intent;
+import android.os.Handler;
+import android.testing.AndroidTestingRunner;
+import android.testing.TestableLooper;
+
+import androidx.test.filters.SmallTest;
+
+import com.android.systemui.SysuiTestCase;
+
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+
+@RunWith(AndroidTestingRunner.class)
+@TestableLooper.RunWithLooper
+@SmallTest
+public class ConnectedDeviceVoiceRecognitionNotifierTest extends SysuiTestCase {
+
+ private static final String BLUETOOTH_PERM = android.Manifest.permission.BLUETOOTH;
+
+ private ConnectedDeviceVoiceRecognitionNotifier mVoiceRecognitionNotifier;
+ private Handler mTestHandler;
+
+ @Before
+ public void setUp() throws Exception {
+ TestableLooper testableLooper = TestableLooper.get(this);
+ mTestHandler = spy(new Handler(testableLooper.getLooper()));
+ mVoiceRecognitionNotifier = new ConnectedDeviceVoiceRecognitionNotifier(
+ mContext, mTestHandler);
+ mVoiceRecognitionNotifier.onBootCompleted();
+ }
+
+ @Test
+ public void testReceiveIntent_started_showToast() {
+ Intent intent = new Intent(BluetoothHeadsetClient.ACTION_AG_EVENT);
+ intent.putExtra(BluetoothHeadsetClient.EXTRA_VOICE_RECOGNITION, VOICE_RECOGNITION_STARTED);
+ mContext.sendBroadcast(intent, BLUETOOTH_PERM);
+ waitForIdleSync();
+
+ verify(mTestHandler).post(any());
+ }
+
+ @Test
+ public void testReceiveIntent_invalidExtra_noToast() {
+ Intent intent = new Intent(BluetoothHeadsetClient.ACTION_AG_EVENT);
+ intent.putExtra(BluetoothHeadsetClient.EXTRA_VOICE_RECOGNITION, INVALID_VALUE);
+ mContext.sendBroadcast(intent, BLUETOOTH_PERM);
+ waitForIdleSync();
+
+ verify(mTestHandler, never()).post(any());
+ }
+
+ @Test
+ public void testReceiveIntent_noExtra_noToast() {
+ Intent intent = new Intent(BluetoothHeadsetClient.ACTION_AG_EVENT);
+ mContext.sendBroadcast(intent, BLUETOOTH_PERM);
+ waitForIdleSync();
+
+ verify(mTestHandler, never()).post(any());
+ }
+
+ @Test
+ public void testReceiveIntent_invalidIntent_noToast() {
+ Intent intent = new Intent(BluetoothHeadsetClient.ACTION_AUDIO_STATE_CHANGED);
+ mContext.sendBroadcast(intent, BLUETOOTH_PERM);
+ waitForIdleSync();
+
+ verify(mTestHandler, never()).post(any());
+ }
+}