Merge "Trust navbar panel and accessibility overlays" into oc-dr1-dev
am: b5f0e91c4c

Change-Id: Ia00e2b0cbaba95b3265171bbe1e13cb86813c511

Merge "Trust navbar panel and accessibility overlays" into oc-dr1-dev

Merge changes Id4647e54,I66b6edbd into oc-dr1-dev
am: bf72631a10

Change-Id: Ib1f5d7cc4841124c4f5fc1ea543ce3984dc26d16

Merge changes I9047b298,Id9554d6b into oc-dr1-dev
am: e6ffd49479

Change-Id: I8ac6a10b071dd5293dda9347d590b92c9cea0124

Merge changes Id4647e54,I66b6edbd into oc-dr1-dev

* changes:
  Fix VR surface attributes.
  DVR API: Implement support for deleting surface attributes.

Merge changes I9047b298,Id9554d6b into oc-dr1-dev

* changes:
  libdvr: Add performance API to platform library.
  libpdx_uds: Fix uninitialized variable bug.

Merge "Add VrFlinger dumpsys to SurfaceFlinger" into oc-dr1-dev
am: ae11e71bb7

Change-Id: Ic3ac3d8b16a14aed3fb3553616c4adec23044f7a

Merge "Add VrFlinger dumpsys to SurfaceFlinger" into oc-dr1-dev

Fix VR surface attributes.

VR surface attributes had two issues that prevented the full extent
of their use:
1. The older clang version missed that templated copy constructor
   and assignment operators do not override the default ones in
   the Variant class. This caused issues with certain types when
   copy constructing / assigning from another Variant of the same
   type. This was noticed by running tests with a newer version of
   clang which provided warnings.
2. C++ rules about implicit conversion to bool from types that
   decay to pointers causes subtle issues with Variants that have
   bool elements.

   For example this assignment compiles but produces the wrong
   result:

   const int array[3] = { 1, 2, 3};
   Variant<int, bool, std::array<int, 3>> variant = array;
   EXPECT_FALSE(variant.is<bool>()); // Actually true.

   Here the programmer might accidentally think that the std::array
   element of the variant can be assigned from the regular array.
   This doesn't work, but instead the compiler decays the array to
   a pointer and assigns the bool element to true.

The first issue is addressed by defining copy/move constructors /
assignment operators on Variant and deleting the default ones from
the internal Union type for extra safety.

The second issue is addressed by making a more restrictive version
of the std::is_constructible trait that rejects bool construction
from types that decay to pointers. Once this was put in place the
erroneous use cases no longer compiled and is fixed as part of
this CL.

Tests are updated to verify the fixes to these issues.

Bug: 62557221
Test: pdx_tests and dvr_api-test passes.
Change-Id: Id4647e54e0a7b1753217fe7fe351462fe5bcfd83

libdvr: Add performance API to platform library.

- Add dvrPerformanceSetSchedulerPolicy API. Only this API is exposed
  through the DVR platform library, the older API will be deprecated.
- Add permission checks to all performanced APIs.
- Allow services with android.permission.RESTRICTED_VR_ACCESS to change
  scheduler policy for VR apps.
- Minor updates to use the updated PDX service API. The older API will
  be deprecated soon.
- Add tests for permission checks and policy API.

Most of the tests are automatic however, there is one manual step for
testing the android.permission.RESTRICTED_VR_ACCESS (e.g. trusted uid)
check. Because there is no reliable way to determine the UID of VrCore
from the gtest, instead the gtest looks for an env var named
GTEST_TRUSTED_UID; if this is set the value is used as a uid and the
trusted uid tests are enabled.

Bug: 62468109
Test: 'GTEST_TRUSTED_UID=<VrCore UID> performance_service_tests' passes.
Change-Id: I9047b298a015a69535b655a299ca26c179e2d57d

DVR API: Implement support for deleting surface attributes.

- Implement support for deleting surface attributes using the "NONE"
  attribute type.
- Add tests for attribute events and attribute deletion.

Bug: 62456002
Test: dvr_api-test passes.
Change-Id: I66b6edbd35077596d89e85829bcbe7c52829ef5b

Add VrFlinger dumpsys to SurfaceFlinger

When we do dumpsys SurfaceFlinger, we get no information about the
layers/what is composed when the device is in VR mode.

Bug: 63113212
Test:
VrFlinger state:
Application Surfaces:
Surface 0: surface_id=11 process_id=5550 user_id=10104 visible=1
z_order=0 queue_ids=29,37
Surface 1: surface_id=13 process_id=5035 user_id=10130 visible=1
z_order=0 queue_ids=53,65

Direct Surfaces:
Surface 0: surface_id=8 process_id=5563 user_id=10104 visible=1
z_order=0 queue_ids=17

Display metrics:     1440x2880 537.882x537.882 dpi @ 60 Hz
Post thread resumed: 1
Active layers:       1

Layer 0: type=Device surface_id=8 buffer_id=19

Hardware Composer Debug Info:
-------------------------------
HWC2 display_id: 0
layer:   30 z: 0 compositon: Device/Device alpha: 255 format:
RGBA_8888_UBWC dataspace:0x00000000 transform: 0/0/0 buffer_id:
0x70b4877500
color modes supported: 0 7 9 current mode: 7
current transform:
  1.08  -0.02  -0.02   0.00
  -0.07   1.03  -0.07   0.00
  -0.01  -0.01   1.09   0.00
  0.00   0.00   0.00   1.00
-------------------------------

This section doesn't appear when the device is not in VR mode.

Change-Id: I2961a05fc3ea303e070be08de355fb6e56c3d0db
Signed-off-by: Karthik Ravi Shankar <karthikrs@google.com>

Merge "Add telephony related dumpsys in the telephony monitor bugreports" into oc-dr1-dev
am: d16d297825

Change-Id: Id02616a3d6084a18bdf6a3b87868eb1e1f905a9a

Merge "Add telephony related dumpsys in the telephony monitor bugreports" into oc-dr1-dev

libpdx_uds: Fix uninitialized variable bug.

In this code path epoll_wait() returns 0 when the operation times
out, but there is no condition to catch this and return the
approptiate status. This allows the event value to be used
uninitialized, which returns a junk pending event mask.

This bug results in BufferHub queues sometimes thinking that the
producer side of the buffer hung up, when the junk value happens
to have the EPOLLHUP bit set.

Bug: 62886596
Test: Observe VR app switching stability.
Change-Id: Id9554d6bf224fd27815ff042ac145f59041d0aae

Add telephony related dumpsys in the telephony monitor bugreports

Collect dumpsys for the following in telephony monitor auto
generated bugreports for better debugging

1) TelephonyDebugService
2) CarrierConfig
3) Connectivity

Test: Verified telephony monintor auto generated bugreport

Bug: 62504502
Change-Id: Ia2a9e1ff5b289dec7a4d0bb4e8ef85fb413df87a

Merge "Merge "SurfaceFlinger: Last minute log removal." into oc-dev am: 56cca289cc am: d5547979ec" into oc-dr1-dev-plus-aosp

Merge "SurfaceFlinger: Last minute log removal." into oc-dev am: 56cca289cc
am: a75a975dc8

Change-Id: I1b4544d48d3248edc5621af1218b38a3ad0d9cae

Merge "SurfaceFlinger: Last minute log removal." into oc-dev am: 56cca289cc
am: d5547979ec

Change-Id: I1a6c13f9fd00d082425b374e440e5d8482260b73

Merge "SurfaceFlinger: Last minute log removal." into oc-dev
am: 56cca289cc

Change-Id: Ib4448b62b985fac82c24fb81aef550ba7d0280a9

Merge "SurfaceFlinger: Last minute log removal." into oc-dev
am: 56cca289cc

Change-Id: I07e2b1c1efd0cc8eec73467fdd4ef88bf36f081c

Merge "SurfaceFlinger: Last minute log removal." into oc-dev

Merge "libgui: New mutex for ConsumerBase frame callbacks" into oc-dev am: fb672ba37b
am: f64e75d2f0

Change-Id: Icf927506ab83ac03c754658625520eca4cd1b529

Merge "libgui: New mutex for ConsumerBase frame callbacks" into oc-dev am: fb672ba37b
am: 1813e093bd

Change-Id: I9ca3c8a6bb058177cf47e8d0f8ca5a38d003c686

Merge "libgui: New mutex for ConsumerBase frame callbacks" into oc-dev
am: fb672ba37b

Change-Id: I90f2ebe102c40c70620ca12bda5000b455109d26

Merge "libgui: New mutex for ConsumerBase frame callbacks" into oc-dev
am: fb672ba37b

Change-Id: I14c99a3137f086eca4e1c1ab2f8877ae6df88a4a

Merge "libgui: New mutex for ConsumerBase frame callbacks" into oc-dev

SurfaceFlinger: Last minute log removal.

Whoops.

Test: Log removed!
Change-Id: I4593352cf565603a4507c6ef13504b167d882eec

Merge "resolve merge conflicts of 40da4609e to oc-dr1-dev-plus-aosp" into oc-dr1-dev-plus-aosp

Merge "Fix hidl_ssvc_poll thread issues" into oc-dr1-dev
am: d8ef90752b

Change-Id: I713984531740a01f33335d516393a522b662d53c

Merge "Fix hidl_ssvc_poll thread issues" into oc-dr1-dev

resolve merge conflicts of 40da4609e to oc-dr1-dev-plus-aosp

Test: I solemnly swear I tested this conflict resolution.
Change-Id: I0c4a9e0648674b78391e257f6a94bd1e325e1964

Merge "libbinder: really make it available to vendors" am: 2bb661ca09
am: 99daab345f

Change-Id: Ief871803389339a3d423787502535c824f35e74e

Merge "libbinder: really make it available to vendors"
am: 2bb661ca09

Change-Id: I58c2069544d39337b3a054a29fe8fdf98d00a33e

Merge "libbinder: really make it available to vendors"

Trust navbar panel and accessibility overlays

Fixes: 62048165
Fixes: 62341768
Test: Enable maximum screen display and font size, enable TalkBack.
Enable Select to Speak with FAB, enable TalkBack.

Change-Id: Ia96373200adf9c5d306e9c61da9c65ba412c9430

Fix hidl_ssvc_poll thread issues

1) Patch poll wake up handling logic so that thread no longer quits
   on spurious wake up.
2) Patch(simplify) thread initialization, so that it will not hang
   on waiting for conditional variable.

Test: 1) send signal 3 to system_server, does not observe thread quit.
Test: 2) adb shell "stop;start" many times, every time thread starts.

Bug: 62933449
Bug: 62933799

Merged-In: Icb7477e9d75338b9742e144fc2687d5ced91e89e
Change-Id: Icb7477e9d75338b9742e144fc2687d5ced91e89e

libbinder: really make it available to vendors

Fixed a bug that vendor variant of the lib is not built due to missing
global include path.

Bug: 36426473
Bug: 36079834
Test: BOARD_VNDK_VERSION=current m -j libbinder.vendor successful

Change-Id: Ice32a23023d7fd9b78c084c3bef455f7008fe4da

libgui: New mutex for ConsumerBase frame callbacks

Adds a new mutex to protect ConsumerBase's mFrameAvailableListener
separately from the main mMutex. This prevents deadlocks when trying
to perform callbacks in response to other calls that come through
ConsumerBase.

Bug: 62915791
Test: EncodeVirtualDisplayWithCompositionTest#
          testRenderingMaxResolutionRemotely

Change-Id: I244dc2d83488e452d9690c0d7847c9ff15a95dbb

Merge "libgui: export EGL headers." into oc-dr1-dev
am: be2e363c9e

Change-Id: Id87eb01463f03ba0bcbe4948a93f3add60fc0b7e

Merge "libgui: export EGL headers." into oc-dr1-dev

Merge "Revert "libui: Load a.h.graphics.mapper passthrough service on library load"" into oc-dr1-dev
am: 274f72eb9b

Change-Id: Ie94b3b6d08c4ca0a13902fd4ec71e4bc7808831f

Merge "Revert "libui: Load a.h.graphics.mapper passthrough service on library load"" into oc-dr1-dev

Revert "libui: Load a.h.graphics.mapper passthrough service on library load"

This reverts commit 75ca86f072f82d38b21e0a13234ecbc2c24cb6ab.

avc: denied { read } name="hw" scontext=u:r:cameraserver:s0
tcontext=u:object_r:system_file:s0 tclass=dir
avc: denied { read } name="hw" scontext=u:r:folio_daemon:s0
tcontext=u:object_r:system_file:s0 tclass=dir
avc: denied { read } name="hw" scontext=u:r:hal_imsrtp:s0
tcontext=u:object_r:system_file:s0 tclass=dir
avc: denied { read } name="hw" scontext=u:r:idmap:s0
tcontext=u:object_r:system_file:s0 tclass=dir
avc: denied { read } name="hw" scontext=u:r:mediaextractor:s0
tcontext=u:object_r:system_file:s0 tclass=dir
avc: denied { read } name="hw" scontext=u:r:mediametrics:s0
tcontext=u:object_r:system_file:s0 tclass=dir
avc: denied { read } name="hw" scontext=u:r:webview_zygote:s0
tcontext=u:object_r:system_file:s0 tclass=dir

Bug: 62353585
Bug: 62912747
Bug: 35197529
Test: build and boot device. Launch camera, take pictures.
Change-Id: Ic6884f7235edb9191034a285414eb93aa44ac23c

Merge "Make VR HWC DPI a configurable property" into oc-dr1-dev
am: 281a0c26c0

Change-Id: I3ec8cc3bea80ab946c7576f9de2c470ace2fb72f

Merge "Make VR HWC DPI a configurable property" into oc-dr1-dev

Merge "IWYU RegionHelper += <limits>" am: bbf3096e22 am: f08008db9f
am: 836513bfbf

Change-Id: I164150132bd6518e52894823b10d108e86eed81b

Merge "IWYU RegionHelper += <limits>" am: bbf3096e22
am: f08008db9f

Change-Id: Ida5d67d96a8c3eb87f25202872b2c30ed5b6c25e

Merge "IWYU RegionHelper += <limits>"
am: bbf3096e22

Change-Id: I9dd08e98eee2924554b623f755869fac35a80605

Merge "IWYU RegionHelper += <limits>"

Disable setting saved_cmdline_size am: ba54d5baf4
am: acc79d7cfe

Change-Id: I8be566b029a6ec93334e7a4dd37eec12846b1f60

Disable setting saved_cmdline_size am: ba54d5baf4
am: 64dac11cc0

Change-Id: I293be9ed5486024c479a0ef133086f825e259048

Disable setting saved_cmdline_size
am: ba54d5baf4

Change-Id: Icd17c5257c03fd9e33114c5760ece6c21c74d025

Disable setting saved_cmdline_size
am: ba54d5baf4

Change-Id: I447ad42ad4c245013e488701a61d8ce2348a368d

IWYU RegionHelper += <limits>

Test: compiles when transitive dependency is removed
Bug: 35363681
Change-Id: Ia405e41102e6ded07720e424167df7e4d080581e

Disable setting saved_cmdline_size

Minimal revert of the addition of increasing
the saved_cmdline_size. This change
exposed kernel bugs that prevented
atrace from working.

Bug: 62375794
Test: manual
Change-Id: Ie946b9246ce9ba22af59dbf0b9fe8a59a5fa1f3f

EGL: add color space validation
am: 15f79786e1

Change-Id: I05600bda5553b6e5f49a055f30cfef60db4fa9e8

libgui: export EGL headers.

Many libgui headers use these headers so not exporting
these breaks BOARD_VNDK_VERSION=current builds.

Test: headers resolved with BOARD_VNDK_VERSION=current
Change-Id: I085faeda4f8ad8a07d91e81222b8031a0dd1858a

EGL: add color space validation

We want Android to support multiple color spaces as
that can be implemented entirely in the eglapi.cpp wrapper.
These additional dataspaces do not require special behavior
by the driver and may not recognize EGL_GL_COLORSPACE_DISPLAY_P3_EXT,
EGL_GL_COLORSPACE_DISPLAY_P3_LINEAR_EXT, and related attributes.
Therefor, need to filter out those attributes before calling
the driver.
In the future there may be new extensions that do require
driver support. When that happens we'll need to update
stripColorSpaceAttribute to treat that correctly.

Test: Android CTS
      adb -d shell am start \
      -n com.drawelements.deqp/android.app.NativeActivity \
      -e cmdLine '"deqp --deqp-case=dEQP-EGL.functional.wide_color.* \
      --deqp-log-filename=/sdcard/dEQP-Log.qpa"'
Bug: 62424735
Change-Id: I0f0867feb73055100636d326213183ef1f052b7c
(cherry picked from commit eeaa52bc65b5cfaca1f0de44635b57919179df29)

Merge "surfaceflinger: discard stale commands" into oc-dev am: f8ca9a0875
am: 3addef29f7

Change-Id: Ica0906d38bbf2df75f10a6108600c3c1646ab80d

Merge "LibGUI remove DEBUG" into oc-dev am: be597daebd
am: 45bf98c36e

Change-Id: Ic223a6dfb10324fbdd2b2609072a0088eff4fbd9

Disable skip Validate on client composition am: 269c236109
am: 0ff681ca77

Change-Id: I7331077ef129e55aed38f10f19732a22eb842bd6

Merge "surfaceflinger: discard stale commands" into oc-dev am: f8ca9a0875
am: 49cdc635a2

Change-Id: I84da842894b25f11b665bd3b4adf597b3aee94ef

Merge "LibGUI remove DEBUG" into oc-dev am: be597daebd
am: d9517fe958

Change-Id: Id677c4ba2c28922c4a3f64fdc2b114d3e3031dbd

Disable skip Validate on client composition am: 269c236109
am: 89283e56d6

Change-Id: Id8e05d219ef73616cdef7152008b487050f25373

Merge "surfaceflinger: discard stale commands" into oc-dev
am: f8ca9a0875

Change-Id: I6b7ce1798d2fe753c577ff3baa954283c372b80c

Merge "LibGUI remove DEBUG" into oc-dev
am: be597daebd

Change-Id: Iece9f1428655c8674cf62a92ecb0a088ad74b629

Disable skip Validate on client composition
am: 269c236109

Change-Id: I4c0a4a90ff0ca2e39075ad643d1892597301d3fd

Merge "surfaceflinger: discard stale commands" into oc-dev
am: f8ca9a0875

Change-Id: Ief69f6fcf0c738612b14b94d65b3ae1555973d6c

Merge "LibGUI remove DEBUG" into oc-dev
am: be597daebd

Change-Id: I65756ce89b4a10d2df403f9f9a4ce2cacdaafda0

Disable skip Validate on client composition
am: 269c236109

Change-Id: Ib0d2f8a9e59cf4618ed029fe35391f566d11bcfb

Merge "surfaceflinger: discard stale commands" into oc-dev

Merge "LibGUI remove DEBUG" into oc-dev

Disable skip Validate on client composition

SurfaceFlinger should not skip validate when GLComposition is
needed. This can happen when SF decides to force GLComposition
(like when a screen rotation occurs).

Bug: 62906801
Test: Manual
Change-Id: I248146eade395a124cb149773287abd88ae36511

surfaceflinger: discard stale commands

When we skip validate, we may have stale commands on the command
buffer.  They should be discarded.

Bug: 62906801
Test: manual
Change-Id: Ie7921ba75ab2252b834cda6924e1fcfdf236f759

LibGUI remove DEBUG

DEBUG_ONLY_CODE flag enables expensive checks such as
BufferQueueCore::validateConsistencyLocked() which when running
represents 15.1% of SurfaceFlinger total runtime. Enabling these
check for "eng" build only make "userdebug" build closer to "user".

Test: Manual
Bug: 62905241
Change-Id: I77065ecf4e62261c3f02b1cad9966d5c4845c639

Merge "Skip Validate am: 249c0ae80a am: 9837ce8210" into oc-dr1-dev-plus-aosp

Skip Validate am: 249c0ae80a
am: 9837ce8210

Change-Id: If81e98cd140426c5cc33595205fd9d4c26bef044

Skip Validate am: 249c0ae80a
am: 8976c6ae37

Change-Id: I400dfd8d484779e5523353b002164f4f5ea41e03

Skip Validate
am: 249c0ae80a

Change-Id: I336fce473d21e5fdfb80ad8eb48f1df5a24497d8

Skip Validate
am: 249c0ae80a

Change-Id: I55a564d600a422915f43111f4e02a10857512fcf

Skip Validate

b/37474580
b/62806392

Test: marlin, ryu

Change-Id: I09e0c52cb7c914dcd883dc771d97a365c89f7037

Merge "Update dumpstate binary file name" into oc-dr1-dev
am: 8d897e6003

Change-Id: Idfd13cc794eb170c16d203f7f8b78077ac37db80

Merge "Update dumpstate binary file name" into oc-dr1-dev

Merge changes from topic 'vendor_available_libgui' into oc-dr1-dev
am: 8c72947af1

Change-Id: Ie7315f7aea25663576402d04a58bbeaa9662e3c4

Merge changes from topic 'vendor_available_libgui' into oc-dr1-dev

* changes:
  Make libgui available to vendors
  Mark EGL/GLES libs as vendor_available
  Change export_include_dirs into export_header_lib_headers

Merge "Check key before edit value in countFlushCompleteEventsLocked" into oc-dr1-dev
am: 8669acc97d

Change-Id: I5fa61c3ffa5f41a5093dba86ad255687d29466b4

Merge "Check key before edit value in countFlushCompleteEventsLocked" into oc-dr1-dev

Update dumpstate binary file name

Keep modem_log_all.tar for a while for vendors

Bug: 62865279
Test: Verify file name in bugreport
Change-Id: Ia64f6c8e978877040eecfbc6066f0d741d91ddf3

Merge "Add DVR method to get native display metrics" into oc-dr1-dev
am: f412b2b9fc

Change-Id: I4cfecf4ccc8bd5a8c3b9c3c11d885ac0233b71d3

Merge "Add DVR method to get native display metrics" into oc-dr1-dev

Merge "Avoid SELinux violation at vndservicemanager" into oc-dr1-dev
am: 5e6abb3f0a

Change-Id: Id757814f2d04cdbbe66f5dcc30fcf34f14338aa2

Merge "Avoid SELinux violation at vndservicemanager" into oc-dr1-dev

Make libgui available to vendors

libgui is now available to vendors, especially for BufferQueue in
between the OMX and Camera HALs for encoding the captured video stream.

In doing so, its headers were moved from frameworks/native/include to
its local directory frameworks/native/libs/gui/include. This is required
because global include path is not supported when building for vendor.
However, in order not to affect platform modules that are still relying
on the global include path, symlinks are provided.

frameworks/native/include/gui -> frameworks/native/libs/include/gui
frameworks/native/include/private/gui ->
frameworks/native/libs/include/private/gui

Bug: 37731063
Test: BOARD_VNDK_VERSION=current m -j libgui.vendor
Change-Id: I0bab8d2e31959085ad58b1ce4c610334ee3da033

Mark EGL/GLES libs as vendor_available

EGL/GLES libs are marked as vendor_available, otherwise vendors are not
able to use OpenGL. The libs were not simply declared as LL-NDK because
doing so causes the dual loading of libui.so in vendor processes;
platform version of libui.so is loaded since it is depended by libEGL.so
whereas there also is vendor version of libui.so in /system/lib/vndk.

When libEGL.so is built for vendors, 1) libgraphicsenv is removed from
its dependency list as the lib must not be available to vendors and 2)
eglGetNativeClientBufferANDROID is unusable since the function relies on
AHardwareBuffer_to_ANativeWindowBuffer which isn't available to vendor.

Bug: 37731063
Test: BOARD_VNDK_VERSION=current m -j libEGL.vendor
Test: BOARD_VNDK_VERSION=current m -j libGLESv2.vendor

Change-Id: I83a128b14a5d852bb0f2cd821a8e3f82c66a313f

Change export_include_dirs into export_header_lib_headers

Remove the duplicated information that headers for libbinder is in the
'include' dir.

Test: m -j
Change-Id: I5527c2709ff9d3a996c619f973345e61407cd3f0

dumpstate: Add the 2nd fd for binary file
am: 9fbfad0b6d

Change-Id: I6c8111188e7fdb8b4b6282aa3511a4d7187cd69e

Merge "Move include/binder to libs/binder/include" am: d46d9f1243 am: b212140bbf  -s ours
am: 7f683a8b8f  -s ours

Change-Id: I0a02efc1cac647857ba32235fac2c160d7dcb8db

Merge "Move include/binder to libs/binder/include" am: d46d9f1243
am: b212140bbf  -s ours

Change-Id: If344d9c14b3cf767956d55e0dd5d13b5a700c7a4

Merge "Move include/binder to libs/binder/include"
am: d46d9f1243

Change-Id: I41744e18ca3731b3027cb459b97945270a4d94ef