Merge "Add GraphicBufferMapper::preloadHal"

Merge "Add VK_COLOR_SPACE_EXTENDED_SRGB_NONLINEAR_EXT."

Merge "Synchronous resource recover  mechanism for ISensorEventConnection" into oc-dr1-dev am: 0db05d3448
am: 4131f89cdc

Change-Id: I2ecb70e3cee09ab533cff12161e234e62aed7640

Merge "Synchronous resource recover  mechanism for ISensorEventConnection" into oc-dr1-dev
am: 0db05d3448

Change-Id: I21f2f66a84de9d5737e9c41c2cfff7f6295c3c12

Merge "Synchronous resource recover  mechanism for ISensorEventConnection" into oc-dr1-dev

Merge changes from topic 'svcmgr_shutdown_critical' into oc-dr1-dev am: 5ebcaf1bb0
am: 546aa9b19b

Change-Id: I4fb053426ef70e6012ee47797d2a368bdf633ba9

Merge changes from topic 'svcmgr_shutdown_critical' into oc-dr1-dev
am: 5ebcaf1bb0

Change-Id: I2e51acc6d0c83fd8d3865cdae676e33111d494ee

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

* changes:
  servicemanager: Mark servicemanager as shutdown critical
  servicemanager: Mark vndservicemanager as shutdown critical

Merge "Fix BufferHub state machine to return errors on invalid transitions." into oc-dr1-dev am: 2b7b5c34d3
am: 2c65dd96dc

Change-Id: Ibc6bc20f7f7ed2d5807c2632ba8deaca25ca4fd1

servicemanager: Mark servicemanager as shutdown critical

Bug: 62951929, 63514363, 62845700
Test: Build
Change-Id: Ib1b7aec4b6d3c590eb469daeed40b954d37efa18

Merge "Fix BufferHub state machine to return errors on invalid transitions." into oc-dr1-dev
am: 2b7b5c34d3

Change-Id: I311d3fa978ca1a7d74bc6403be59705dd9ea995e

servicemanager: Mark vndservicemanager as shutdown critical

vndservicemanager needs to be alive during shutdown to allow vndk
components which may run during shutdown to communicate. The system
ndk servicemanager keeps running throughout shutdown, so this brings
the vndservicemanager on par with that behavior.

Bug: 62951929, 63514363, 62845700
Test: Build
Change-Id: Iba154820ef69c6491a5cb05747b6b905763a825d

Merge "Fix BufferHub state machine to return errors on invalid transitions." into oc-dr1-dev

Synchronous resource recover  mechanism for ISensorEventConnection

Add synchronous destroy() function to recover resource used by
remote ISensorEventConnection object.

Bug: 63542033
Test: SensorDirectReportTest pass
Change-Id: If98782ee12c7b1a733eb15a2fd8d7c5dacde243b

Merge "Make default permission checks in performance service more restrictive." into oc-dr1-dev am: 9481328acc
am: 0db2b19500

Change-Id: I750969fed641d36b9a3d4f6ea2c2c664d68c1f41

Merge "Fix concurrent access bug in VrFlinger." into oc-dr1-dev am: d90e5399a4
am: 4a20620e2b

Change-Id: I235b954716ed8f73d22391977b7ea6718f06a028

Merge "Make default permission checks in performance service more restrictive." into oc-dr1-dev
am: 9481328acc

Change-Id: I813414e808bc1f7a04ad4f457f291bffddfad43f

Merge "Fix concurrent access bug in VrFlinger." into oc-dr1-dev
am: d90e5399a4

Change-Id: I73730c662401aded83ffcae6430937a2472ae071

Merge "Make default permission checks in performance service more restrictive." into oc-dr1-dev

Merge "Fix concurrent access bug in VrFlinger." into oc-dr1-dev

Merge "Clear callbacks when destroying DvrHwcClient" into oc-dr1-dev am: 222b760089
am: cf14ce0995

Change-Id: I955f9dfa14e9772103fd7fc7a00a29757228f26a

Merge "Clear callbacks when destroying DvrHwcClient" into oc-dr1-dev
am: 222b760089

Change-Id: Ic4344b56bf1ea766114bf6a0faaeb0b47c064651

Merge "Clear callbacks when destroying DvrHwcClient" into oc-dr1-dev

Merge "Validate the size of all dex paths eagerly" into oc-dr1-dev am: c3be65b533
am: 7d70bbf154

Change-Id: I558a6faa56d1d57570f0f447ec30fa65b38e989d

Merge "dvrReadBufferQueueDequeue: allow empty metadata" into oc-dr1-dev am: 3a674573f3
am: 33ed00b331

Change-Id: I9ffaf735a3aeffe1f452a266f5f526c4cf9e63ab

Merge "Validate the size of all dex paths eagerly" into oc-dr1-dev
am: c3be65b533

Change-Id: I0f37ecbe7f47fc011b62f366f88d38c3cffa7908

Merge "dvrReadBufferQueueDequeue: allow empty metadata" into oc-dr1-dev
am: 3a674573f3

Change-Id: Ib9ce46939355fdb46802fe83741a6c10f3c8a4df

Merge "Validate the size of all dex paths eagerly" into oc-dr1-dev

Merge "Include tracking state bits to DvrPose." into oc-dr1-dev am: 0d346a0cb1
am: 57056ecaaa

Change-Id: I7d9f0423278aac160b1ec47ef0a51f7cabd7063c

Merge "dvrReadBufferQueueDequeue: allow empty metadata" into oc-dr1-dev

Merge "Include tracking state bits to DvrPose." into oc-dr1-dev
am: 0d346a0cb1

Change-Id: I1fe407462296a93ba23cc1d00fe95379cae21e05

Merge "Include tracking state bits to DvrPose." into oc-dr1-dev

Merge "Check wide-color support before adding extensions"

Merge "Mark media_plugin_headers as vendor_available"

Merge "vulkan: update Vulkan headers to 1.0.53" am: 38bcc110b8 am: d0b439e33a am: 06ac13f0e5
am: 4245bd827e

Change-Id: I210790b562a9ed4a46ca11789538f60d13fe6947

Merge "vulkan: update Vulkan headers to 1.0.53" am: 38bcc110b8 am: d0b439e33a
am: 06ac13f0e5

Change-Id: Ie5587cd9ad0dee432ae3e50eff347a4d7d8602cd

Merge "vulkan: update Vulkan headers to 1.0.53" am: 38bcc110b8
am: d0b439e33a

Change-Id: I6f5b93b8b6a56e90a51d8779c7d76a4ac321cf9a

Merge "vulkan: update Vulkan headers to 1.0.53"
am: 38bcc110b8

Change-Id: I986ac1445697fb93fd72ab196087145a6b9945a0

Merge "vulkan: update Vulkan headers to 1.0.53"

Merge "Add target binderDriverInterfaceTest_IPC_32" am: 86c1fa9527 am: 940286e38f am: ceb942b99f
am: cf7de46622

Change-Id: Ie2e8455e90d94f57993e302451317c21b4e21537

Merge "Add target binderDriverInterfaceTest_IPC_32" am: 86c1fa9527 am: 940286e38f
am: ceb942b99f

Change-Id: If0c39eda327dc49e08ab13dc6e0da2038c6ae815

Merge "Add target binderDriverInterfaceTest_IPC_32" am: 86c1fa9527
am: 940286e38f

Change-Id: Iaefd8b3a1e9c0f4bde0b3e04342233be93eb7b18

Merge "Add target binderDriverInterfaceTest_IPC_32"
am: 86c1fa9527

Change-Id: I86a28976f575c314bcd74cb8edc18334ced9c575

Merge "Add target binderDriverInterfaceTest_IPC_32"

Mark media_plugin_headers as vendor_available

The headers should be available to vendors since vendors implement the
headers.

Bug: 37342627
Test: BOARD_VNDK_VERSION=current m -j camera.device@1.0-impl.vendor
Change-Id: I9936764039f2a5388707bdc093c869ec3551faad

Add VK_COLOR_SPACE_EXTENDED_SRGB_NONLINEAR_EXT.

Bug: 63077212
Test: CTS basicExtensionTest

Change-Id: I9c9b5de99249a6fdd49df1995971c14915f6a7c8

Check wide-color support before adding extensions

Don't want applications seeing the wide-color EGL extensions
if the device or display cannot support wide-color.
Bug: 63170158
Test: adb shell /data/nativetest/test-opengl-gl2_basic/test-opengl-gl2_basic
Verify that EGL_EXT_gl_colorspace_scrgb, EGL_EXT_gl_colorspace_scrgb_linear,
EGL_EXT_gl_colorspace_display_p3_linear and EGL_EXT_gl_colorspace_display_p3
are not present on devices that do not support wide-color, e.g. Nexus 6P

Change-Id: I46a26a67f2d6da9c5aad50d884ef02a62ccb6945

Add target binderDriverInterfaceTest_IPC_32

Add the target binderDriverInterfaceTest_IPC_32 which tests 32-bit
kernel and is independent from product_variables.

Bug: 62124459
Test: vts-tradefed run commandAndExit vts -m VtsKernelBinderTest
Change-Id: Ifad9ef24c35060179650955e24d1f046af7b7385

vulkan: update Vulkan headers to 1.0.53

Key changes since 1.0.49:
* Rename some extension enumerants to conform to naming style
* Add VK_EXT_sampler_filter_minmax
* Add VK_EXT_blend_operation_advanced
* Add VK_AMD_texture_gather_bias_lod
* Add VK_AMD_gpu_shader_int16
* Add VK_NV_fragment_coverage_to_color
* Add VK_NV_framebuffer_mixed_samples
* Add VK_NV_fill_rectangle

Test: build
Change-Id: I6b7931f4b7dfe4cae4fbc85fa961249534ccc11e

Merge "vulkan: update Vulkan headers to 1.0.49" am: fa3c1590b9  -s ours am: 08fa90eaab  -s ours am: 56d1521b02  -s ours
am: f1eec615c8  -s ours

Change-Id: I4699ddba3a479d19049e8787a3dfce38570a50a4

Merge "vulkan: update Vulkan headers to 1.0.49" am: fa3c1590b9  -s ours am: 08fa90eaab  -s ours
am: 56d1521b02  -s ours

Change-Id: I355b8cb7488b6878ec49a8d09f85512d69ad72b8

Merge "vulkan: update Vulkan headers to 1.0.49" am: fa3c1590b9  -s ours
am: 08fa90eaab  -s ours

Change-Id: Ibd45109195a20a14435e9caceb8b5197f2265164

Merge "vulkan: update Vulkan headers to 1.0.49"
am: fa3c1590b9  -s ours

Change-Id: I1d22e6ffa33b6cd4479d6b09bb9d54c67f72ebfa

Merge "vulkan: update Vulkan headers to 1.0.49"

Validate the size of all dex paths eagerly

Also, increase the PKG_PATH_MAX which is used as the max path length to
1024. This should accommodate reasonably long dex path (to cover for
longer secondary dex files paths).

If a dex path  exceeds 1024 characters the dex file will not be compiled.

Bug: 63285397
Test: manual
1) create an artifical long paths under gmscore:
2) update /data/system/package-dex-usage.list to simulate the use of those
dex file
3) run `adb shell cmd package compile -r bg-dexopt --secondary-dex
com.google.android.gms`
4) observe that a 256+ characters dex path is ok while a 1024+ long one
fails.
5) run `adb shell cmd package reconcile-secondary-dex-files
com.google.android.gms` and check no errors are present.

Change-Id: I23b34013e32c5c64ca9c1381cc4d8d67e7121cc8

Merge changes from topic 'VelocityTracker - fix debugging flags'

* changes:
  Convert String8 to std::string
  Fix SIGABRT caused by integer sanitizer.

Convert String8 to std::string

String8 is deprecated.

Test: m -j and uncomment DEBUG flags
in VelocityTracker to view coordinates
in logcat.

Change-Id: I968685765f06418be73fbb8ad2747a1fe8ed324e

Fix SIGABRT caused by integer sanitizer.

Test: m -j and use flings in settings menu.
Change-Id: I7c15c610ed2d74b128a2924c097fb7dc351ea5f4

Merge "Revert "Otapreopt: Work around bug 38186355""

Fix BufferHub state machine to return errors on invalid transitions.

The consumer side of the BufferHub flow did not have an adequate
state machine to track transitions and return errors when Acquire
or Release are requested from the wrong state. This bug allowed
other buggy usage to go unnoticed.

- Fix the consumer state machine to correctly validate all requests.
- Add tests to verify correctness of the fix.

Tested BufferHub with the new test before and after the fix and
verified that the test catches the problem and that the fix solves
the problem.

Bug: 62886596
Bug: 63149525
Test: bufferhub_tests passes.
Change-Id: I802679ed74c7f505b9243ba4048824350d4e37be

Restore PDX support to atrace. am: a6c0a721bd
am: 4c5f5779da

Change-Id: Iaa02a88978878b1c74d57d279a1a7c8bde9ce8ea

Restore PDX support to atrace.
am: a6c0a721bd

Change-Id: I39bd28bde2cad630c8891727670724df3c234ab6

Add GraphicBufferMapper::preloadHal

Plumbing to allow Zygote to explicitly preload the a.h.graphics.mapper
HAL. An earlier attempt did this implicitly with a static initializer in
libui.so, but the library is also used by non-Zygote processes that
don't have access to HALs. This way, the HIDL libraries and HAL are only
loaded in Zygote and its children, not random other processes.

Bug: 62353585
Test: check that a.h.graphics.mapper is in /proc/`pid zygote64`/maps
Change-Id: I4989c8089e3a4cfd627f8940f5552950c1c617fc

Restore PDX support to atrace.

Restore PDX service tracing in atrace. This time only enable poking PDX services
when the "pdx" trace category is included.

Bug: 62882199
Test: Run systrace with/without pdx category and observe correct poke behavior.

Change-Id: I478332a661c3b8bc6061fda43f64b75d8113b305

Fix concurrent access bug in VrFlinger.

VR surface queues have erroneous concurrent access from two threads
because of misplaced lock guards. Move the locking to cover the
queue accesses.

Bug: 62886596
Bug: 63149525
Test: Extensive logging shows that buffers are accessed in the
 correct order and that the consistency of the queues is now
 maintained.

Change-Id: I649e3d75a2230cf3aff50886df9958917d3aa4bf

Merge "emulator: update EGL emulation library path"

Revert "Otapreopt: Work around bug 38186355"

This reverts commit 12375069e753adea6c123fca7706b1018d358c92.

Bug: 38186355
Bug: 38199123
Test: m
Test: manual OTA

Merge "Change storage migration to use quota APIs."

Merge "Add libandroid_sensor_headers"

Clear callbacks when destroying DvrHwcClient

When a DvrHwcClient is destroyed, the VrComposer retains a reference to its
observer. Because registerObserver returns a non-OK status if an observer
is already registered, this was causing subsequent DvrHwcClients to not be
created.

This was an issue with the CompatibilityUiOverlay app on standalone
headsets. CompatibilityUiOverlay was destroying its DvrHwcClient when turning
the screen off, but upon turning the screen back on, CompatibilityUiOverlay
was unable to create another DvrHwcClient.

Bug: 63139142
Test: Turn the screen off/on for a standalone headset, observe
CompatibilityUiOverlay running.

Change-Id: Ie45e961d2bf977ab9a661e23c558354d3a96ae20

Change storage migration to use quota APIs.

New quota APIs are much faster than trying to measure manually, and
removing this last user of calculateDirectorySize() means we can
remove it once and for all.

Bug: 36056324
Test: builds, boots
Change-Id: I60bb197ffc161ed24fe4aa8093495c8874095cdb

Merge "Package media/*/*.h headers as a headers lib: media_plugin_headers"

Merge "dumpstate/Android.mk is converted to Android.bp"

Merge "fix a signed/unsigned comparison warning"

fix a signed/unsigned comparison warning

The comparison was farther down; we'd compare a size_t vector index
with an int index.

Complaint from clang:

vr_hwc.cpp:152:11: warning: comparison of integers of different signs:
'size_t' (aka 'unsigned long') and 'int'

Bug: None
Test: mma shows no more warning.
Change-Id: Icabb5d4bb3c554ffe01a25af975a05bd58f336a1

Package media/*/*.h headers as a headers lib: media_plugin_headers

Headers under frameworks/native/include/media serves as an interface
that is meant to be implemented by vendors. For example.
media/hardware/CryptoAPI.h is implemented under
vendor/widevine/libwvdrmengine/mediacrypto.

When building with BOARD_VNDK_VERSION, the headers are not accessible to
vendors since they have been included via the global include paths and
the paths are not available when building with BOARD_VNDK_VERSION.

So, we need to make a "headers library" for the media headers and modify
vendors to use the headers lib. Usually, we have been doing this by
moving the headers into a subdirectory (usually 'include') of a module
that implements the headers. But, this approach can't be used at this
time since the media headers are implemented by many modules.

The chosen solution is to a separate directory headers/media_plugin and
define a headers lib named media_plugin_headers there.

Note: frameworks/native/headers will now be the home for such
header-only libraries.

Bug: 63120269
Test: build
Change-Id: I67d625706b40d06c8f0602284fbcc1cc8b174c6f

emulator: update EGL emulation library path

Change it from system/ to vendor/, due to
treble requirement.

Test: build sdk_phone_x86-eng, boot emulator
and surfaceflinger should not complain missing
emulation library and emulator should boot to
homescreen.

Change-Id: I08c5ac22358e464c5522deef93cd40ec3bc508c9

dumpstate/Android.mk is converted to Android.bp

Converted to soong. And libdumpstateutil is marked as
vendor_available:true.

Bug: 37342627
Test: BOARD_VNDK_VERSION=current m libdumpstateutil.vendor -j
Change-Id: Id1594d9aa2a7035aaad901d4c3881bd6128c431e

Add libandroid_sensor_headers

This is a subset of NDK headers for vendor modules to use.
It only includes headers for sensors.

Test: m libsensorndkbridge -j
Test: BOARD_VNDK_VERSION=current m libsensorndkbridge.vendor -j

Bug: 33241851
Change-Id: Ia5ab876441f2f7d437a516aa97961b7a3540048d

dvrReadBufferQueueDequeue: allow empty metadata

Currently, when there is no metadata on the read buffer queue, we have
to pass in dummy out_metadata; otherwise the API always return
-EINVAL. This CL allows (nullptr, 0) as legit input and add test to
cover that.

Bug: 62958474
Test: Ran the updated DvrBufferQueueTest
Change-Id: I0ad666006a5d93b67a48b9c34f27e325dfa9a419

surfaceflinger: refresh after latching any buffer am: a36bf92663
am: 81876de166

Change-Id: I6a62ab9052f47f9d97bc30029aa20546094ab9a7

surfaceflinger: refresh after latching any buffer
am: a36bf92663

Change-Id: If437e00a4c8aeda42f26150066a4fd7baaad485e

surfaceflinger: refresh after latching any buffer

We should not skip refresh when a latched buffer happens to have an
empty dirty region.  Also, we should signal a layer update when we
skip refresh.

Bug: 62752640
Test: manual
Change-Id: Ia603e7eeb37491c6ece7ea08d5d1e3b7ba93e6fa

Make default permission checks in performance service more restrictive.

- Check sending process in unrestricted policies.
- Allow IsTrustedUid() to cache results for better runtime efficiency.
- Update tests to verify unrestricted policy behavior.

Bug: 62468109
Test: performance_service_tests passes.
Change-Id: I0614879c8bde35ffeda2f673a3da68092433dc1a

Merge "Fix potential access to invalid memory during shutdown" into oc-dr1-dev am: 77e9eeba69
am: 0e2e44357e

Change-Id: Ic061370a3ef751a8e36a372832e931fa3a516549

Merge "Fix potential access to invalid memory during shutdown" into oc-dr1-dev
am: 77e9eeba69

Change-Id: I5b5703ab3b70755d3f8b8e2674c7c32ce0e1c05b

Merge "Fix potential access to invalid memory during shutdown" into oc-dr1-dev

Merge "Add dvr api entries for sensor enable/disable" into oc-dr1-dev am: 0a9bae53bc
am: 18dcaf1795

Change-Id: Idbe478fd131ac74966bfdabb26d9422b79731b3e

Merge "Add dvr api entries for sensor enable/disable" into oc-dr1-dev
am: 0a9bae53bc

Change-Id: I8ad0fedbac9cf950bf6917a524cb50839b41a362

Merge "Add dvr api entries for sensor enable/disable" into oc-dr1-dev

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

Change-Id: Iefedb1d008614884b1bd3d0abd5bbb7d1b1788fe

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

Include tracking state bits to DvrPose.

These will be used internally for debugging so VrCore
has an idea about what's going on with Tango.

Bug: 62917278
Test: No functional change.
Change-Id: I7f5f7a2ccad4f0259ee5219726bf0e6890f34187

Add dvr api entries for sensor enable/disable

Follow-up to ag/2263549 because we want to use it in VrCore now.

Bug: 63070540
Test: Api call in VrCore
Change-Id: Ic3b2c68ade31008ada8078a74ebe2945748143ae

Fix potential access to invalid memory during shutdown

Since binder threads have a reference to the callback, need to make sure
it is properly reset before deleting the DvrHwcClient object.

Bug: 63139142
Test: Ran on device to ensure VrCore restart don't result in crashes in
DvrHwcClient
Change-Id: I559844a70d4483fee3148526f704d234994a96d4

Merge "Fixed CropLatchingTest that was failing"

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

Change-Id: Ie396445c41942ab6bbeaf35c504a3be004d5e473

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

Change-Id: I29e56cb95b74097c70ba19cc356472ee06e2fac1

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

Change-Id: Ib1f5d7cc4841124c4f5fc1ea543ce3984dc26d16