From 19557de7676441a5e3f972b9dca128a2edb7a036 Mon Sep 17 00:00:00 2001
From: Phil Weaver <pweaver@google.com>
Date: Fri, 6 May 2016 08:32:38 -0700
Subject: [PATCH] Make AccessibilityService#disableSelf atomic.

This API was using a oneway aidl call, which meant that the
service was disabled some time after the method returned. That
confused tests that were turning a service off during tearDown
and then turning it back on again in setUp.

Bug: 28621277

Change-Id: I75984df0613bdbb1bc876e2a15caf59106027337
---
 .../accessibilityservice/IAccessibilityServiceConnection.aidl   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core/java/android/accessibilityservice/IAccessibilityServiceConnection.aidl b/core/java/android/accessibilityservice/IAccessibilityServiceConnection.aidl
index d5e8a112a485e..3783fcac55697 100644
--- a/core/java/android/accessibilityservice/IAccessibilityServiceConnection.aidl
+++ b/core/java/android/accessibilityservice/IAccessibilityServiceConnection.aidl
@@ -65,7 +65,7 @@ interface IAccessibilityServiceConnection {
 
     boolean performGlobalAction(int action);
 
-    oneway void disableSelf();
+    void disableSelf();
 
     oneway void setOnKeyEventResult(boolean handled, int sequence);