Expand Chrome OS ARC support to create one tree source per package

components/arc/common/accessibility_helper.mojom

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Next MinVersion: 2
 
 module arc.mojom;
 
 import "screen_rect.mojom";
 
@@ skipping 138 matching lines @@
 // accessibility service.
 [Extensible]
 enum AccessibilityFilterType {
   // No events will be sent.
   OFF,
 
   // Only send focus events along with the source focus node.
   FOCUS,
 
   // Send a complete tree from the event source's root for every event.
-  ALL
+  ALL,
+
+  // Send complete subtrees for root nodes with whitelisted package names.
+  WHITELISTED_PACKAGE_NAME
 };
 
 // AccessibilityEventData is a struct to contain info of
 // AccessibilityEvent in Android.
 // https://developer.android.com/reference/android/view/accessibility/AccessibilityEvent.html
 struct AccessibilityEventData {
   AccessibilityEventType eventType;
   int32 sourceId;
   array<AccessibilityNodeInfoData> nodeData;
 };
 
 // Next method ID: 2
 interface AccessibilityHelperHost {
   OnAccessibilityEventDeprecated@0(AccessibilityEventType eventType,
                                    AccessibilityNodeInfoData? eventSource);
 
   // OnAccessibilityEvent is called when a converted Android accessibility event
   // is sent from Android.
   OnAccessibilityEvent@1(AccessibilityEventData eventData);
 };
 
-// Next method ID: 3
+// Next method ID: 4
 interface AccessibilityHelperInstance {
   Init@0(AccessibilityHelperHost host);
 
   // Perform the specified action on a node requested by a Chrome client.
   PerformAction@1(int32 id, AccessibilityActionType action);
 
   // Set a filter on the event types received.
   SetFilter@2(AccessibilityFilterType filterType);
+
+  // Sets native Chrome accessibility support on or off for the current package.
+  SetCurrentPackageNativeAccessibility@3(bool enabled);

[yawano, 2017/04/27 10:01:27]

Wouldn't this cause race condition? i.e. current package might not match between
Chrome and Android. How about to change this as
setChromeAccessibilitySupportEnabled(bool enabled, string package_name)?

[David Tseng, 2017/04/28 00:27:44]

Yes, you're right. I was adding this to avoid another review, but since this
ins't being called at all, I'll defer this to another cl, where it would be
exercised by ChromeVox.

It's pretty clear I should add a test suite for everything in this cl as well.

 };