Add unit test for onBufferFreedListener

As a follow up to cover new functionality introduced in ag/1998938

Bug: 35114769
Test: run libgui_test with --gtest_filter='BufferItemConsumerTest.*'

Change-Id: I461dc88eac84e165f80a1838b63bfd6bb22a56d7

Merge "Refactor IonBuffer to use GraphicBuffer"

Changing from ro.product.device to ro.product.name for bugreport file name

ro.product.device returned dragon for ryu, when the original intent
would have been to have ryu.  ro.product.name does this.

Bug: 33257711
Test: Compiled and flashed to a device, and took a bugreport
Change-Id: Ib2a11e084f9c1c550c28e0c89adc92a14652ee37

Merge "Refactor VrFlinger to use BufferHubQueue"

Merge "BufferItemConsumer: Add onBufferFreedListener"

Merge "[HWC2on1Adapter]: Add support for HWC2_PFN_GET_CLIENT_TARGET_SUPPORT"

Merge "Fix getFrameTimestamp test flakes."

Merge "Use hasSyncFramework value from configStore"

BufferItemConsumer: Add onBufferFreedListener

This callback can reliably notify its listener when a GraphicBuffer is
being freed at the consumer end of a BufferQueue. This is extremely
useful when user of BufferItemConsumer caches the GraphicBuffer, so they
can listen to this callback and clean up resource properly.

Bug: 35114769
Test: Built system, flash marlin (should have no functional change as
the listener is optional).
Change-Id: I591cfd3ece697b1b4fb3efbeff987c7960422d24

Merge "Add getDataSpace to layer class"

Merge "Set default color mode to trigger update"

Use hasSyncFramework value from configStore

Change-Id: I41c6b1a26001eb6ba08cbc419dc8a683f5722aa3

[HWC2on1Adapter]: Add support for HWC2_PFN_GET_CLIENT_TARGET_SUPPORT

Test: Ryu, Marlin
Change-Id: I6209c1a4290010b04fedbcb3d30c3a1d56cfb3e9

Merge "Remove access of DisplayClient from vr_wm"

Add getDataSpace to layer class

Test: TBD
Change-Id: I1d583bc6dae4cfb1569f053b2480e94b6ca9cc6f

Set default color mode to trigger update

Start with invalid color mode so that we are sure
to call downstream for any setColorMode.

Test: manually verify display is in sRGB mode
Change-Id: Iaf511fac1e4cd8acef149528877188708c96856f

Merge "fix remote_method type conversion for channal handles"

Merge "Update extension enums to match upstream."

Update extension enums to match upstream.

Test: make
Change-Id: I0c48e3d7112ed009c088b0336fd974ee0dd22afe

Refactor VrFlinger to use BufferHubQueue

1/ Remove DisplayRPC::AllocateBuffer, as individual buffer allocation
is now handled by BufferHubQueue.
2/ Reimplement native_buffer_queue using bufferhubqueue.
3/ Hook up consumer queue in DisplaySurface.
4/ Remove epoll_event_dispatcher as its no longer being used.

Bug: 36033302
Test: Built and ran particles.apk
Change-Id: I38ee1c57195888ede935ebc50119bcb7e4ab4e36

fix remote_method type conversion for channal handles

* Required for upcoming change that mixes borrowed and local
  channel handles

Bug: None
Test: None
Change-Id: I2efd874d975913133e89eb596af3cc69e793be6b

Remove access of DisplayClient from vr_wm

It seems like this functionality relied on another pending patch that
will not come into the picture just yet. I am not reverting the full set
of patches, as they contained some other changes which are best left in.

Bug: None
Test: *pending*
Change-Id: I2d6b6ec8ba6acc98388e5f13a54ad2cea25c1a00

Use configStore for maxVirtualDisplay Size

Test: Manual and AUPT
Change-Id: I17ec13f782a7c7c693d84d11c9b5e87624ee71d0

Merge "Expose acquire_fence though ConsumerQueue::Dequeue"

Merge "Revert "Use configStore for maxVirtualDisplay Size""

Revert "Use configStore for maxVirtualDisplay Size"

This reverts commit 56a5dbd3b050cbfd324308d224c7d12f4aec3320.

Change-Id: I66224bb48d3146846200ff7f2d5a66d1d8ff5cf7

Merge "Remove AHARDWAREBUFFER_USAGE0_GPU_CUBEMAP as it has no corresponding bit in gralloc1."

Expose acquire_fence though ConsumerQueue::Dequeue

The current compositor implementation will be refactored to use
BufferHubQueue instead of ad-hoc BufferHub operations. We need this to
expose release_fence to compositor so that it can wait for buffers
to become avaiable by checking fence properly.

Bug: 36033302
Bug: 36148608
Test: Built and ran buffer_hub_queue-test
Change-Id: I75cfcb02e06a4b9e7e89b89690ca2d92ee09a678

Merge "Use configStore for maxVirtualDisplay Size"

Remove AHARDWAREBUFFER_USAGE0_GPU_CUBEMAP as it has no corresponding
bit in gralloc1.

Bug: 34050596
Test: build, non-functional change
Change-Id: I61e46d6035f8477615f23ab245493336e61abbc3

Merge "Add implementation for android.frameworks.sensorservice@1.0"

Refactor IonBuffer to use GraphicBuffer

- Replace gralloc0 based implementation with GraphicBuffer implementation.
- Removed incompatible ion-buffer unit-tests.
- Added libui dependencies to dependants.

Bug: 34879523
Test: Tested on Lucid, validated graphics and poses are working
Change-Id: I49a129f269d54c81bda93b44d879d4b8dee2006a

Use configStore for maxVirtualDisplay Size

Test: Manual and AUPT
Change-Id: Iefc09b06ab473c4029756279920135761547dd65

Merge "Support profile guided compilation for secondary dex files" am: cb2e477f14 am: f01cd5f89f
am: 95c5f4b85d

Change-Id: I432fd0a8e7056b5d41e6beb9a71d8fd74c3a34b8

Merge "Force vrhwc/wm to report the native resolution"

Merge "Support profile guided compilation for secondary dex files" am: cb2e477f14
am: f01cd5f89f

Change-Id: I5e012812cc71ed2030c5c06c151cfa68f89d6979

Merge "Support profile guided compilation for secondary dex files"
am: cb2e477f14

Change-Id: Ib763c8cf38a04243b1a4834a70b15a6533f7bf5d

Merge "Support profile guided compilation for secondary dex files"

Force vrhwc/wm to report the native resolution

Due to some Yak-shaving, this also involved a partial conversion of our
libraries to soong.

Notes:
* I also changed the name of libsensor to libvrsensor to avoid a naming
  clash with an existing library
* The remaining libraries will be soongified in a separate CL

Bug: 36139334
Test: Build and run vr apps.
Change-Id: Ib39687bd01e3d2e1c30adc54e18f362a85954ab9

Support profile guided compilation for secondary dex files

Test: adb shell cmd package bg-dexopt-job works for sercondary dex files
Bug: 26719109
Change-Id: I4011a780f65378c23a4631c07c37cdb4f5fee41e

Use forceHwcCopyForVirtualDisplay in configStore

Test: Manual
Change-Id: Ie6f816cf8f48b1cca41d51f40e9f0ae49aef2908

Fix getFrameTimestamp test flakes.

This fixes both the dEQP CTS flakes relating to
compositor timing and the lib_gui test flakes that
don't initialize properly.

Test: adb shell /data/nativetest/libgui_test/libgui_test
  --gtest_filter=*GetFrameTimestamps*
Bug: 35995043

Change-Id: If8e59f0dc9a916bab28bd1a36190cef9a56cb64f

Merge "Add feature tag android.hardware.vr.headtracking."

Merge changes I66c13b20,I0f1cc83c,I653d0d8a

* changes:
  vulkan: Implement usage flags query for KHR_shared_presentable_image
  vulkan: Implement KHR_get_surface_capabilities2
  vulkan: Update header for KHR_get_surface_capabilities2

Add implementation for android.frameworks.sensorservice@1.0

Test: marlin boots
Test: lshal -ipm | grep android.frameworks.sensorservice@1.0
Bug: 35219747
Change-Id: Idc2823c674ef452102766a1331c962d3f3f66c53

Merge "Revert "Get rid of IGraphicBufferAlloc""

Revert "Get rid of IGraphicBufferAlloc"

This reverts commit 78491c9f694cb0767996503c629776a8eda950d7.

Change-Id: I78d5c0a30ab80265f697f681387872b6763b2d1e

vulkan: Implement usage flags query for KHR_shared_presentable_image

This was a late addition to the spec. Until we have a reason to do
otherwise, report the same set of supported usage flags for shared
present modes as for all other modes.

Test: build
Change-Id: I66c13b20a1a0f5aa9be13bc16a323f42874ba45a

vulkan: Implement KHR_get_surface_capabilities2

Implemented by forwarding to the KHR_surface equivalents. This requires
a bit of rejigging in the case of GetPhysicalDeviceSurfaceFormats2KHR
to deal with the stride difference.

Change-Id: I0f1cc83cfc885955a39fc02d4a15609e83d36798
Test: dEQP-VK.wsi.android.surface.*2

vulkan: Update header for KHR_get_surface_capabilities2

.. and new struct added to latest version of
KHR_shared_presentable_image.

Test: build
Change-Id: I653d0d8ad384c256e9a5c4cb79554360a5b2606e

Merge "Get rid of IGraphicBufferAlloc"

Merge "Add a retry mechanism to potentially allow recovery from HIDL failures."

Add feature tag android.hardware.vr.headtracking.

Different versions of this feature specify different levels of
headtracking capability.

Bug: 36294374
Test: build
Change-Id: I3ca56822c9f8a0d48431762b7c4eba5cdf904512

Get rid of IGraphicBufferAlloc

This reverts commit 527747dce1ab4714bd424e5c1a25ebad3506c2cb.

selinux policy allowing this change fixed.

Test: took a screenshot
Test: ran ImageReaderTest CTS
Bug: 36194109
Change-Id: I72ac17b6c252750aa2a66cd1d94bd8b4e21b5e9d

Merge "[SF] Use presentTimeOffset from configStore"

Merge "Remove bad make rules causing system.img to rebuild every time"

Merge "Add a simple SurfaceFlinger stress test."

Add a retry mechanism to potentially allow recovery from HIDL failures.

Retry a few times to see if HIDL failures persist.
Add some logging to track failures.

Bug: 36088202
Test: Ensure sensors stream normally after change.
Change-Id: I194eaffd455ba782fff041f03ef89da384e3a901

[SF] Use presentTimeOffset from configStore

Change-Id: If9c872c565e68e8abe552ee11d2c7d48f44aec4b

Merge "vulkan: Implement V7 of ANDROID_native_buffer in nulldrv"

Remove bad make rules causing system.img to rebuild every time

ALL_DEFAULT_INSTALLED_MODULES is supposed to contain paths, not
module names.  Module names are phony targets, which causes
anything that depends on it to be always dirty, resulting in
rebuilding system.img on every incremental build.

ALL_DEFAULT_INSTALLED_MODULES shouldn't be used anyways, use
PRODUCT_PACKAGES in a product makefile instead, and all_modules
isn't used at all any more, remove both of them.

Bug: 36274890
Test: builds
Change-Id: I2db49dcc27515357b0306d0c3e9b38026012b1ea

Merge "Add uid and pid information into recent sensor registeration log"

Merge "Use assignment instead of reset for profile unique_fd" am: fd695a61c1 am: 32d41fecc8
am: 621bc30864

Change-Id: I7635c11b945098d39ae6d557b3df27854ec19d42

Add a simple SurfaceFlinger stress test.

Obviously such tests are always a little arbitrary but this one
seems pretty useful. It runs in ~2 seconds, I got 10 passes in a row with
the fix for 36117490 applied, and 3 system restarts in a row (before giving up)
without the fix applied. Clean up the other tests in preparation for presubmit enabling.

Test: Quis custodiet ipsos custodes?
Bug: 36117490
Change-Id: Ifef2ff4e51aa9af193ddf003fe4aba053b28738f

Merge "Use assignment instead of reset for profile unique_fd" am: fd695a61c1
am: 32d41fecc8

Change-Id: I660b56126fc76dab6df9ac92b8f43d409b088a77

Merge "Use assignment instead of reset for profile unique_fd"
am: fd695a61c1

Change-Id: Ifcf742625e9c05379b063ba5480db406a9b81459

Merge "Use assignment instead of reset for profile unique_fd"

Merge "Remove dvrDisplayManagerClientGetSurfaceBuffers"

Merge "Make IGraphicBufferProducer a hybrid interface."

vulkan: Implement V7 of ANDROID_native_buffer in nulldrv

Walks the struct chain in GPDP2, and claims support for shared
presentable image. This is enough to have libvulkan expose the
extension for us.

Change-Id: Id22593932cd567e79d18aae788935148d6278b3f
Test: setprop ro.hardware.vulkan default, observe extension exposed

Remove dvrDisplayManagerClientGetSurfaceBuffers

This should be replaced with BufferHubQueue

Bug: 36033302
Test: Build device targets
Change-Id: I658514e1b56b0bdbd523d3fa322f0f2b6a1fc7e8

Signed-off-by: Jiwen 'Steve' Cai <jwcai@google.com>

Merge "Drop non-API headers from the NDK vulkan headers." am: c416f2eb23 am: 44e0f0a5a5
am: d7cf69be63

Change-Id: I0eb74c0a12a39875b4e4d417b844f1bc287f87d0

Merge "Drop non-API headers from the NDK vulkan headers." am: c416f2eb23
am: 44e0f0a5a5

Change-Id: I4894ca20c2ad52565490beb92a1a40efdb05496e

Merge "[SF] Delete unused NEVER_DEFAULT_TO_ASYNC_MODE"

Use assignment instead of reset for profile unique_fd

Using reset with a unique_fd will just convert the unique_fd to int
without releasing it.

Test: adb shell cmd package compile -m speed-profile com.google.android.googlequicksearchbox
Bug: 36234017
Change-Id: I577197ac4a147765191777a5ebc6ef150dea1027

Merge "Drop non-API headers from the NDK vulkan headers."
am: c416f2eb23

Change-Id: I72c6dd7f474010b5410209efad69b0c3fbc88f85

Merge "Drop non-API headers from the NDK vulkan headers."

Merge "BufferHubQueue: ClearAvailable on consumer import"

[SF] Delete unused NEVER_DEFAULT_TO_ASYNC_MODE

Test: AUPT
Change-Id: I0abbbdcbd2a7ccb2b60514b54e600eb9489dc6a4

Merge "test-hwc2: fix build error from ui/mat4.h include"

Merge "vrwm: Add support for multiple independent displays"

Merge "Add EGL_SCALE_METADATA enum"

Merge "EGL: Grab relevant upstream colorspace changes"

vrwm: Add support for multiple independent displays

This doesn't change much, just splits ShellView and shuffles code
around.

Display-related things go into DisplayView - everything related to
processing and drawing buffers, hit detection, visibility.

ShellView retains overall state management, touchpad, controller and
is the initial recipient of all incoming frames

The composer library is modified to accept and pass multiple displays

Bug: 35996499
Test: Works for the existing single-display output
Change-Id: Ied5061b4dad9e7d68bb187bf86c3d0f2f3b7a55e

Merge "test-hwc2: fix build errors on other branches" am: ea8d46f1ab am: acfd2a846b
am: 96074086df

Change-Id: Iac974deaf0c4a0df9512ff666ff382a18ad270b6

Merge "test-hwc2: fix build errors on other branches" am: ea8d46f1ab
am: acfd2a846b

Change-Id: I7ed1e6c8de18111275a128d44a523edd02e9833d

Merge "test-hwc2: fix build errors on other branches"
am: ea8d46f1ab

Change-Id: I15fb43deec4eac3bb2867fa4dd101606cb593389

Merge "test-hwc2: fix build errors on other branches"

Merge "[SurfaceFlinger]Use configStore useContextPriority"

Merge "Use d=0.039 for Display P3 instead of sRGB's 0.04045"

test-hwc2: fix build error from ui/mat4.h include

ui/mat4.h is located at math/mat4.h on internal master. Change
test cases to reflect that.

Test: run the test-hwc2 tests

Change-Id: I1d2fb59efc2fcea40dc901664dc43841c5454353

Drop non-API headers from the NDK vulkan headers.

The other headers in this directory are just for the platform and
shouldn't be in the NDK. Fortunately we never exposed the symbols in
the stub libraries so users couldn't actually do anything with the
other headers, but we shouldn't be exposing these headers anyway.

Test: make checkbuild
Bug: http://b/36224616
Change-Id: I9e162964e349a256e7c7ad25eca3c33c5631e2c8

Merge "Add ANativeWindow_setBuffersTransform"

test-hwc2: fix build errors on other branches

Currently, test-hwc2 does not build on other branches due to
differences in GraphicBufferAlloc. Switch to directly allocating
graphic buffers.

Test: run the test-hwc2 tests

Change-Id: I63a058a9b0bf48d1e3236f5e124ecaa1dfb4593a

Add ANativeWindow_setBuffersTransform

Bug: 32634406
Test: build
Change-Id: I16080008187bd334e64917274fc86afe12dde8e7

Merge "Preserve phones screen aspect ratio for VR Window Manager."

[SurfaceFlinger]Use configStore useContextPriority

Change-Id: I329b5da0e92822eb0878c8866c8ab87ea13f7e07

Preserve phones screen aspect ratio for VR Window Manager.

We are a portrait device rotated to landscape mode, adjust rendering
inside vr_wm accordingly.

Bug: 36201035
Test: Built, ran permissions gen, denied permission.
Change-Id: I27382ea6f5417a92791eafa9daaa81cc4af6db11

Merge "Support multiple virtual touchpads."

Merge "vr_wm: Disable 2D mode until we can detect it correctly"