diff options
| author | Tyler Gunn <tgunn@google.com> | 2015-06-12 09:26:45 -0700 |
|---|---|---|
| committer | Tyler Gunn <tgunn@google.com> | 2015-06-12 09:39:47 -0700 |
| commit | 84f381b4eb9880b929ac40286b17b3f16271666b (patch) | |
| tree | 683e5d84c3c1ea9dd93e2d3ab1f661db6828bace /tools/aapt2/diff/Diff.cpp | |
| parent | 3d1f7c26be89a252c6e83c0b813df27295d06212 (diff) | |
Correct issues with VideoProvider discovered via CTS tests.
While authoring the CTS tests I discovered a few issues with the
VideoProvider.
1. The VideoProvider's default handler should be on the main Looper,
similar to other Telecom components.
2. When calling IVideoCallbacks, the try/catch for RemoteExceptions was
around the for-loop which iterates over the various VideoCallbacks. If
one was to fail, the other VideoCallbacks would not be called.
3. The mVideoCallbacks hashmap should use a ConcurrentHashMap; in CTS
tests I ran into a few concurrency exceptions related to the various
InCallServices registering callbacks while other methods on the provider
were being called.
Bug: 21802841
Change-Id: Ib0d46daf03554309044e9efaa991a15cb2c4b46b
Diffstat (limited to 'tools/aapt2/diff/Diff.cpp')
0 files changed, 0 insertions, 0 deletions