Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a0f30f5a authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "gtbs: Fix sending notification on CallControl point" am: e87480b9 

Original change: https://android-review.googlesource.com/c/platform/packages/modules/Bluetooth/+/1956178

Change-Id: I00aa6c0b88c94ec08c11ce8c88224daff6a35822
parents 9de056f2 e87480b9

android/app/src/com/android/bluetooth/tbs/TbsGatt.java

+5 −1
Original line number Diff line number Diff line
@@ -355,6 +355,10 @@ public class TbsGatt { 
            return success;

        }



        public boolean setValueNoNotify(byte[] value) {

            return super.setValue(value);

        }



        public boolean clearValue(boolean notify) {

            boolean success = super.setValue(new byte[0]);

            if (success && notify && isNotifiable()) {
@@ -409,7 +413,7 @@ public class TbsGatt { 
            value[1] = (byte) (callIndex);

            value[2] = (byte) (requestResult);



            super.setValue(value);

            super.setValueNoNotify(value);



            // to avoid sending control point notification before write response

            mContext.getMainThreadHandler().post(() -> mNotifier.notify(device, this));

android/app/tests/unit/src/com/android/bluetooth/tbs/TbsGattTest.java

+4 −3
Original line number Diff line number Diff line
@@ -465,8 +465,8 @@ public class TbsGattTest { 
        Assert.assertTrue(Arrays.equals(characteristic.getValue(),

                new byte[] {(byte) (requestedOpcode & 0xff), (byte) (callIndex & 0xff),

                        (byte) (result & 0xff)}));

        verify(mMockGattServer).notifyCharacteristicChanged(eq(mCurrentDevice), eq(characteristic),

                eq(false));

        verify(mMockGattServer, after(2000)).notifyCharacteristicChanged(eq(mCurrentDevice),

                eq(characteristic), eq(false));

        reset(mMockGattServer);



        callIndex = 0x02;
@@ -477,7 +477,8 @@ public class TbsGattTest { 
        Assert.assertTrue(Arrays.equals(characteristic.getValue(),

                new byte[] {(byte) (requestedOpcode & 0xff), (byte) (callIndex & 0xff),

                        (byte) (result & 0xff)}));

        verify(mMockGattServer, times(0)).notifyCharacteristicChanged(any(), any(), anyBoolean());

        verify(mMockGattServer, after(2000).times(0)).notifyCharacteristicChanged(any(), any(),

                anyBoolean());

    }



    @Test