Merge changes Ie7574cfe,Icc302d2b

* changes:
  remove unneeded forward declaration
  fix typo (minor)

Merge "dumpstate HAL uses "default" service name"

remove unneeded forward declaration

this is already taken care of by vk_platform.h

Test: compiled & booted
Bug: 35164655

Change-Id: Ie7574cfe985fed86922c923ca3b2225e017c2d5a

fix typo (minor)

Test: compiled & booted
Bug: 35164655
Change-Id: Icc302d2b8301d877f76a3b652a42abcaeb2c99c7

Merge "docs: Add brief and detailed descriptions for native trace API. Test: Verified via technical review and manual build."

Merge "gralloc HAL uses "default" service name"

docs: Add brief and detailed descriptions for native trace API.
Test: Verified via technical review and manual build.

Change-Id: Iccd0a78ea4e0f5b9b7d6397bef99084876759058

split libandroid.so

- ANativeWindow (native_window.h) and
  AHardwareBuffer (hardware_buffer.h) now live
  in libnativewindow.so, which is a vodka library.

- libandroid.so still provide those symbols
  via a static library version (libnativewindow.a)

- removed libbinder dependency as well

Test: compiled & booted
Bug: 35164655
Change-Id: Ib4bc8e8fc9ec1891bcbee63c28fd0131d82edf29

Merge "MediaCas: add plugin APIs"

Merge "Create VR HWComposer without locks held"

Create VR HWComposer without locks held

Problem:
Surfaceflinger is occassionally deadlocking when switching to VR Mode.

Root-cause:
During the creation of the VR HWComposer through hwbinder,
hwservicemanager notifies surfaceflinger the service is available through a
binder callback. Because Surfaceflinger holds the state-lock during the
HWComposer creation, its possible all of SurfaceFlinger's binder threads are
handling other transactions waiting for the state-lock to be released.
This prevents the hwservicemanager callback ever to be handled resulting in
a deadlock.

Solution:
Don't hold the state-lock when instantiating the VR HWComposer.

Test: Validated deadlocks no longer occur when switching to VR mode.
Bug: 35680107
Change-Id: I932c81fce293a8b57983f4242432b87522ce964c

Merge "Fix crash in sensord that can occur during system instability"

dumpstate HAL uses "default" service name

The getService() and registerAsService() methods of interface objects
now have default parameters of "default" for the service name. HALs
will not have to use any service name unless they want to register
more than one service.

Test: builds

Bug: 33844934
Change-Id: I87d3f4f00965e8cccb18b9bd158bd8d393ea8d5a

Merge "hwcomposer HAL uses "default" service name"

Fix crash in sensord that can occur during system instability

Bug: 35624417
Test: Manually ran CubeSea and closed/openned it several times
Change-Id: I57810531d0ceccfa02afcf0af265587ca88e05be

Merge "docs: Configure Doxygen to ignore conditional compilation statements. Test: Verified by manual build."

Merge "Expose display manager client event fd"

Merge "VR: Replace sw sync fences with egl sync fences"

hwcomposer HAL uses "default" service name

The getService() and registerAsService() methods of interface objects
now have default parameters of "default" for the service name. HALs
will not have to use any service name unless they want to register
more than one service.

Test: builds

Bug: 33844934
Change-Id: I6befd081add61e355bc25dd0c630ac07ea0103c4

docs: Configure Doxygen to ignore conditional compilation statements.
Test: Verified by manual build.

Change-Id: I96256700597c4dc498b36f52037ded2ea5a25e0d

Expose display manager client event fd

This will allow the display manager client to use epoll to wait and
respond to changes in the current set of VR display surfaces.

Bug: None
Test: Compiled and ran with simple standalone binary application.
Change-Id: I7680f5a56348abfb686e78d13e0e737466735ac6

Merge "Logic for atmoic/tombstone behavior; split mode."

VR: Replace sw sync fences with egl sync fences

HWComposer::presentDisplay() synchronously calls into the VR WM
presentation code to queue the new frame. If VR WM is showing (drawing)
it will return an EGL fence that will be used to signal when the last
showing frame is no longer used in VR WM. Otherwise an invalid FD (-1)
is returned symbolizing that the last frame isn't in use anymore and the
caller can re-use those buffers immediately.

Bug: b/35096352
Test: Verified SurfaceFlinger output shows correctly in vr_wm
Change-Id: I2b7c2c767d4aa85370dd6519113cb9a2269c7477

Merge "Ensure display metrics on display service creation"

Logic for atmoic/tombstone behavior; split mode.

Flesh out logic for cache directories that request new atomic and/or
tombstone clearing behaviors.  Atomic directories are considered for
deletion as a single all-or-nothing unit, and tombstone directories
truncate any removed files instead of unlinking them.

Since these behaviors can be mixed together, add local tests that
quickly verify several different permutations.

Reduce memory footprint of CacheItem objects by only storing name
and pointer to parent (instead of full path).  Fix ordering bug by
switching to std::stable_sort.

Add "V2_DEFY_QUOTA" flag so we can split clearing into two distinct
phases: clearing data for apps above their quotas, and then pushing
deeper by clearing data for apps below their quotas.

Test: adb shell /data/nativetest64/installd_cache_test/installd_cache_test
Bug: 34692014, 33811826
Change-Id: I156897de1d1d1c371b2b837128b2e286bf33d40d

Merge "lshal: Fix timeout causes unexpected exits." am: 057e74631b am: fdf61aa557
am: 91a4bd9cd2

Change-Id: I4663a64806ddb285e8bf5e9235759fee49bc8a93

Merge "lshal: Fix timeout causes unexpected exits." am: 057e74631b
am: fdf61aa557

Change-Id: Ie3f025b610fd8b523b4f03c2c9b5d62b7966f309

Merge "lshal: Fix timeout causes unexpected exits."
am: 057e74631b

Change-Id: Iadfbc11f8bbfc45759f2214d0811922cbb173758

Merge "lshal: Fix timeout causes unexpected exits."

Ensure display metrics on display service creation

This fixes a potential race condition where a VR app could request the
display metrics from the VR display service before it actually queries
them from the HWC.

Bug: None
Test: Compiled and ran Vr app.
Change-Id: Ie95b7f5a8ef2e286c7b2994ca94fd87214567e24

Merge "Add missing GLES 3.x metadata"

Merge "cleanup libui’s includes"

lshal: Fix timeout causes unexpected exits.

with thread::detach the background thread keeps running even after
the thread object is destroyed; the background thread will access
caller's stack memory, causing segfault. Change it to a thread join
to avoid the issue. To avoid waiting too long on the child thread,
send a SIGINT if timeout (child thread's signal handler will then
call pthread_exit() to terminate the thread).

Since we are using pthread_* functions, change usage of std::thread
to pthread_t for consistency.

Test: lshal
Test: run lshal with IPC_CALL_TIMEOUT set to zero will no longer
      cause SIGSEGV and SIGABRT (test 10 times)

Bug: 35623669
Change-Id: I4eef8ffd8ff399793648e861ca4c1a2bdcc7ec50

Merge "Use --receiver-include-background instead of package name on SendBroadcast()."

cleanup libui’s includes

- remove unneeded includes
- include a cpp’s header first in all cpps

Test: compiled & booted
Bug: 35164655
Change-Id: I0890913fcc6614cce156aff19fccdef183145beb

Merge "Remove erroneous log message"

Merge "Method to test for quota support."

Remove erroneous log message

It seems we always get a duplicate remove from the layer-cleaner
so this was showing up any time any layer was removed. We didn't have
this message before the child layers CL so lets get rid of it quickly.

Test: Deleting a line of logging code. It builds!
Change-Id: I16cc0c0b325e43d36cae684b06a93c099baa1aa6

Merge "vr_wm: Import cloned buffer in VR HWC"

Merge "Refactor libposepredictor to be self-contained."

Merge "Removed build time variable ENABLE_CPUSETS." am: 52a60e08fb am: 33f4b64f23
am: eb8be8f5e9

Change-Id: I9624618aae00134414e946e3f6df97e89c5b7495

Merge "Removed build time variable ENABLE_CPUSETS." am: 52a60e08fb
am: 33f4b64f23

Change-Id: Ia6f3940b52c28741a43be171c930d0cdd77fb454

Merge "Removed build time variable ENABLE_CPUSETS."
am: 52a60e08fb

Change-Id: Ic5d351f18a0c839212987c5fcbb803a7c07ac5cb

Refactor libposepredictor to be self-contained.

The CL seems big but it is mainly moving stuff around to
make libposepredictor self contained.

libposepredictor/include/predictor.h exposes all functionality and
it only includes Eigen stuff.

Also adds a shared library build step to the makefile.

Bug: 35362498
Test: No functional change.
Change-Id: I0c850711ffde16ee75e4c88b09e47f761c100385

vr_wm: Import cloned buffer in VR HWC

Bug: b/35619417
Test: Compiled and tested on device. Ensured SurfaceFlinger content is
visible in VR mode.

Change-Id: Ic2e653c9771fd022079486704b9e7a412d18bfdc

Method to test for quota support.

Framework logic needs to change its behavior depending on how quickly
disk usage calculations can be performed.

Test: builds, boots
Bug: 34690590
Change-Id: I6139d1ea6a6084b1fd42062e2032ec9f515f6d98

Merge "Removed build time variable ENABLE_CPUSETS."

Update list of output files in vr_window_manager hidl genrule

The output files in the android.dvr.composer@1.0_genc++_headers rule
is not correct, which causes ninja to rebuild the rule on every
incremental build.  Fix the list of files to match what is actually
created.

Bug: 33554989
Bug: 35562758
Test: builds, but doesn't rebuild on every incremental build
Change-Id: I2938ad02b8feade4de6d8a5cc969f37e793e93a8

Add missing GLES 3.x metadata

Prevent a crash when glGetIntegerv(GL_NUM_PROGRAM_BINARY_FORMATS) returns 0

Bug: 33106868
Test: Manual test app
Change-Id: I5c9fdf4f6f14aa08961e70a0cf63fb78b88738fb

Merge "move vector math out of libui"

Merge "Refactor HWC2on1Adapter"

Removed build time variable ENABLE_CPUSETS.

Use runtime check, cpuset_enabled(), instead of build time varianble
usually set in BoardConfog.mk.

Bug: 34726944

Test: Tested on Sailfish
      without cpuset mounted
      with cpuset mounted, but without schedtune
      with cpuset mounted and with schedtune

Change-Id: I00153628db62abfbaa0ba75d571ba1bda0611ff3

move vector math out of libui

created a new header-only static libmath lib

Test: built & ran
Bug: n/a
Change-Id: Ic63ef5f54d9a0de07a9ab9e4d67be01ab6169fc0

Refactor HWC2on1Adapter

- Simply memory allocation with a single, consolidated calloc().
- Remove latch system which did not prove useful.
- Converted { to be before \n.

Change-Id: Id1daec2343225598160395bfecddc86955e2b610

Merge "libdvrgraphics: Add libarect as whole static lib to fix compile"

Merge "Discard surfaceDamage for HWC1 v < 1.5"

Merge "Fix lshal cannot be run without root" am: 6481ebfbc3 am: 8d6362fc44
am: b40360f7c3

Change-Id: Id53e3d9648060bbdecc5f1b67d4ac85183162740

Merge "Fix lshal cannot be run without root" am: 6481ebfbc3
am: 8d6362fc44

Change-Id: Id188e2657e393b0463c936435d2d8273bc3a80cf

Merge "Fix lshal cannot be run without root"
am: 6481ebfbc3

Change-Id: Ic269ab86b933557ee171961f78d6e49e0d4df315

Merge "Fix lshal cannot be run without root"

Merge "Polynomial pose prediction."

Merge "Fix typos in EGL_ANDROID_get_native_client_buffer spec."

MediaCas: add plugin APIs

bug: 22804304
Change-Id: I87913f39a95278d0d5eb282f2b83b1fa4d362cec

Merge "Add tool to generate color space conversion 3D LUTs"

Discard surfaceDamage for HWC1 v < 1.5

Test: Manual
Change-Id: Ia9193a88dc6df206c8f197adf144466f164afa1c

libdvrgraphics: Add libarect as whole static lib to fix compile

Changes in frameworks/native cause compile errors looking for
android/rect.h.

Bug: None
Test: Build completes.
Change-Id: Ie9ba9aca0cb620c0874d47113b9ea9875277a0f7

Merge "Fix typo in Region to avoid unexpected values"

Fix lshal cannot be run without root

Root cause is invoking copy constructor on a Return<T> object
implicitly, while the old object contains a failed status and
gets destroyed. Use the move constructor instead to fix this.

Test: adb unroot && adb shell lshal
Change-Id: I44710166cc5d7da30bf54b10d1860be1b91dc98a

Fix typos in EGL_ANDROID_get_native_client_buffer spec.

Bug: 34050596
Test: N/A
Change-Id: Ic15f074c019b0973408436781a00f9bd0260f036

Merge "Break a dependency of libui on libandroid"

Merge changes from topic 'GPU_replace_GL'

* changes:
  Fix unexpected FrameEvents on BufferQueue reconnect
  Add FrameEvents::isValidTimestamp
  Change GL references to GPU for getFrameTimestamps.
  Clean up FrameTimestamp log messages.

Add tool to generate color space conversion 3D LUTs

The tool needs to be run on an Android device and must be compiled
manually (i.e. it's not compiled as part of a regular Android build).

The tool outputs 3D LUTs in a format that can be included directly
in a C/C++ file/header.

Bug: 32984164
Test: ran lutgen and checked output
Change-Id: I96f7d0d088965c3b63c0fb4079ddeaeee7246418

Merge "Move FloatRect from libgfx to libui"

Break a dependency of libui on libandroid

This was only a header dependency, but it still
created a circular dependency, which will cause
problems in another CL.

We fix this by creating a libarect static library
containing only that header. both libui and
libandroid now depend on it and reexport the
header.

We also make sure rect.h ends-up in the right
place.

Test: built and booted device
Bug: 35164655

Change-Id: Iba25b8b801b26b26ec1401c00caf367a06f197ca

Merge "Proper fix for vr_wm: the hwc_callback function is no longer called"

Merge "Verbose logging when API_MEDIA is force-disconnected"

Merge "Add RGBA 10:10:10:2 format for surfaces"

Merge "atrace: Don't output service lookup/notification errors." am: 9753b43355 am: acc03d821b am: b4951bb3c5
am: c874bf6032

Change-Id: I2f82d450a9a07d1a30b8017e34ad1f158fac3eec

Merge "atrace: Don't output service lookup/notification errors." am: 9753b43355 am: acc03d821b
am: b4951bb3c5

Change-Id: I1795408b3731916b08d5c716bacebc1892a9760a

Merge "atrace: Don't output service lookup/notification errors." am: 9753b43355
am: acc03d821b

Change-Id: Id61287c635379735bc0c78dea8571720594de52f

Move FloatRect from libgfx to libui

Moves FloatRect from libgfx (which is being disintegrated) to libui

Test: Builds and sailfish boots
Change-Id: I68c1b7d86a363066fe4d6f0c038ca9d92d7ab9c7

Merge "atrace: Don't output service lookup/notification errors."
am: 9753b43355

Change-Id: I08da77d7664543aa1922a23792fa68f9094323ef

Merge "atrace: Don't output service lookup/notification errors."

Merge "libui: Remove STL from Fence"

Proper fix for vr_wm: the hwc_callback function is no longer called

Another change removed the call to GetBufferFromHandle but didn't
delete it and an alternate implementation is used. This replicates
the change into the second implementation and removes the original one

Bug: None
Test: Tested in dev branch; built in master
Change-Id: I2bdb38c7b692e30cf1b5e41476ecdd062b5d5d69

Merge "Add HDR Metadata extension"

atrace: Don't output service lookup/notification errors.

Bug: 34933232
Test: android.atrace.cts.AtraceHostTests
Change-Id: If981106fb1bf7db24b9fd173c6936a1e189756f0

Fix typo in Region to avoid unexpected values

b/34193533 showed that Framework can sometimes use very large rect
(presumably to hide a window **very** far out of the screen visible
rectangle). Which results in SF freezing and using 100% of CPU.

max_value is not initialized to the right value. Really it should be
0x7FFFFFFF instead of
0x7FFFFFF
Since Region was templatized (originally to potentially use int16_t)
the fix uses C++11 numeric_limits instead of INT_MAX.

Test: Manual
Change-Id: Iaee64727226730dbea1f05618a97b801254cde4d

Verbose logging when API_MEDIA is force-disconnected

Media is seeing intermittent failures during CTS when mediaplayer
connected to a surface successfully but subsequent dequeueBuffer
fails. We suspect that media is getting force-disconnected. Adding
a verbose log when this happens to help track down the failure.

bug: 35364355

Change-Id: I0c48e3d141648a3d08c6abeb6793f8363e9502fc

Add HDR Metadata extension

Basic framework to support HDR Metadata extension. Will
add system plumbing in separate CL.

Test: CTS dEQP-VK.wsi.android.hdr_metadata.scale_none.fifo.*
Change-Id: I5de639ef87a988e02d194a2bd91d16ffe6102c6e

Merge "Expose extension for obtaining an EGLClientBuffer from an AHardwareBuffer"

Polynomial pose prediction.

Adds PolynomialPosePredictor. It is not enabled by default in sensord.

I will do a grid search for the best parameters before turning it on.

You can set dvr.predictor_type to 1 and restart sensord to test.

Bug: 35362498
Test: Run predictor_eval to test the numerical and performance. Set dvr.predictor_type to 1 for qualitative testing.
Change-Id: I435fcc4d141f5949047d0f9d972fcfc8f2a94519
(cherry picked from commit 0b024945982fd2630709534ce448e55657768696)

libui: Remove STL from Fence

Renames Fence::hasSignaled to Fence::getStatus, and changes its return
type from std::experimental::optional (STL is no longer permitted in
libui) to an enum class. Also updates its one caller.

Test: Builds and sailfish boots
Change-Id: I682039b9fd88bf79f1e2a955944c4a564ed6f60b

gralloc HAL uses "default" service name

The getService() and registerAsService() methods of interface objects
now have default parameters of "default" for the service name. HALs
will not have to use any service name unless they want to register
more than one service.

Test: builds

Bug: 33844934
Change-Id: I65d9d283fe7b72ddb286ff41bd23ca33345d466c

Merge "VR: Move VR HWC initialization in SurfaceFlingerView"

Expose extension for obtaining an EGLClientBuffer from an AHardwareBuffer

Bug: 34050596
Test: deqp test checks for extension
Change-Id: I0da1173688cd27b8dd9e907e535100b9a0605290

Merge "Move libgvr to vendor/unbundled_google/libraries/vr."

VR: Move VR HWC initialization in SurfaceFlingerView

Initialize the VR HWC at first use since it is also needed to register
the frame observer VR WM uses to display SurfaceFlinger output.

Note, need the real implementation, rather than the passthrough
interface received via IComposer::getService().

Bug: b/35403318
Test: Compiled
Change-Id: I7569136edcef57f545e3d1c3daf644f48fd6b907

Merge "surfaceflinger: disable VR mode while SELinux issues are resolved."

Merge "Merged vr window manager and vr composer"