diff options
| author | Phil Weaver <pweaver@google.com> | 2018-03-30 17:52:34 -0700 |
|---|---|---|
| committer | Phil Weaver <pweaver@google.com> | 2018-04-04 04:49:12 +0000 |
| commit | 985617969f559334b2bab7767cff9098d51d2b9e (patch) | |
| tree | 9e1f2060d13d51cbf37af90ea44f0d0f78d06d93 /tools/aapt2/diff/Diff.cpp | |
| parent | f0e74ae8ac115f415ec1e3f104ea688c55a4dddb (diff) | |
Don't clear a11y focus with lock held
The bug I'm fixing is kind of weird, but in the trace it
showed lock contention with onWindowsForAccessibilityChanged
holding the lock for a long time. So I've cleaned up that
method a bit, looking for trouble. The only concrete thing
I see that's wrong is that I was clearing a11y focus, which
involves calling the app that last reported having focus.
That was done with the lock held. Now doing it through the
handler.
Bug: 77298850
Test: Ran a11y CTS and unit tests.
Change-Id: I667707412a0e3ba98639d0cb43858e8c6b52a5a7
Diffstat (limited to 'tools/aapt2/diff/Diff.cpp')
0 files changed, 0 insertions, 0 deletions