libpdx: Fix bug in Variant type.

Fix a subtle bug in the Variant value destruction along the
EmptyVariant assignment path:

    some_variant = EmptyVariant{};

The problem arises from the private utility method Destruct(),
which does not set the type index to "empty" after destroying the
current sub-element. For most paths this is okay because the type
index is immediately set to the new sub-element type. However, the
EmptyVariant path does not assign a new type because the Variant
should become empty. Since the type is not set to "empty" the
Variant incorrectly double destroys the previous value when a new
value is assigned.

Also clean up a superfluous overload of Assign() that is skipped
due to the stronger binding of the universal reference overload
Assign(T&&).

Update tests to properly cover this case. In the process discovered
two incorrect tests related to this issue and updated them.

Bug: None
Test: pdx_tests passes.
Change-Id: I106f863b34f2719820d04d0e701968332f659c3e

Disable in-place vdex update am: 3291cd9eaa
am: 1b8bd96ef8

Change-Id: I0389d08b4c6e32e10c9ee83d7177409bff98e270

Disable in-place vdex update
am: 3291cd9eaa

Change-Id: I9289c4cd1dea5299c842a935943101370ca609e4

Merge "Installd: deleteOdex's outputPath can be null" am: 0c189de962 am: 6f0d10dbde am: 084434a475
am: 902bf61f51

Change-Id: Ib790d04fb59565ea6737a215deb95ddac30a195a

Merge "Installd: deleteOdex's outputPath can be null" am: 0c189de962 am: 6f0d10dbde
am: 084434a475

Change-Id: I50dc3cfa3f19ae0bc45269b1f15a45a6328106b9

Merge "Installd: deleteOdex's outputPath can be null" am: 0c189de962
am: 6f0d10dbde

Change-Id: Id73731fd8d7e2e67d17f9ac3599d27520849ac2c

Merge "Installd: deleteOdex's outputPath can be null"
am: 0c189de962

Change-Id: I2220e465027a31b84a9bb473e18e4cfd4fdb0dc7

Merge "Installd: deleteOdex's outputPath can be null"

Merge "Sync up the DvrConfig struct with Google3"

Disable in-place vdex update

Vdex files can be corrupted by dexlayout or by previously dexlayout-ed
files being recompiled if they are used as both input and output.
Disable this feature.

Bug: 38113240
Test: OTA and dexopt
Change-Id: Iee2d9a6c1ba8cddda25290f68f50649d5288d588
(cherry picked from commit e446fede9258359b61d4c8753d7e96477fa29388)

Merge "Add dvrBufferGlobalLayoutVersion which returns the version number for the global buffers."

Merge "Fix the build"

Fix the build

Bug: n/a
Test: compile
Change-Id: I39179d83ae748434dcfd2043143e7154053071d7

Sync up the DvrConfig struct with Google3

Bug: 38506720
Test: No functional change. Just make.
Change-Id: Ib159dd156cef353a7881bc6b4004c46da5e2d449

Merge "surfaceflinger: fix layer count"

Merge "Use wide gamut only when necessary"

Installd: deleteOdex's outputPath can be null am: c523409f0f
am: 7b28d34ee2

Change-Id: I8d700639d5056dd68b41e01b994f074dafe9bedc

Installd: deleteOdex's outputPath can be null
am: c523409f0f

Change-Id: I36aa3e4c04967567e1f1cfa5f967be416bdeca8e

Installd: deleteOdex's outputPath can be null

Add @nullable, as the outputPath can be null for system apps.

(cherry picked from commit c523409f0fd03aa498cedc486f85e9a4b7257f3a)

Bug: 62236497
Test: m
Test: Fill up /data, apply OTA
Change-Id: I61d86721b485f85195b153d76ce25412dcb68be6

Add dvrBufferGlobalLayoutVersion which returns the version number for
the global buffers.

Bug: 62228826
Test: dvrBufferGlobalLayoutVersion == kSharedBufferLayoutVersion
Change-Id: I08eaca128ea5c84739924ab839651088e10c65bc

Delete vdex file in delete_odex. am: 3040d9db98
am: eea2061b34

Change-Id: I20ecc830c5607a24be3f194bf665618e8091b6ad

Delete vdex file in delete_odex.
am: 3040d9db98

Change-Id: I5382e82ebf64bf8fc409cb3ba4cb1481cdca0e58

Installd: deleteOdex's outputPath can be null

Add @nullable, as the outputPath can be null for system apps.

Bug: 62236497
Test: m
Test: Fill up /data, apply OTA
Change-Id: I61d86721b485f85195b153d76ce25412dcb68be6

Delete vdex file in delete_odex.

(cherry picked from commit 192fb9662ce8b2a60da2b280d8000283fb3b6f6b)

bug: 38513395
Test: unfortunately no unit test here, so after discussing
      with agampe@ just relied on build.

Change-Id: I305ddaadc813673fb55473f5e8ca357f39cf98f3

surfaceflinger: fix layer count

When a layer is removed, mLayerCount is always decremented by 1 even
though the layer has children.  This commit fixes that by making
sure mLayerCount is properly decremented, and makes sure that a
parent layer must be a valid layer in mCurrentState.

Bug: 37121786
Test: manually open and close Camera app
Change-Id: I5a3e18be41646b167b4d361dfe08db4d06e1c2d0

Merge "Add documentation for the dvr_hardware_composer_client"

dumpstate: use tombstoned/debuggerd for java traces.

- debuggerd_dump_traces now supports Java traces as well, so
  use that when dalvik.vm.stack-trace-dir is set. The output
  FD we use for the intercept is currently a regular file
  written by mkostemp, but this can be cleaned up once the
  old way of doing things is removed.

- We're no longer writing traces to a global trace file, so
  add bug report entries for all java traces written in the past
  30 minutes just as we do for native traces.

The minor refactoring that this CL undertakes was done under the
assumption that the old way of doing things will be removed in the
near future.

Bug: 32064548
Test: manual

Change-Id: I8af6f0a644115296dc41affc3b7cd98a2db32c48

Merge "Remove error when trying to get a global buffer that doesn't exist"

libgui: Add Surface::getLastDequeueStartTime am: 932f008485
am: 46b9841653

Change-Id: Id3dd6920b3341c5675b5308a9c80e276a5b03ce2

libgui: Add Surface::getLastDequeueStartTime
am: 932f008485

Change-Id: I77404c1b97cb997a057558362b9bfdf9df65c8da

Use wide gamut only when necessary

Set the dataspace of the screenshot based on the result of
the composition. When no wide gamut app is visible, the screenshot
will be in sRGB, otherwise in Display P3. On device that do not
support wide gamut, the color space will be unknown (native).

Bug: 29940137
Test: screencap and Android Studio
Change-Id: I93f3f8e5afebb9f3f17b835fdf5bc215b0856d55

Merge "Disable in-place vdex update" am: d9917d78a0 am: 2521871a1f am: b8e9326fd8
am: 8425a02d5c

Change-Id: I7ca8d02b2e43ae1a2d69aeb0551f7297f6da83ee

Merge "Disable in-place vdex update" am: d9917d78a0 am: 2521871a1f
am: b8e9326fd8

Change-Id: Ia2297f8cc31830e4e3909580f3af45dfb96e5901

Merge "Disable in-place vdex update" am: d9917d78a0
am: 2521871a1f

Change-Id: I9c4d606bcb054ad96b28a4da76a7ddf241d8687b

Merge "Disable in-place vdex update"
am: d9917d78a0

Change-Id: Ieb4a1235dc99cac32b6fb441ee98e0cbdf4b7661

Remove error when trying to get a global buffer that doesn't exist

* Calling Get on a missing buffer isn't system error,
  The error should be handled by the caller

Bug: None
Test: None
Change-Id: I0a49a4d104e9db025f4cfd213182f565d141bbf8

Merge "Disable in-place vdex update"

Merge "Various fixes for wide color gamut rendering"

libgui: Add Surface::getLastDequeueStartTime

Adds the getLastDequeueStartTime method, which allows the caller to
determine whether the last dequeueBuffer call fell into a given time
window.

Bug: 62213889
Test: SurfaceTest.TestGetLastDequeueStartTime in libgui_test
Change-Id: I1684deb09273745a3adc751554624d2257fa890e

Add documentation for the dvr_hardware_composer_client

Bug: 62058491
Test: Compiled
Change-Id: Ifcaf7a495aa733217c46c29b308947efae94fd32

Various fixes for wide color gamut rendering

This CL addresses multiple issues:
- A logic issue where the wide gamut color mode was not set at the right time
- The presence of scRGB surfaces was not detected
- The sRGB to Display P3 matrix was transposed
- The color matrix was applied in gamma space instead of linear space*
- The GPU code path was doing a division by w for each pixel when a
  color transform is set, which shouldn't be necessary (the code now
  checks that the matrix has the expected form)
- Incorrect comment
- Dead code in Description.cpp (mDirtyUniforms was never used)
- Screenshots were taken using the last render engine configuration,
  the configuration is now properly set every time

* This can in theory cause a discrepancy when we switch to/from hardware
  composer mode. I was not able to create a visible discrepancy in practice
  using Night Light, color blindness compensation modes and color inversion.
  More importantly, how/where the hardware composer applies the color
  transform is not specified: it could be gamma or linear space, before or
  after the hardware color LUT. We'll address this in a future CL if needed.
  In addition, this code assumes that fp16 surfaces are encoded in non-linear
  space (scRGB-nl instead of scRGB) but we do not have EGL/Vulkan extensions
  to specify this behavior. We need to address this as well

This CL also fixes potential divides by 0 in the GPU code path.

Bug: 29940137
Test: CtsUiRenderingTestsCases, CtsGraphicsTestCases

Change-Id: I9ae15850f8b9d48c39ebc2724ca3da202be9b008

Merge "Consistent "low storage" behavior." into oc-dev am: 1e4088fc68
am: 666ff7278b

Change-Id: Ifeab043cb89ce953845a6db67cde1100ec7f715d

Merge "Consistent "low storage" behavior." into oc-dev
am: 1e4088fc68

Change-Id: I8a2328710d4bc770badfcc9778534fccaaf91405

Merge "Consistent "low storage" behavior." into oc-dev

Merge "dumpstate: Track debuggerd API changes." am: ccc5db441b am: 62b9a75c4c am: ea3784b686
am: aece2afa84

Change-Id: I4c7780e8f1bcea90a41d0b5b720292602caa8d55

Merge "dumpstate: Track debuggerd API changes." am: ccc5db441b am: 62b9a75c4c
am: ea3784b686

Change-Id: I82e5974f96a2a617d31a63813dac22035532c59b

Merge "dumpstate: Track debuggerd API changes." am: ccc5db441b
am: 62b9a75c4c

Change-Id: Id1320f3373a4bbcdcfd945ea9299bd8a805adc12

Merge "dumpstate: Track debuggerd API changes."
am: ccc5db441b

Change-Id: I05214262704a572a4645df427a76227f8d9eeac4

Merge "dumpstate: Track debuggerd API changes."

dumpstate: Track debuggerd API changes.

dump_backtrace_to_file_timeout now needs an additional argument
that specifies the dump type.

Bug: 36024548
Test: make; manual bug report capture
Change-Id: I6ce3c172dd571e296d9ced9c576b88ea487b2fbb

Merge "EGL: Do not look up default namespace when sphal is defined" into oc-dev am: 74a4049e5a
am: 63a869d221  -s ours

Change-Id: I8bf781dbb9896b6b3f8c3fd367667fa4aa7e7a65

Merge "EGL: Do not look up default namespace when sphal is defined" into oc-dev
am: 74a4049e5a

Change-Id: I34fa6f48116712bb44db77923fcedcd4f93b3781

Merge "EGL: Do not look up default namespace when sphal is defined" into oc-dev

Merge "Mark libui as vendor_available" into oc-dev am: 375a5b563c
am: 849d806329  -s ours

Change-Id: Icfb037b98d26bdc582fa3432a91ccf80ef2decdd

Merge "Mark libui as vendor_available" into oc-dev
am: 375a5b563c

Change-Id: I170d012b8b13b914c2d228dfa3255b23cd5d0900

Merge "Mark libui as vendor_available" into oc-dev

Merge "add missing lib dependency to libvndksupport" into oc-dev am: 09e388ffeb
am: fde18d5e75

Change-Id: Ie32cadf7fd42babdabd1566412a4ac82fe6d7849

Merge "add missing lib dependency to libvndksupport" into oc-dev
am: 09e388ffeb

Change-Id: I2030be9640c8fc01416208f1e114719b8784c6ad

Merge "add missing lib dependency to libvndksupport" into oc-dev

Consistent "low storage" behavior.

When freeing cached data, the caller can now provide a "reserved"
size which we won't clear cached data from.

Bug: 38008706
Test: cts-tradefed run commandAndExit cts-dev -m CtsJobSchedulerTestCases -t android.jobscheduler.cts.StorageConstraintTest
Test: cts-tradefed run commandAndExit cts-dev -m CtsAppSecurityHostTestCases -t android.appsecurity.cts.StorageHostTest
Change-Id: Ieb91b3e5345a950d4785fd7915f520f0a68a567a

add missing lib dependency to libvndksupport

libutils is now depending on libvndksupport. So any client that
statically links to libutils needs to link against libvndksuppor as
well.

Bug: 62084107
Test: start an app, run 'adb shell atrace gfx --stream'. atrace from the
graphics driver are visible.

Change-Id: I98917dc92a1b8bde52e69e1923a1c1530a088bd6

Disable in-place vdex update

Vdex files can be corrupted by dexlayout or by previously dexlayout-ed
files being recompiled if they are used as both input and output.
Disable this feature.

Bug: 38113240
Test: OTA and dexopt
Change-Id: Iee2d9a6c1ba8cddda25290f68f50649d5288d588

use 64-bits usage bits almost everywhere

Revert "Revert "use 64-bits usage bits almost everywhere""
This reverts commit 1671de0068de40f57288628d4b757a1c84962a62.

Test: manual, build
Bug: 33350696
Bug: 38466700
Change-Id: Ia2f51b97137dba754b1938dfb1e235f4babc8a88

Merge "Add pids for processes that dlopens implementations" am: 2410cd179d am: 977066464e am: 479f9d0c7b
am: 564a599e94

Change-Id: I0af1a0c6c899a7d6344977b73d2457b01d1d2ceb

Merge "Add pids for processes that dlopens implementations" am: 2410cd179d am: 977066464e
am: 479f9d0c7b

Change-Id: I896d198a1680ad1a5020a712f925db3c3dfc2084

Merge "Add pids for processes that dlopens implementations" am: 2410cd179d
am: 977066464e

Change-Id: Ib2b23816c0032bc6303380f05ee1bb4b45de7f43

Merge "Cleanup dvr api headers for exporting"

Merge "Add pids for processes that dlopens implementations"
am: 2410cd179d

Change-Id: Ifd75354a22ef848b28113e2d1d1ccf00baf1e976

Merge "Add pids for processes that dlopens implementations"

Merge "Playback activity monitoring: add player types" into oc-dev am: 8caf56ef7c
am: 883d667c42

Change-Id: Ifb03e4fbfce459cbe6ad245f3557b2267591cbbc

Merge "Playback activity monitoring: add player types" into oc-dev
am: 8caf56ef7c

Change-Id: Iae04c321f57e33019861ebf60ea29256b0d63e29

Merge "Playback activity monitoring: add player types" into oc-dev

Merge "atrace: trace binder_set_priority."

Merge "Remove GLES2Renderer dependency on config stores"

Cleanup dvr api headers for exporting

Bug: b/38379392
Test: ran generate_dvr_api.py
Change-Id: I8f07e0c28da77df7b1beccfad427f445ad59e64a

Add pids for processes that dlopens implementations

Test: lshal -itrpc
Test: lshal
Test: lshal --init-vintf
Test: lshal_test

Bug: 37639822
Change-Id: Ia415e5f8dbcd10f24e9a5cfdd591eb829a1da89e

Merge "Move virtual touchpad IDC file to be shared across devices."

atrace: trace binder_set_priority.

Useful to debug priority inheritance issues.

Test: binder_set_priority events traced
Change-Id: I89d6ad463aa670ce9b838cb051e58d58e80f07f6

Merge "Delete vdex file in delete_odex." am: 99dedeb690 am: f6b79dd073 am: 6dc664460d
am: 362e56e41a

Change-Id: I7594ff3af6fc72e4d155afab41ef3ae4cb1b60ef

Merge "Delete vdex file in delete_odex." am: 99dedeb690 am: f6b79dd073
am: 6dc664460d

Change-Id: I0116a003cd1985e15d2beb9a60a4262549647774

Merge "Delete vdex file in delete_odex." am: 99dedeb690
am: f6b79dd073

Change-Id: I28ec2908c795bf3e88960732d69e2945b1eac422

Merge "Delete vdex file in delete_odex."
am: 99dedeb690

Change-Id: Idccc7fa241cdbb72b0ac5672978350a269ac8378

Merge "Delete vdex file in delete_odex."

Delete vdex file in delete_odex.

bug: 38513395
Test: unfortunately no unit test here, so after discussing
      with agampe@ just relied on build.

Change-Id: I305ddaadc813673fb55473f5e8ca357f39cf98f3

EGL: Do not look up default namespace when sphal is defined

If sphal name space exists, try to load drivers from sphal.
This patch stops loading the driver if it fails to find the driver
from sphal, instead of falling back to the default namespace.

Bug: 38435840
Test: sailfish builds and boots
Change-Id: Ib8ba73f57aa25431c4a2efe7c61a3699237cc3d2
Merged-In: Ib8ba73f57aa25431c4a2efe7c61a3699237cc3d2
(cherry picked from commit b732030e847e74f00ecd37ff15a36b103efd97b9)

Merge "EGL: Do not look up default namespace when sphal is defined"

Merge "Add thread usage output to lshal." am: 4a3e0a9767 am: 9a130151a2 am: de314f2df2
am: 42f3d38f35

Change-Id: I5160ee0c8a8f309ffab6a854c9d03c67a88b84fe

Merge "Add thread usage output to lshal." am: 4a3e0a9767 am: 9a130151a2
am: de314f2df2

Change-Id: I321f393d2fc7252ea6d009c743fe797b083d718a

Merge "Add thread usage output to lshal." am: 4a3e0a9767
am: 9a130151a2

Change-Id: I8045623cc47804816df7304cabea1633a18aeada

Merge "Add thread usage output to lshal."
am: 4a3e0a9767

Change-Id: Ie0a5eb6b3f1c502e9cf6c70e10354ce133985348

Merge "Add thread usage output to lshal."

Merge "Newly create idmap only when it is stale" into oc-dev am: 4bf5dcce19
am: 951a77b675

Change-Id: Iaf155251ac7d62b236f356fd0f66f1f621038b9c

Playback activity monitoring: add player types

Add player types for playback activity monitoring:
 - AAudio
 - hardware sources
 - proxy for external players

Test: n/a
Bug: 62027849
Change-Id: I17f4879dd79cc89eb7db373754a79e2e38c97457

Merge "Newly create idmap only when it is stale" into oc-dev
am: 4bf5dcce19

Change-Id: Ibafdbb1406b822a7e7f50ca229b8aefe9981b547

Merge "Newly create idmap only when it is stale" into oc-dev

Merge "DvrBufferQueue: Add basic event callbacks"

libgui: fix BQ buffer allocation error checking am: feec3b1a49
am: cc09fccf62

Change-Id: I5203cd613cb764a8d72d98532d43ec7826f79c57

libgui: fix BQ buffer allocation error checking
am: feec3b1a49

Change-Id: Ie63b629a8eb0f4c08eadbcf3a40ceaf8346a71ba

Merge "sensors: pass sensor handle along with injected event"

Move virtual touchpad IDC file to be shared across devices.

Bug: 62066423
Test: observe IDC appears in targets
Change-Id: Ic990cccbcf8f4f7862ff0a9a59fa76a90800f339