SF: Fix for a leaked mock reported in processesVirtualDisplayAdded

I'm not sure why it only started being reported  recently, but removing
the injected virtual display from the SurfaceFlinger current/drawing
state fixes the leak.

Bug: None
Test: adb shell libsurfaceflinger_unittest
Change-Id: Ibaf1a8878e296707b9fbbd0a647908cc1205cdc2

Merge "[SurfaceFlinger] Add GPU protected content support."

Merge "Game Driver: add GpuStats class to process and dump stats"

Merge changes I7c51f161,I9231f4b4

* changes:
  SF: Remove LayerBE
  SF: Refactor onLayerDisplayed

Merge changes from topic "display_policy"

* changes:
  SurfaceFlinger: add DISPLAY_EVENT_CONFIG_CHANGED
  SurfaceFlinger: add setAllowedDisplayConfigs

Merge "Don't store empty objects on SF traces"

Game Driver: add GpuStats class to process and dump stats

Bug: 123529932
Test: adb shell dumpsys gpu
Change-Id: I2d524b1eecb00be71d344c07e8e18244a44bbcb8

Merge changes from topic "frameworks/opengl/sync-gen"

* changes:
  glgen: sync generator with manual changes
  Sync generator script with manual Java changes

Merge "Use DEX2OATBOOTCLASSPATH in installd." am: 95ea7b751a am: 0fde25e90d
am: 8525a5557e

Change-Id: Ibeec24813af48a5142959fd77ec206486afed204

Merge "Use DEX2OATBOOTCLASSPATH in installd." am: 95ea7b751a
am: 0fde25e90d

Change-Id: I6d71f82f41f4341f9866591f978de333bf8092a5

Merge "gui: add test for region sampling"

Merge "Use DEX2OATBOOTCLASSPATH in installd."
am: 95ea7b751a

Change-Id: Iddddc8bbef7feb2188b991d46ef4c3d28d50011c

[SurfaceFlinger] Add GPU protected content support.

Switch to protected context when a layer with protected contents presents.
Previously we black out layers with protected contents, now that we add support
of protected contents in GPU composition, we can turn it on. In screenshot
cases for Recents, we still black out layers with protected contents.

BUG: 35315015
Test: Build, flash and boot
Test: Watch Youtube movie, verifed by force GPU composition.
Change-Id: I556bdd6f82c06c3d54ce62fab06a8c6f47599dcf

Merge "Use DEX2OATBOOTCLASSPATH in installd."

Merge "Add output fb cache to renderengine"

glgen: sync generator with manual changes

Syncs generator to accomodate manual changes to checked-in sources.

Adds note to EGL15.cpp that file is automatically generated.

Bug: 126352534
Bug: 124064285

Test: gen and diff based on generated command lines.
Change-Id: I31909de4c077679dec0323761fb06f388d128a14

Sync generator script with manual Java changes

Incorporate manual changes to generated Java files back into the
generator.

Bug: 126352534
Bug: 124338141
Test: ./gen and check output has no commands for java files to be run.

Change-Id: Ie5f48d704b1485316718ae19b20016a08e688916

Merge "Query the enable_apex_image flag from native experiment." am: 2cb4a0538f am: 15326fb133
am: 6186bfd2b6

Change-Id: I1a45757cec416206b34b8c63fb68a9d77242c0c7

Merge "Query the enable_apex_image flag from native experiment." am: 2cb4a0538f
am: 15326fb133

Change-Id: I305d96d9d5764a843cb8a070c70c987723b308be

Merge "Query the enable_apex_image flag from native experiment."
am: 2cb4a0538f

Change-Id: I35770107e30df01ef38c64926eec072b2a76a951

Merge "Query the enable_apex_image flag from native experiment."

Use DEX2OATBOOTCLASSPATH in installd.

Bug: 119800099
Bug: 122937705
Test: dexopt_test
Change-Id: Ie31762910a51ddb1211eed6a24456f0a70379d9c

Merge "add display_primary* to sysprop"

Merge "Whitelist the staging directory in installd" am: 99ba118644 am: 3c86bd1dba
am: 67e702ae57

Change-Id: I5fc45c2b1864b52a8b23dea383c0ba9708e0b716

gui: add test for region sampling

Adds a test that checks the correct luma is produced under a few
scenarios with different layerings.

Bug: 119639245
Test: ./libgui_test --gtest_filter="RegionSamplingTest.*"
Change-Id: Ic85a97f475a3414a79d3719bbd0b2b648bbccfb0

Merge "Whitelist the staging directory in installd" am: 99ba118644
am: 3c86bd1dba

Change-Id: I3541435ea245841079e34ea5bb7dc704dd3330df

Merge "Whitelist the staging directory in installd"
am: 99ba118644

Change-Id: I5be2f0d4a82e7e352453371a3ca5f8d6e46ab2d3

Merge "Whitelist the staging directory in installd"

[automerger skipped] Merge "DO NOT MERGE - Merge pi-dev@5234907 into stage-aosp-master" into stage-aosp-master am: d65a2e6283 -s ours
am: e93f01cd75 -s ours
am skip reason: subject contains skip directive

Change-Id: I0509d61ba16b2919e7145e3763f7054c41464b87

[automerger skipped] Merge "DO NOT MERGE - Merge pi-dev@5234907 into stage-aosp-master" into stage-aosp-master
am: d65a2e6283 -s ours
am skip reason: subject contains skip directive

Change-Id: I4c6db4219a673da35e0666c1b51051d93ab38da9

Merge "Add enum type for supported HDR metadata"

[automerger skipped] DO NOT MERGE - Merge pi-dev@5234907 into stage-aosp-master am: 5f4277821a -s ours
am: 3f29cb565f -s ours
am skip reason: subject contains skip directive

Change-Id: Ic02ee51e4fe9dfe34ad4440da057bd258b6397d1

Add enum type for supported HDR metadata

Test: Builds

Bug: 112508112
Bug: 120990898
Change-Id: If6e01bad9275b057d9bdbe963a25b7a26b7198a2

[automerger skipped] DO NOT MERGE - Merge pi-dev@5234907 into stage-aosp-master
am: 5f4277821a -s ours
am skip reason: subject contains skip directive

Change-Id: I3d4582885b96c96308856ff969fce86a2b51edc3

Don't store empty objects on SF traces

Proto3 doesn't store default values for object, however, it stored the
object start-end tags if an empty object was created.

Do not create objects when they are empty

Test: Flash a device. Record SF trace. Open them in the new winscope
version with "display defaults" unchecked. Check if there are no empty
objects

Change-Id: I3f48047e43e9c62c572d5b83e63a0f96527ef146

Merge "Exclude secure layers from most screenshots taken by the system server."

Merge "Report DispSync model updates in the correct place."

Merge "Rename resampling property"

SurfaceFlinger: add DISPLAY_EVENT_CONFIG_CHANGED

Add a new event to DisplayEventReceiver for display
configuration change. This event is sent by SF when display config
is changed.

Test: adb shell /data/nativetest64/libsurfaceflinger_unittest/libsurfaceflinger_unittest
Bug: 122905403
Change-Id: Ibb7e0ce7b83b91259ccb0e9c982f5e378b0ebfaf

SurfaceFlinger: add setAllowedDisplayConfigs

Add an API to ISurfaceComposer to set allowed display configurations.
This API is expected to be called by DisplayManager depends on the
current policy in place. Once setAllowedDisplayConfigs is called,
SF can only set a new display config if it is part of the allowed
configurations list.

Test: call setAllowedDisplayConfigs() from backdoor and
      observe config change.
Bug: 122905403
Change-Id: I1d0a3649bbe7a08efeb72dc270f0b2df330b021c

Merge "Rename IDumpstateListener errorcode" am: d7d075fbf6 am: c943242ff9
am: c6c5cdde65

Change-Id: I898f3d7363946d9fdda92afac5babf43b5ef2dc1

Merge "Log duration reporter to logcat" am: 86d900ff2d am: 46fe8a946d
am: 66cc69fe9c

Change-Id: I3eba5e7434484ac1add403109b69ce754a3bb26a

[automerger skipped] Merge "installd: parameterize user data snapshots with snapshot_id" am: 8a3c8b1a94 am: bfa5131193
am: a20fc9399a -s ours
am skip reason: change_id I4177a44a60cdfdc5429a28c613ce661b24482b8b with SHA1 b68f0d1493 is in history

Change-Id: I7627b17fffb6749937c925498a06af3721cd92b1

Merge "dumpstate: Fix IO Errors during zip writing." am: 01f59ce0f6 am: 48d93b7298
am: 8cc3318b1c

Change-Id: I9965667c7ce7762d567f1a59a0995324138542da

Add output fb cache to renderengine

On some devices this can shave off multiple milliseconds.

Bug: 123107664
Test: manual tests, systrace
Change-Id: If29b1753f899fec03852fb1ddaaa1a245f68424b

Report DispSync model updates in the correct place.

Otherwise, metrics sometimes aren't updated in the right spot.

Also, add the reference time to detailed trace info, as it's important
for event calculation.

Bug: 124383894
Test: systrace
Change-Id: I4ad834fc2076600116831e0f6c38c6ee8f178c11

Merge "Rename IDumpstateListener errorcode" am: d7d075fbf6
am: c943242ff9

Change-Id: I2a8b3bf992243d3d5508248127e690e050f925e3

Merge "Log duration reporter to logcat" am: 86d900ff2d
am: 46fe8a946d

Change-Id: I787245655aecb9dadc8c868fe63d68a5d147d971

Merge "installd: parameterize user data snapshots with snapshot_id" am: 8a3c8b1a94
am: bfa5131193

Change-Id: Ic32854f8ae690a7ca745a0513515b7c499f59c05

Merge "dumpstate: Fix IO Errors during zip writing." am: 01f59ce0f6
am: 48d93b7298

Change-Id: I503c787c5e24ac2082f387ee2fac319f722cfd94

Rename resampling property

The resampling property is currently '*.noresample', which is confusing
because it is false by default.

Test: enable 'DEBUG_RESAMPLING', in InputTransport,  then touch  the screen to see that
events are being resampled. This can be seen in the logcat.
Next, run 'adb shell setprop ro.input.resampling 0', adb shell stop &&
start and touch screen again. See that resampling is no longer active.
Bug: 119214052

Change-Id: I9cea4515f7b064b6a745abeef3309a6e97205c54

Query the enable_apex_image flag from native experiment.

Bug: 119800099
Test: m
Change-Id: Ibcd90840b4342fae14307ef7f30eec96d58a698b

Merge "Test InputDeviceIdentifier::getCanonicalName"

Merge "Game Driver: add dumpsys interface to GpuService"

Game Driver: add dumpsys interface to GpuService

This change also factored some formats in GpuService.

Bug: 123529932
Test: adb shell dumpsys gpu
Change-Id: I9a89c044cc78a43fa4ff2ce1d389a518679bc6b7

Merge "Merge metadata from transaction instead of replace"

Merge "DO NOT MERGE - Merge pi-dev@5234907 into stage-aosp-master" into stage-aosp-master

Merge "blast: adding a buffer doesn't make the frame dirty"

Merge "Remove spamming logs from ProducerChannel"

Merge "Game Driver: add required sphal libraries to game driver namespace"

Remove spamming logs from ProducerChannel

Change-Id: I86aba99d6c07e0606a6f512129bca48314f16d20
Fix: 125846981
Test: run on walleye_xr target and no more spamming logs.

blast: adding a buffer doesn't make the frame dirty

The frame shouldn't be marked as dirty when a new buffer is set.
When a frame is marked as dirty, it forces SurfaceFlinger to do
extra work on the CPU which hurts power/performance.

Bug: 125859110
Test: Take a systrace when running the Blast Chrome.apk. Check that
      there is no extra work done.

Change-Id: I0cf9802f495a7e9f2e5b1f4e58f0a7a4a75d8225

[sf] Implement addSamplingListener

Implements ISurfaceComposer::addSamplingListener, which allows a client
to receive streaming median luma updates for a given region of the
screen.

Bug: 119639245
Test: Manual using SamplingDemo in libgui
Test: Automated libgui_test in Ic85a97f475a3414a79d3719bbd0b2b648bbccfb0
Change-Id: Ic52359aeab884e734a806372be0eb4e327c45298

Merge "Rename IDumpstateListener errorcode"
am: d7d075fbf6

Change-Id: I2d37f696f9236fe36378a56ee5a0937771ce43a1

Merge "Rename IDumpstateListener errorcode"

add display_primary* to sysprop

The display_primary* properties is added for getDisplayNativePrimaries
function. Since this function is changed from configstore to sysrpop for
removing configstore 1.2

Bug: 124531214
Test: m -j && boot test && atest libsurfaceflinger_unittest

Change-Id: Ibe22aadab1095e624dc498a743f343f301dc4ea6

Merge "Plumb fence from dequeueBuffer into renderengine"

Plumb fence from dequeueBuffer into renderengine

This defers blocking from the ui thread, to the gpu driver, so
SurfaceFlinger can continue work without waiting on an old frame.

Bug: 123107664
Test: manual tests
Change-Id: Ied4ba84dd3fe63c65470ae3396dec0cb667a5ff0

Merge "More accurate frames missed for client composited frames."

Merge "Use InputDeviceIdentifier.getCanonicalName"

Merge "Add synchronous transaction to wait for setInputWindow to complete (4/n)"

Merge "Replace non-alphanumerics with _ in VirtualKeyMap"

Exclude secure layers from most screenshots taken by the system server.

In pre-P versions of Android, it was allowed to screenshot secure layers if the
buffer queue producer which was the target of the screenshot was owned by
the system (in this case SurfaceFlinger). This really was a synonym for:
The screen rotation animation was allowed to capture secure layers, but the other
code paths weren't. In O we mistakenly changed this check to always allow the system server
to capture secure layers via the captureScreen path (the captureLayers path used for
TaskSnapshots was unaffected). This can result in data leakage in cases where the
system server takes screenshots on behalf of other parts of the system (e.g. for
the assistant). To mitigate this we provide an explicit switch for the system server
to specify whether it wishes to capture Secure layers. While this is dangerous, I think
it is less dangerous than the previous implicit switch of capturing secure layers based on which
type of BufferQueue was passed in. The flag defaults to not capturing secure layers
and we set it to true in the one place we need it (for the screen rotation animation).
Non privileged clients can still not capture secure layers at all directly.

Test: SetFlagsSecureEUidSystem
Bug: 120610669
Change-Id: I288ad3bbb0444306e90fe3bb15e51b447539dea5

Test InputDeviceIdentifier::getCanonicalName

We recently added a function to InputDeviceIdentifier from
InputDevice.h. Add a test for that function here.

Bug: 113575658
Test: atest libinput_tests inputflinger_tests
Change-Id: Ifc6aa1a0731a3800526746b989d32c7e72fdb3f3

Use InputDeviceIdentifier.getCanonicalName

Currently, there are 2 places with identical logic in InputDevice.cpp.
This is because the file name adjustment to only contain
alphanumerics, dash, and underscore is done for virtual key maps and for
other devices together.
Use the recently added api to simplify this. Also, this will prevent
having to iterate through Vendor_*_Product_* filenames, since those are
already guaranteed to be compliant.

Bug: 113575658
Test: atest libinput_tests inputflinger_tests
Change-Id: Icb22f6e76b83b6d7d170b9e537787f0455db1c1f

Replace non-alphanumerics with _ in VirtualKeyMap

We currently rewrite device name for finding the matching .idc, .kl,
.kcm files, but we aren't allowing this for virtual key maps.

Rewrite the name for virtual devices to provide some flexibility in
matching for the right files in sys/board_properties.

Bug: 113575658
Test: atest libinput_tests inputflinger_tests
Change-Id: I54d159e9ecf02327cae388ae14c0bcf21c415e6e

Merge "Log duration reporter to logcat"
am: 86d900ff2d

Change-Id: I2ac8e95d3700764b2e98a49bfcd40fe8dbdf4a18

Merge "Log duration reporter to logcat"

Merge "installd: parameterize user data snapshots with snapshot_id"
am: 8a3c8b1a94

Change-Id: Id1393797cda2a83ae39c4dd016657b9ad57b451a

Merge "Refactor VirtualKeyMap"

Merge "Add 2 addtional VkFormat's to vkGetPhysicalDeviceSurfaceFormatsKHR"

DO NOT MERGE - Merge pi-dev@5234907 into stage-aosp-master

Bug: 120848293
Change-Id: Icba28487cde3a6e33898450eaf53ee2be58a3dc8

Merge "installd: parameterize user data snapshots with snapshot_id"

Add synchronous transaction to wait for setInputWindow to complete (4/n)

Wait for callback from InputDispatcher before returning to client.

If the request to SF has the flag syncInputWindows, wait until the
callback setInputWindowsFinished has been called from InputDispatcher
before returning to the client.

Bug: 123041491
Test: Builds, runs
Change-Id: I9d3fb83442cf344c43449bb072a1c31b300b2879

Merge "Add synchronous transaction to wait for setInputWindow to complete (3/n)"

Rename IDumpstateListener errorcode

BUG:123571915
Test: adb shell /data/nativetest64/dumpstate_smoke_test/dumpstate_smoke_test --gtest_filter=DumpstateBinderTest.*
Change-Id: I5533f3bfed647a4068893b61dee371f5a4678650

Log duration reporter to logcat

This is to help debug bugreport generation latency
BUG: 124089395
BUG: 123315882
Test: manual

Change-Id: I65b40054d48d36c7b51b3b78feac2295266a4720

installd: parameterize user data snapshots with snapshot_id

- snapshot_id will be provided by RollbackManagerService and for a given
  rollback will be equal to its rollback_id.
- snapshotAppData now creates a folder to snapshot data to, in case it
  does not exist.
- removed ceDataInode from restoreAppDataSnapshot API, since it's only
  called if users phone is unlocked, meaning that CE data is available and
  doesn't require inode to be accessed.
- refactored installd_service_test to reduce test set up boilerplate.
- added a test to cover case when snapshots for different packages have
  the same snapshot_id.

Bug: 124029909
Test: installd_utils_test installd_service_test, RollbackTest
Change-Id: I4177a44a60cdfdc5429a28c613ce661b24482b8b
Merged-In: I4177a44a60cdfdc5429a28c613ce661b24482b8b
(cherry picked from commit b68f0d14931905472b93ad08a753a01a33bd315b)

Merge "installd: parameterize user data snapshots with snapshot_id"

Merge "dumpstate: Fix IO Errors during zip writing."
am: 01f59ce0f6

Change-Id: I4a051d10c07877ea4fcfa328bef18c4d346c9de1

Merge "dumpstate: Fix IO Errors during zip writing."

Game Driver: add required sphal libraries to game driver namespace

Bug: 124448366
Test: Build, flash and boot. Verify if those libraries are loaded.
Change-Id: I2673c47ee75c08d283efdec650a082b742c45768

Merge "Game Driver: update the format of driver build date"

Merge "SF: correct 90hz debug flags"

More accurate frames missed for client composited frames.

Leaving backpressure to only be for hwc composited frames so that there
is no functional change.

Bug: 123107664
Test: systrace, dumpsys
Change-Id: If343230c19a8619bfca3b2fe42e270734ba0f936

Merge "[RenderEngine] Add OWNERS file to renderengine/ directory."

Merge "SF: Add a flag to guard the video detection feature"

Refactor VirtualKeyMap

Currently, there's a lot of new and delete going on in VirtualKeyMap.
Simplify this by using unique_ptr instead.

Bug: 113575658
Test: atest libinput_tests inputflinger_tests
Change-Id: Ib2a68ae23a4300b0e2cf72902371f4b9604cfee5

Merge "Remove unused defines from Keyboard.h"

SF: Add a flag to guard the video detection feature

Video feature is still under development, so we should not be populating
layer history in scheduler if the feature is not turned on.

See go/surface-flinger-scheduler for more info

Test: manual
Bug: 123956502
Change-Id: I83eae3b5203a4e6b8976f9b7400711963eacbb34