d116d7c78a9c53f30a73bf273bd7618312cf3847 |
|
22-Nov-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Fixing memory leaks in the accessiiblity layer. 1. AccessibilityInteractionConnections were removed from the AccessiiblityManagerService but their DeathRecipents were not unregistered, thus every removed interaction connection was essentially leaking. Such connection is registered in the system for every ViewRootImpl when accessiiblity is enabled and inregistered when disabled. 2. Every AccessibilityEvent and AccessiilbityEventInfo obtained from a widnow content querying accessibility service had a handle to a binder proxy over which to make queries. Hoewever, holding a proxy to a remote binder prevents the latter from being garbage collected. Therefore, now the events and infos have a connection id insteand and the hindden singleton AccessiiblityInteaction client via which queries are made has a registry with the connections. This class looks up the connection given its id before making an IPC. Now the connection is stored in one place and when an accessibility service is disconnected the system sets the connection to null so the binder object in the system process can be GCed. Note that before this change a bad implemented accessibility service could cache events or infos causing a leak in the system process. This should never happen. 3. SparseArray was not clearing the reference to the last moved element while garbage collecting thus causing a leak. bug:5664337 Change-Id: Id397f614b026d43bd7b57bb7f8186bca5cdfcff9
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
b6eca6e6691d4563d8395b3c3843d9932a2a6560 |
|
27-Sep-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Accessibility test automation API not working. 1. Due to a previous change that disabled accessibility if not enabled and installed serivces are present the automation APIs stopped working since they use fake automation service that is not installed. 2. Added clean up of death recipients when binders die. bug:5374662 bug:5239044 Change-Id: I1f3c8cd1d1c79753a4a64e2b8b2963025abb2939
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
1e741b2705023be64389ac3e31c5f63d330470a5 |
|
21-Sep-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
System not checking if enabled services are installed when disabling accessibility. The system is didabling accessiblity if no accessibility serivces are enabled to avoid sending events across processes if no recipients are present. The check considers enabled services which may not have been installed. Now the check is made against enabled and installed serivces. bug:5347273 Change-Id: Iad391a1a5bf0bbca470584bc8392f35821ba768c
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
ea515aeafa01de6f50c854ee381b972ef2478284 |
|
15-Sep-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Update the public APIs for finding views by text to optionally use content description. 1. Added flags to the search method to specify whether to match text or content description or both. 2. Added test case for the seach by content description. 3. Updated the code in AccessibilityManager service to reflect the latest changes there so test automation service works - this is the fake service used for UI automation. Change-Id: I14a6779a920ff0430e78947ea5aaf876c2e66076
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
d07d60b9bc0fd2e88f58d6e68dbfadad1bdd31cf |
|
14-Sep-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
AccessibilityManagerService not registered for removed packages. The pakcage monitor in the AccessibilityManagerService was not watching for packages that are removed. This is needes since 1) we need to remove the package from the enabled accessibility serivces and clean up after the removed serivice; 2) we need to disable accessibility if the last access serivices went away. Change-Id: I06d33b411ce60703e5a2843107323ffc87046c16
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
37fedf8b78dea969b97188fd3079fea5e71704be |
|
14-Sep-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Disable accessibility if all accessibility serivces are disbaled. Accessibility was kept enabled even if all accessibility services are disabled (explicitly by the user or removed) which was causing the system to fire accessibility events that will never be consumed. Change-Id: Ifb03e786ac0106687252bd1979725ffd724ad1c5
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
8bd69610aafc6995126965d1d23b771fe02a9084 |
|
23-Aug-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Intra-process view hierarchy interrogation does not work. The content retrieval APIs are synchronous from a client's perspective but internally they are asynchronous. The client thread calls into the system requesting an action and providing a callback to receive the result after which it waits up to a timeout for that result. The system enforces security and then delegates the request to a given view hierarchy where a message is posted (from a binder thread) describing what to be performed by the main UI thread the result of which it delivered via the mentioned callback. However, the blocked client thread and the main UI thread of the target view hierarchy can be the same one, for example an accessibility service and an activity run in the same process, thus they are executed on the same main thread. In such a case the retrieval will fail since the UI thread that has to process the message describing the work to be done is blocked waiting for a result is has to compute! To avoid this scenario when making a call the client also passes its process and thread ids so the accessed view hierarchy can detect if the client making the request is running in its main UI thread. In such a case the view hierarchy, specifically the binder thread performing the IPC to it, does not post a message to be run on the UI thread but passes it to the singleton interaction client through which all interactions occur and the latter is responsible to execute the message before starting to wait for the asynchronous result delivered via the callback. In this case the expected result is already received so no waiting is performed. bug:5138933 Change-Id: I382e2d8689f5189110226613c2387f553df98bd3
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
a31f5e63743f3fc788acbd85474ec80eaf5c8b4d |
|
28-Jul-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
InputFilter not updated on global accessibility change. Change-Id: I35fd5fa078840abf62a3d5eac66b2eae1a79d01d
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
4e2a762eae1f6981d32e6098a95498865ad7f795 |
|
27-Jul-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Not user generated accessibility events can change the interrogation allowing window. 1. Events not generated by the user can change the interrogation allowing window unpredicatably. For example when a ListView lays out its children it fires an accessibility events and changes the currently active window while the user interaction may be happening in another window say a dialog. Now the interrogation allowing window is changed when a new window is shown or the user has touch explored it. bug:5074116 Change-Id: I8dde12bbec807d32445a781eedced9b95312b3e2
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
00aabf7d187bc05408199bd687a538b2e68bdc17 |
|
21-Jul-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Touch exploration state set to clients asynchronously and depended on talking service being enabled. 1. Upon registration of an accessibility client the latter received only the accessiiblity state and waiting for the touch exploration state to be sent by the system in async manner. This led the very first check of touch exploration state is checked a wrong value to be reported. Now a state of the accessibility layer is returned to the client upon registration. 2. Removing the dependency on talking accessibility service to be enabled for getting into touch exploration mode. What if the user wants to use an accessibility service that shows a dialog with the text of the touched view? bug:5051546 Change-Id: Ib377babb3f560929ee73bd3d8b0d277341ba23f7
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
35bfedeaba724aeadc6f6c890269cb6bf7ef42f5 |
|
15-Jul-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Touch exploration separate setting and API to poll the latter state. 1. Seperated touch exploration to be a seperate setting rather being magically enabled by the system of accessiiblity is on the there is at leas one accessibility service that speaks enabled. Now there is a setting for requesting touch exploration but still the system will enabled it only if that makes sense i.e. accessibility is on and one accessibility service that speaks is enabled. 2. Added public API for checking of touch exploration is enabled. 3. Added description attribute in accessibility service declaration which will be shown to the user before enabling the service. 4. Added API for quick cloning of AccessibilityNodeInfo. 5. Added clone functionality to SparseArray, SparseIntArray, and SparseBooleanArray. bug:5034010 bug:5033928 Change-Id: Ia442edbe55c20309244061cd9d24e0545c01b54f
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
4a49d9fe52a76c7f91f81113e0a8e9f97ac3f24c |
|
17-Jul-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Throw an exception if accessibility service has not requested to retrieve window content. 1. If an accessibility service does not request access to the window content and does so, an exception is thrown to point the developer to the reason. bug:5038284 Change-Id: Ibf08f4d2c8ad8939c4f4c2e288048a4f8ff1e31b
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
63c04eeb00dd71380d5ebba701014793d8f9a2ea |
|
14-Jul-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Touch exploration gesture events change the window id. 1. Touch exploration start and end events are generated by the sytstem to provide additional information for accessibility services. Since such events do not come from any particular window they whould not change the id of the window that currently allows exploring its content. 2. Touch exploration start and end events were lealing the touch explorer class wich is private. bug:5026258 Change-Id: Icaf3e2bd9566716f2afb876cf8e0d50813b0c76e
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
e8f95352cf4fd2bb38fd124bf5fcefddddeb0692 |
|
07-Jul-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Addressing comments during security review of the interrogation feature. 1. Pretty much minor fixes for improving readability. Change-Id: I76efdcc574af9a277b54eeb9fb5caf34948fcfde
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
b96a41136a9afd8e719c19488094f641f9673362 |
|
28-Jun-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Fatal system crash due to NullPointerException 1. Due to thread interleaving it was possible that two messages are sent for requesting dispatch of the same accessibility event and since the first one sends the event and removes it from the pending list the second message pulls null during the event lookup. Look at the patch's comments for a detailed scenario and rationale of the fix. bug:4886129 Change-Id: If8b272ceaec7709c659ae502c3a730e63c939172
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
a0156177cdc809795dd8bc5a19943dd2b6f82b66 |
|
27-Jun-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Added scroll and text selection change accessibility events. 1. Added scrolling accessibility event to provicde feedback when a view is scrolled. Note: We need scroll events for ICS since even though we have touch exploration the user does not know when something is scrollable and not feedback is provided while scrolling. bug:4902097 2. Added a text selection change event to provide feedback for selection changes including cursor movement. Note: We need the text selection change events for ICS since even though the IME supports navigation in text fields the user receives no feedback for the current selection/ cursor position. bug:4586186 3. Added a scrollable property to both AccessibilityEvent and AccessibilityNodeInfo. The info has to describe the source in terms of all properties that make sense for accessibility purposes and the event has this property (kinda duplicated) since clients will aways want to know if the source is scrollable to provided clue to the user and we want to avoid pulling the info of the source for every accessibility event. Change-Id: I232d6825da78e6a12d52125f51320217e6fadb11
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
1ecc1e098a531f57707218b0b38a73f6fa2759d8 |
|
22-Jun-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Minor tweak I forgot in the presious change. Change-Id: I17854e3c824bea5b6c0ffeea45f986f0b5297054
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
7961be75f4161769f620e44dec3f5911e50ebe03 |
|
21-Jun-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
AccessibilityNodeInfo bounds inconsistent with compatibility mode. 1. In compatibility mode a window wide scaling is applied to stretch the content. However, AccessibilityNodeInfos retrieved from that window contain bounds in application's view of the world and need to be scaled to properly relect what a sighted user sees. Change-Id: Iebbb99526fc327f45b5cede89ba8c32e6ebd8845
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
9dc98e3d1764dbcf7828599b70e7a71a3c82a407 |
|
17-Jun-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Accessiiblity in right state upon connection of a test agent. 1. Enabling accessibility and disabling all enabled accessibility service when a test client connects the the AccessibilityManagerService. Change-Id: I2f40cccaa0035ac1454d8c5ac84678c1542a0229
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
eeee4d2c01d3c4ed99e4891dbc75c7de69a803fa |
|
11-Jun-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Final polish of the interrogation feature. 1. Added a new event type for notifying client accessibilitiy services for changes in the layout. The event is fired at most once for a given time frame and is delivered to clients only if it originates from the window that can be interrogated. 2. Exposed the findByText functionality in AccessibilityNodeInfo. This is very useful for an accessibility service since it allows searching for something the user knows is on the screen thus avoiding touch exploring the content. Touch exploring is excellent for learning the apps but knowing them search is much faster. 3. Fixed a bug causing an accessibiliby service not to receive the event source in case of more than one service is registered and one of them does not have paermission to interrogate the window. The same event was dispatched to multiple services but if one of them does not have interrogation permission the event is modified to remove the source causing subsequent serivices not to get the later. 4. Moved the getSource setSource methods to AccessibilityRecord instead in AccessibilityEvent. 5. Hiden some protected members in AccessibilityRecod which should not be made public since getters exist. 6. Added the View absolute coordinates in the screen to AccessibilityNodeInfo. This is needed for fast computation of relative positions of views from accessibility - common use case for the later. 7. Fixed a couple of marshalling bugs. 8. Added a test for the object contract of AccessibilityNodeInfo. Change-Id: Id9dc50c33aff441e4c93d25ea316c9bbc4bd7a35
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
8643aa0179e598e78d938c59035389054535a229 |
|
20-Apr-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Interrogation of the view hierarchy from an AccessibilityService. 1. Views are represented as AccessibilityNodeInfos to AccessibilityServices. 2. An accessibility service receives AccessibilityEvents and can ask for its source and gets an AccessibilityNodeInfo which can be used to get its parent and children infos and so on. 3. AccessibilityNodeInfo contains some attributes and actions that can be performed on the source. 4. AccessibilityService can request the system to preform an action on the source of an AccessibilityNodeInfo. 5. ViewAncestor provides an interaction connection to the AccessibiltyManagerService and an accessibility service uses its connection to the latter to interact with screen content. 6. AccessibilityService can interact ONLY with the focused window and all calls are routed through the AccessibilityManagerService which imposes security. 7. Hidden APIs on AccessibilityService can find AccessibilityNodeInfos based on some criteria. These API go through the AccessibilityManagerServcie for security check. 8. Some actions are hidden and are exposes only to eng builds for UI testing. Change-Id: Ie34fa4219f350eb3f4f6f9f45b24f709bd98783c
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
c321c19fc91bfec9a74039ed4d37b815dd4ef379 |
|
04-Jun-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Fixing a bug I introduced during my last merge. Change-Id: I6474ca94a6ef9bd17a5670a4baa51673f3c2f5d0
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
f9886f3ee9db23cc5a553b92e2aeff0ae842c915 |
|
11-May-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
AccessiblityManagerService not cleaning up dead binders. 1. AccessibilityManagerService was keeping handles to dead IAccessibilitymanagerClients - now doing so. 2. AccessibilityManagerService was lazily cleaning up dead IAccessibilityServiceConnections - now using a callback. 3. Cleaned up the book keeping of enabled services. 4. Fixed a bug that the input filter is still enabled when disabling accessibility. Change-Id: I5e9af7ab684a3b71e8ee51125b1262a17e960eb0
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
cc4053e031371456fe54d51bbad1db721db4ae38 |
|
23-May-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Accessibility serviceconfiguration via meta-data Note: This is a part of two CL change and contains the system changes without updates to the settings. 1. Added a mechanism for configuring an accessibility service via XML file specified in a meta-data tag (similar to IMEs). 2. Added property for specifying a settings activity for an accessibility service. 3. Refactored the APIs in AccessibilityManager to return lists of AccessiblityServiceInfo instead ServiceInfo since the former describes an AccessibilityService in particular (similar to IMEs). Change-Id: Ie8781bb7e0cdb329e583b6702a612a507367ad7b
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
a3e261d506551713477adb2fd47d1a65b94e64d8 |
|
27-Apr-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
The accessibility input filter state was not unregistered when no spoken feedback accessibility services are regitstered. bug:4343996 Change-Id: Ie709ac24baa084cf3627d6adb6caf1550c399f8b
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
736c2756bf3c14ae9fef7255c119057f7a2be1ed |
|
23-Apr-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Touch exploration feature, event bubling, refactor 1. Added an Input Filter that interprets the touch screen motion events to perfrom accessibility exploration. One finger explores. Tapping within a given time and distance slop on the last exlopred location does click and long press, respectively. Two fingers close and in the same diretion drag. Multiple finglers or two fingers in different directions or two fingers too far away are delegated to the view hierarchy. Non moving fingers "accidentally grabbed the device for the scrren" are ignored. 2. Added accessibility events for hover enter, hover exit, touch exoloration gesture start, and end. Accessibility hover events are fired by the hover pipeline. An accessibility event is dispatched up the view tree and the topmost view fires it. Thus predecessors can augment the fired event. An accessibility event has several records and a predecessor can optionally modify, delete, and add such to the event. 3. Added onPopulateAccessibilityEvent and refactored the existing accessibility code to use it. 4. Added API for querying the currently enabled accessibility services by feedback type. Change-Id: Iea2258c07ffae9491071825d966dc453b07e5134
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
3fb3d7c4e756bd32d5abde0abca9ab52d559bc84 |
|
23-Apr-2011 |
Adam Powell <adamp@google.com> |
Revert "Touch exploration feature, event bubling, refactor" This reverts commit ac84d3ba81f08036308b17e1ab919e43987a3df5. There seems to be a problem with this API change. Reverting for now to fix the build. Change-Id: Ifa7426b080651b59afbcec2d3ede09a3ec49644c
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
ac84d3ba81f08036308b17e1ab919e43987a3df5 |
|
05-Apr-2011 |
Svetoslav Ganov <svetoslavganov@google.com> |
Touch exploration feature, event bubling, refactor 1. Added an Input Filter that interprets the touch screen motion events to perfrom accessibility exploration. One finger explores. Tapping within a given time and distance slop on the last exlopred location does click and long press, respectively. Two fingers close and in the same diretion drag. Multiple finglers or two fingers in different directions or two fingers too far away are delegated to the view hierarchy. Non moving fingers "accidentally grabbed the device for the scrren" are ignored. 2. Added accessibility events for hover enter, hover exit, touch exoloration gesture start, and end. Accessibility hover events are fired by the hover pipeline. An accessibility event is dispatched up the view tree and the topmost view fires it. Thus predecessors can augment the fired event. An accessibility event has several records and a predecessor can optionally modify, delete, and add such to the event. 3. Added onPopulateAccessibilityEvent and refactored the existing accessibility code to use it. 4. Added API for querying the currently enabled accessibility services by feedback type. Change-Id: Iec03c6c3fe298de3f14cb6efdbb9b198cd531a0c
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
43a17654cf4bfe7f1ec22bd8b7b32daccdf27c09 |
|
07-Apr-2011 |
Joe Onorato <joeo@google.com> |
Remove the deprecated things from Config.java. These haven't been working since before 1.0. Change-Id: Ic2e8fa68797ea9d486f4117f3d82c98233cdab1e
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
0029c66203ab9ded4342976bf7a17bb63af8c44a |
|
30-Mar-2011 |
Jeff Brown <jeffbrown@google.com> |
Add input filter mechanism for accessibility. This patch adds a mechanism for capturing, filtering, transforming and injecting input events at a very low level before the input dispatcher attempts to deliver them to applications. At this time, the mechanism is only intended to be used by the accessibility system to implement built-in system-level accessibility affordances. The accessibility input filter is currently just a stub. It logs the input events receives and reinjects them unchanged, except that it transforms KEYCODE_Q into KEYCODE_Z. Currently, the accessibility input filter is installed whenever accessibility is enabled. We'll probably want to change that so it only enables the input filter when a screen reader is installed and we want touch exploration. Change-Id: I35764fdf75522b69d09ebd78c9766eb7593c1afe
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|
6e6cd7a5660af1a4b5a9ad091c41ef1c72ad2000 |
|
30-Mar-2011 |
Jeff Brown <jeffbrown@google.com> |
Move accessibility manager service to own package. Change-Id: I0d73737a21615f0ff0abc80b757fb8c9bcb50be5
/frameworks/base/services/java/com/android/server/accessibility/AccessibilityManagerService.java
|