Merge changes I87a57825,I3721d163 into oc-dev
am: e44ea13766

Change-Id: I258fd8dd5fd03975a28c606dcd022ef506554fb6

Merge changes I87a57825,I3721d163 into oc-dev

* changes:
  SurfaceFlinger: Fix final cropping of child layers.
  SurfaceFlinger: Update parent pointer while performing transaction.

Merge "correct value of TYPE_MAGNIFICATION_OVERLAY" am: 9e94328e22 am: ce9203d628
am: d396aa6869

Change-Id: Ibffb945b0db31ac1a2ff053595e82476cc8a3fa4

Merge "correct value of TYPE_MAGNIFICATION_OVERLAY" am: 9e94328e22
am: ce9203d628

Change-Id: I51e54f6e11287c98ff684616be91efcbfc7b2e1e

Merge "correct value of TYPE_MAGNIFICATION_OVERLAY"
am: 9e94328e22

Change-Id: I993c5680157f45e1912e7a24ae5ee1d6cd652f92

Merge "correct value of TYPE_MAGNIFICATION_OVERLAY"

SurfaceFlinger: Fix final cropping of child layers.

Straightforward mistake documented by the test case. Not sure how it
made it this long.

Test: Included in Transaction_test.cpp. Manual from bug. go/wm-smoke.
Bug: 37511473
Change-Id: I87a57825cca879c5519fe72fef199168f7c23479

SurfaceFlinger: Update parent pointer while performing transaction.

To understand the change at a high level, notice that all the usage
of getParent() is assosciated with the drawing state. We see in this way
that the parent is a part of the drawing state and should only be updated
when transactions occurs.

More specifically we can consider the following scenario:

1. Imagine that we have a visible surface, with a visible child surface
2. Now imagine we create a new surface, which is hidden, and post a buffer to it.
   It is configured such that it would be visible if it were not hidden.
3. We open a transaction
4. We reparent the child from the old surface to the new one
5. We show the new surface
6. We close the transaction

At this point we would expect the child to remain visible, as it is atomically
reparented with the visibility change of the parent. However prior to this CL
we see a flash, as the sequence can continue as follows:

7. Closing the transaction triggers setClientState, causing Layer::reparentChildren
   which would call setParent updating the parent pointer.
8. setClientState updates the current but not drawing visibility of the new parent and marks
   a transaction to occur.
9. We return to the main thread, MESSAGE_REFRESH occurs before MESSAGE_TRANSACTION
   and we render a frame
10. We observe the child window as invisible, as it has the new parent
    but it's new parent is not yet visible.

We simply have to ensure the parent pointer is updated at transaction time.
I chose a location in commitChildList where mDrawingChildren was copied
from mCurrentChildren as it seemed to express the intent well.

Test: Difficult to automatically test. Manually try resizing Chrome/Youtube/etc in docked stack before flicker should be ~1/10 after flicker should be <1/100 (I haven't seen one yet)
Bug: 62099658
Change-Id: I3721d16361b53128146510d3fda43e1c12f93223

correct value of TYPE_MAGNIFICATION_OVERLAY

It should be sync to the same value in WindowManager.java
or the InputWindowInfo::isTrustedOverlay() function will fail

Change-Id: I11bb518049724252f2795a6400efc17ddd545fbb

Merge "egl: handle failed eglMakeCurrent EGL_NO_CONTEXT" into oc-dev
am: 2153f5e317

Change-Id: I6c36e59dc595bd6a494561a49b98b0f2d479ecfe

Merge "egl: handle failed eglMakeCurrent EGL_NO_CONTEXT" into oc-dev

Disable in-place vdex update
am: 3291cd9eaa

Change-Id: I9289c4cd1dea5299c842a935943101370ca609e4

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"

egl: handle failed eglMakeCurrent EGL_NO_CONTEXT

The recovery code for failed eglMakeCurrent didn't correctly
handle the case when the previous context was EGL_NO_CONTEXT.
Leave the original EGL context unrestored in that case.
The driver should be able to handle that case without help.

Bug: 38341617, 37244059
Change-Id: I38f3450005a4eec5f1a684ee7d66363e88f6d92f
Fixes: 38341617
Test: android.server.cts.ActivityManagerDisplayTests, Run Minions Rush app.

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)

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

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

libgui: Add Surface::getLastDequeueStartTime
am: 932f008485

Change-Id: I77404c1b97cb997a057558362b9bfdf9df65c8da

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

Merge "Disable in-place vdex update"

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

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

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

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

Change-Id: I170d012b8b13b914c2d228dfa3255b23cd5d0900

Merge "Mark libui as vendor_available" into oc-dev

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

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 "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

Change-Id: Iae04c321f57e33019861ebf60ea29256b0d63e29

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

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 "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 "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."

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

libgui: fix BQ buffer allocation error checking
am: feec3b1a49

Change-Id: Ie63b629a8eb0f4c08eadbcf3a40ceaf8346a71ba

libgui: fix BQ buffer allocation error checking

The error path was broken since we moved away from
IGraphicBufferAlloc.

Bug: 37956614
Test: simulate buffer allocation failures
Change-Id: I1e1ec840fafd3075b84126c6dbf6194568175a42

Merge "SF: Call makeCurrent after presenting each display" am: 7bb8779a79 am: 5079756e8e
am: 53df8fac8b

Change-Id: I9860424ffa54807eccc6ef7ab4dbba99ed18d46a

Merge "SF: Call makeCurrent after presenting each display" am: 7bb8779a79
am: 5079756e8e

Change-Id: I6075c26cb0cd73de62f1607712c720e2d2a6248c

Merge "SF: Call makeCurrent after presenting each display"
am: 7bb8779a79

Change-Id: If22c0369b968a72dcbac8b97061f5555c18e8432

Merge "SF: Call makeCurrent after presenting each display"

Merge "Delete SurfaceFlinger BYPASS codepath" into oc-dev
am: d8ac4fd17a

Change-Id: Ieae1107f4da7b4a3eaa489175bda4bdb62e9d245

Merge "Delete SurfaceFlinger BYPASS codepath" into oc-dev

Add thread usage output to lshal.

This output shows how many threads are in use at a give time.

Test: lshal, manual
Bug: 35099601
Change-Id: I3a22bb131b828cdd77e73e7810229d9c68a496d3

Merge "libgui: Add test for OOB slot number in dequeue" into oc-dev
am: c1225bdafc

Change-Id: Ia6a3f61406928e3293423a4d8981a4ef594779ca

Merge "libgui: Add test for OOB slot number in dequeue" into oc-dev

SF: Call makeCurrent after presenting each display

HWC can change EGLContext in each present. Restore to
default EGLContext after commiting each display.

Test: see https://android-review.googlesource.com/#/c/372703/
Change-Id: I4abc84fb3d4b74964fabcad5a2568ff01a6c5dbb

Merge "Fix a anr bug caused by sendFinishedSignal logical error" am: 77bee3b429 am: 9d5bddcded
am: 251baedfaf

Change-Id: I34210fcd813c2118f0fa4814884b4adadacc415f

Merge "Fix a anr bug caused by sendFinishedSignal logical error" am: 77bee3b429
am: 9d5bddcded

Change-Id: I9ff24304c580bba54afe9b0f4bc4394d513c04cf

Merge "Fix a anr bug caused by sendFinishedSignal logical error"
am: 77bee3b429

Change-Id: If794d0ec576c91fc8554bf974d7409966b306cba

Merge "Fix a anr bug caused by sendFinishedSignal logical error"

Delete SurfaceFlinger BYPASS codepath

Test: AUPT, Manual
Change-Id: I8d6139fbb8a3ec113d7cb973420a5874d3e31030
Merged-Id: I8d6139fbb8a3ec113d7cb973420a5874d3e31030

Newly create idmap only when it is stale

For now, OverlayManagerService calls Installd.idmap() whenever a user
is changed, and then a idmap is re-generated even though there are no
changes on its target apk and overlay apk.
This CL is to avoid such unnecessary re-generation of idmap. Instead
only a group id will be updated when the idmap isn't outdated.

To correctly check staleness of idmap, "--verify" function of idmap is
invented as well.

Test: building succeeded and tested on sailfish.
Bug: 37179531
Change-Id: I874be9765d37bfa6c562c3f39a395040dc6a7d1f

Merge "Do not set the trace clock in atrace.rc." into oc-dev
am: 27ef38ca4c

Change-Id: I068e661eba9c7e668a6157dc4d704b4b73e8514c

Merge "Do not set the trace clock in atrace.rc." into oc-dev

Merge "Surface: Use buffer id to synchronize slots" into oc-dev
am: adf0ad2090

Change-Id: I01b8012c941001530678f944670a083e34b7ebc6

Merge "Surface: Use buffer id to synchronize slots" into oc-dev

Do not set the trace clock in atrace.rc.

This reverts only the atrace.rc change from commit ea826794184be368f65f099ceac4fc31234700f6.
"Set the atrace clock to the best available value: boot, mono, or global."

This change caused a 100+ms regression in boot time. We also confirm that
the clock is set to the correct value when we call atrace, so we will
still see correct behavior after reverting this change.

Bug: 38504158
Test: Manually tested that the trace clock is updated when atrace is
called.
Change-Id: Ifa302694964a4f9fc54db16e7125d5279067bbfd

Surface: Use buffer id to synchronize slots

For the same buffer, GraphicBuffer's buffer_handle_t* may change when
doing attach/detach. Use Id instead.

Test: Call attach/detach, and observe mRemovedBuffers is updated
Bug: 38238747
Change-Id: I6ab1ae5340565f98cb981b7e38c31a4f85190e56

Merge "SurfaceFlinger: setGeometryAppliesWithResize crop latching fixes." into oc-dev
am: e2fad8108e

Change-Id: I3fae02f6d1c03e69f55a05bfdf6229580ae1c4ee

Merge "SurfaceFlinger: setGeometryAppliesWithResize crop latching fixes." into oc-dev

Otapreopt: Add some static_asserts
am: ef21fd2b6b

Change-Id: I9f72d69a74a945f3c822401e466028d94330fdb3

Otapreopt: Add some static_asserts

Add some checks that indicate version issues.

Bug: 38501464
Test: m
Change-Id: Iec2043eda19126166139d7220618d11032e98af0

Fix a anr bug caused by sendFinishedSignal logical error

Because of eliminate multiple benign overflow conditions see:
https://android-review.googlesource.com/#/c/172237/
Changed the do while loop resulting in a logical difference.

When the chainIndex-- to 0, the loop is not run.
It is the most important cycle, it will push head seq to mSeqChains.
If not run, will lead to a batch of seq can not corrected finish.
Eventually leading to the occurrence of anr.

Signed-off-by: gaoshang <gaoshang@xiaomi.com>
Test: Input dispatche process
Bug: 38366215
Change-Id: I87e609dfcb00ac7b8e82c6de789df094e7c25efd

Merge "vulkan: Translate usage flags before passing to driver" into oc-dev
am: b667c05f03

Change-Id: I974356cbc64e3e52b52bb85defa2f548286ece63

Merge "vulkan: Translate usage flags before passing to driver" into oc-dev

Merge "Set the atrace clock to the best available value: boot, mono, or global." into oc-dev
am: bfe6bf6f31

Change-Id: I4b198e07cdf1742aff8a771721021ca6ba28f20a

Merge "Set the atrace clock to the best available value: boot, mono, or global." into oc-dev

vulkan: Translate usage flags before passing to driver

Previously we just cast the 32bit flags and jammed them in the two
gralloc1 slots. Use the helper which does the correct translation.

(Yes, all these gralloc1isms are going away in favor of a single 64bit
flags field, but Vulkan HALs expect us to implement our side of
ANDROID_native_buffer correctly.)

Test: boot sailfish, run vulkan demo
Bug: 37686587
Change-Id: I806011986e8140d7f608d281798605bd9ca9a86b

Merge "DO NOT MERGE Revert "Logging for catching memory issue in Parcel"" into oc-dev
am: 1967749cdc

Change-Id: I1ef9cd112030f317518c483364a9fb505a40d7bc