diff options
| author | Abhijeet Kaur <abkaur@google.com> | 2019-03-29 17:04:48 +0000 |
|---|---|---|
| committer | Abhijeet Kaur <abkaur@google.com> | 2019-04-11 18:11:29 +0100 |
| commit | 86311f8a53b146e178f3342ba1a2d08a09f0f9fb (patch) | |
| tree | 16d1405e0f5de58202d3a2dfdb73282fbaed9b46 /tools/aapt2/java/JavaClassGenerator_test.cpp | |
| parent | dc7b7de85c60158f2ed514b245310dd2537f264f (diff) | |
Pass valid file descriptor to binder
screenshotFd needs to be optional in Bugreport API. For some bugreports
such as wifi,telephony, interactive etc taking a screenshot is not
required.
Initially, the API was sending invalid file descriptor to the Binder, but
that binder transaction could not be completed as Binder validates the
file descriptor to be valid and not-null.
Adding a tmp.png screenshot file to pass to bugreport API call. In a
separate CL in frameworks/native, added check that the bugreports that
don't require screenshots would not use this file descriptor value.
Bug: 128981582
Test: Tested by taking bugreports using Bugreport API in shell
Merged-In: I3233f5753506ae159c9fa591742e6b99e361039b
Change-Id: I3233f5753506ae159c9fa591742e6b99e361039b
(cherry picked from commit b462e2e49964ed500b49ad07cbe80d96866d61bb)
Diffstat (limited to 'tools/aapt2/java/JavaClassGenerator_test.cpp')
0 files changed, 0 insertions, 0 deletions