HWC2: Clear transform for SolidColor layers
am: f1098abecf

Change-Id: Iff664310ed2275c0ce9b7afaf8041b8a30ea3a6c

HWC2: Use SolidColor composition for DimLayer
am: 0183f7acac

Change-Id: If9accf1d6ce13c34892ca72bda1778a624542337

HWC2: Kill logspam for non-HWC virtual displays
am: 179533d107

Change-Id: Ib8398383cb606990920033538938855eda248071

HWC2: Fix virtual displays
am: 5450fe5268

Change-Id: I6710be1bb5c5fb998143b4ad898c94a6eee98382

HWC2: Don't set null handle for client layers
am: 6be002845f

Change-Id: I1fc9959536c40511c4fc49fa613d6fd7a97d43ad

HWC2: Hook up to BoardConfig variable
am: 8beb15f220

Change-Id: I012809dac02ab2d87ac1650be2fc619f2b68fbbf

HWC2: Skip validate/present if display is off
am: 00ac359a3b

Change-Id: I31fad8a631fc6b52cb4b6b4cfc26185ebb0214f2

Merge "Fix struct vs. class mismatch in forward definitions"
am: 1d6c149d89

Change-Id: Iba83f822a72416e0b63b796f16065e8a672d66d9

Merge "Fix struct vs. class mismatch in forward definitions"

Fix struct vs. class mismatch in forward definitions

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I46208cec43599f775f7e2cb22b39d19b0ea762e1

Merge "Fix weak vtable warnings"
am: d93a40c920

Change-Id: I54d2cad55131ac3a8786012d692d4627f188a0a7

Merge "Rewrite multicharacter constant using constexpr"
am: 991667ba8c

Change-Id: Iecaee1af6b787187f971e57d8ca3d4d223c79627

Merge "Fix rule of three warnings"
am: dfd195160a

Change-Id: Ifcc0ef0c71f465ada5c62cbcccd40580df802a51

Merge "Fix weak vtable warnings"

Fix weak vtable warnings

Move virtual destructors into cpp file so that the compiler knows which
translation unit to put the vtable into.  Hide the warning for
DeathRecipient, which has no virtual methods to move.  The warnings were
being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I25329d66bfc1f6a5064d05ce7d12ad9b090601f8

Merge "Rewrite multicharacter constant using constexpr"

Rewrite multicharacter constant using constexpr

Multi character constants are implementation defined behavior, use a
constexpr instead.  The warnings were being hidden by the use of
-isystem to include frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I22435d1f66a073c303202da7a1940cb14b0b9fd8

Merge "Fix rule of three warnings"

Fix rule of three warnings

Sensor and BufferItem have destructors, the copy constructor and
assignment operators should not be implicit. The warnings were being
hidden by the use of -isystem to include frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: If375d85cd3d6ded4ee6b27b841bf10f563254682

HWC2: Clear transform for SolidColor layers

Clears the transform for SolidColor layers, since it doesn't make sense
to transform layers without a buffer.

Test: Cherry-pick from internal branch
Bug: 30429522
Change-Id: Ib2f4889531556c7ab00e3b270319c321e9f32c84

HWC2: Use SolidColor composition for DimLayer

Switches SurfaceFlinger to try to use SolidColor HWC device
composition for DimLayers instead of always falling back to client
composition.

Test: Cherry-pick from internal branch
Bug: 30216498
Change-Id: Ia8af95d214ba72b5657f32f7827a1e6278629f17

HWC2: Kill logspam for non-HWC virtual displays

Removes some logspam and fixes some non-errors which were reported as
errors when running a virtual display that is not backed by hardware
composer.

Test: Cherry-pick from internal branch
Bug: 30022738
Change-Id: Ie966e37d95f53bcc050eef064d24bbe26748b7fd

HWC2: Fix virtual displays

Close the output buffer release fence per the contract in
hwcomposer2.h.

Also changes the HWC2 device to hold a weak pointer to the display,
which allows it to properly remove it when it gets torn down.

Test: Cherry-pick from internal branch
Bug: 29945684
Change-Id: Ib1b5d81e42a63d0e65b189f7c6c62e661e0ab6ae

HWC2: Don't set null handle for client layers

Removes the code that explicitly set a null buffer handle for client
layers.

Test: Cherry-pick from internal branch
Bug: 29463310
Change-Id: I570e6041ba72d64971bb6c364e8e7b6f84eeff08

HWC2: Hook up to BoardConfig variable

Uses TARGET_USES_HWC2 to enable the HWC2 code path in SurfaceFlinger

Test: Cherry-pick from internal branch
Bug: 27211042
Change-Id: I7b7e3e3f7a918e74df0be27418196c81b4332564

HWC2: Skip validate/present if display is off

Adds checks to ensure that the display is still on before attempting
to call into prepareFrame/commit (which in turn call into
validate/present).

Test: Cherry-pick from internal branch
Bug: 28596387
Change-Id: Iad87f02c3e44e86db43a15c258ee66b0cde14faa

Merge "Fix implicit sign casts in Parcel.h"
am: 9984fa0ed0

Change-Id: I7a5fb0652ccfd88094ceb2e48a8ff2291166f7f7

Merge "Fix implicit sign casts in Parcel.h"

Fix implicit sign casts in Parcel.h

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I20d9a5712c77894f9048ef78264d7dc9f59208ea

Merge "Installd: Add a delete_odex command"
am: d6df7e33ad

Change-Id: Ife9a2c8c7ee8339dc4e416a397519590fca2cdf6

Merge "Installd: Add a delete_odex command"

Merge "Fix libinput warnings"
am: a4767442e3

Change-Id: I2bf4442e4785a3a15331d7c0482925326a9f38c4

Merge "Fix libinput warnings"

Fix libinput warnings

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I13f03892c75ea3c3139ca676666b0d403871d241

Merge "Fix google-explicit-constructor warnings in  HardwareAPI."
am: c8de4a06ab

Change-Id: Ic8d39038699dd9bfa56bbdeb6f29293bad47d4f5

Merge "Fix google-explicit-constructor warnings in gui and ui."
am: 170454b0fa

Change-Id: I1b95e35d622827f27b1637af00e312c055fd2c27

Merge "Fix google-explicit-constructor warnings in  opengl."
am: 9ccba3c090

Change-Id: Icf73c478090ef837c24aa3b4b75152ad4eef3480

Merge "Fix google-explicit-constructor warnings in  HardwareAPI."

Merge "Fix google-explicit-constructor warnings in gui and ui."

Merge "Fix google-explicit-constructor warnings in  opengl."

Merge "Fix order of operations warning in libui"
am: 73d4186757

Change-Id: Ib93a8ddcb912b05cd38db1dea7ac0ff21a788bcd

Merge "Fix struct vs. class mismatch in forward definitions"
am: 8cdcd15c38

Change-Id: If6768b8a9468833f0fec5025f95c1796dc2869ca

Installd: Add a delete_odex command

Add a command to delete odex files.

Bug: 31347757
Change-Id: I29bca8751bcee8d6981c682fbbc816c73b78ac68
(cherry picked from commit 3964da0636624ff9af42d1f316bed0752e37c490)

Merge "Fix order of operations warning in libui"

Fix order of operations warning in libui

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I21f34567927125d3eeeecd8f00271a175f93af21

Merge "Fix struct vs. class mismatch in forward definitions"

Merge "Add missing liblog dependency"
am: c4dd0c3b83

Change-Id: I710f932670b8374e11a19fe96f2a53b622753468

Merge "Add missing liblog dependency"

Fix struct vs. class mismatch in forward definitions

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: If2bae8b2fbc4238d6251cdd78bc7b5216684f99d

Merge "Include what we use."
am: be7fdfc2a1

Change-Id: Ifb6f3d6424dc3f1295a8c50a8773b58410b562d9

Merge "Include what we use."

Include what we use.

Test: make checkbuild
Bug: None
Change-Id: If803e761c6d7e0c6d043bbcd2ace47f373495db3

Merge changes I7748934f,I008ef537,I1027e87f
am: 132fb22957

Change-Id: Idb0a92939564554becffe4cfa7209f9c220176e6

Fix constructor parameters shadowing member variables
am: 382ecd3d63

Change-Id: I89fc7c1780eab42ab17e028997db2d125f59c2d8

Use uint32_t consistently for region op
am: 8f279960c6

Change-Id: Ibe726f674fc22ca69cd33242ff0059ac6dd70c48

Fix trailing semicolon warnings
am: 17576de056

Change-Id: I6bb34741553c12225d62fd4fdcde74e14502a15c

Merge changes I7748934f,I008ef537,I1027e87f

* changes:
  Fix constructor parameters shadowing member variables
  Use uint32_t consistently for region op
  Fix trailing semicolon warnings

Merge "Migrate libandroid headers to ndk_headers."
am: fd9d1fb52b

Change-Id: I3762d8549bc449283c874f420d0fa72bda40f244

Merge "Migrate libandroid headers to ndk_headers."

Fix constructor parameters shadowing member variables

Using the same name for the constructor parameter and the member
variable causes a warning that was being hidden by the use of -isystem
to include frameworks/native/include.  Prefix the parameter with an
underscore.

Bug: 31752268
Test: m -j
Change-Id: I7748934f1e9515176e2ae98d2c0e58d165a7a5c2

Use uint32_t consistently for region op

An implicit conversion warning was being hidden by the use of -isystem
to include frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I008ef537a24311d58816fae12ef8c4aeaba07547

Fix trailing semicolon warnings

Remove semicolons after inline function definitions.  The warnings were
being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I1027e87f459c36a575cd04b3afb645736832b07b

Add missing liblog dependency

Bug: http://b/31289077
Bug: http://b/27171986
Test: remove liblog from libcutils and recompile
Change-Id: I01e7a0ca758308b5f85f2a08f34a9671eaa48b10

Merge "Add missing liblog dependency"
am: b32a59e797

Change-Id: I8ed998f6342914164f89ce8584bca61b8daa4244

Merge "Add missing liblog dependency"

Add missing liblog dependency

Bug: http://b/31289077
Bug: http://b/27171986
Test: remove liblog from libcutils and recompile
Change-Id: I5d6ded54613e8a29f20495d8f977064f0af2ffd6

Merge "libgui: add a test for Sensor serialization."
am: 01a0c92da0

Change-Id: Idac427f7e0fa170b1a70ad274f39994361fa452d

Merge "libgui: add a test for Sensor serialization."

Merge "Add btsnoop_hci.log to bugreport zip (1/2)"
am: d95b2eb2f6

Change-Id: Ida600de84569296a3c146ca54aba86f9b88a5816

Merge "Add btsnoop_hci.log to bugreport zip (1/2)"

Migrate libandroid headers to ndk_headers.

Test: make native
Bug: http://b/30465923
Change-Id: I0ef0b3630670241c4efd22515b6ab13eb41a81b3

Add btsnoop_hci.log to bugreport zip (1/2)

Bug: 28672558
Test: Manual
Merged-In: I2995a9fd0e8be031c022ef8f275aae35265ba452
Change-Id: I2995a9fd0e8be031c022ef8f275aae35265ba452

libgui: add a test for Sensor serialization.

On a few test boards we've hit a problem with Sensor serialization
causing a SIGBUS due to misaligned access. This is being fixed in
Flattenable.h, but adding a test here to verify the fix and detect
regression seems like a good idea.

Bug: http://b/31671510
Test: adb shell /data/nativetest/libgui_test/libgui_test \
                --gtest_filter=SensorTest.*

Change-Id: I233359b04789dabe2246b95051290e101e0bd2c1

Merge "Rename unique_fd::clear to unique_fd::reset."
am: f9f04003f5

Change-Id: I856ff1bdb5735a90e3b6c13d2f7a2deeff0388e0

Merge "Rename unique_fd::clear to unique_fd::reset."

Rename unique_fd::clear to unique_fd::reset.

Test: m checkbuild
Change-Id: I8cbf2b538c270d8ef50ab6fdcf4fecbcd65899a8

Merge "libvulkan: Fix count returned for incomplete wsi queries"
am: 75bb94fe7e

Change-Id: I0567dca837c6a56d0af14eb0ace73318386b444d

Merge "libvulkan: Fix count returned for incomplete wsi queries"

libvulkan: Fix count returned for incomplete wsi queries

When vkGetPhysicalDeviceSurfaceFormatsKHR,
vkGetPhysicalDevicePresentModesKHR, and vkGetSwapchainImagesKHR
returned VK_INCOMPLETE, they would overwrite the count parameter with
the number of available items, rather than the number of returned
items.

Change-Id: I6a736770f90b95ad15bfcfbe5afb4d2886817dcb
Fixes: 31490492
Test: https://gerrit.khronos.org/#/c/158/

Merge "Fix google-explicit-constructor warnings in binder."
am: 8b2d3da4c4

Change-Id: I3293141f444e547957df9579fe6d258df696bbe7

Merge "Convert libui to Android.bp"
am: f0346ab68e

Change-Id: I1c38e092cc404510e038e246e8d67c9cd28faaed

Merge "Fix google-explicit-constructor warnings in binder."

Merge "Convert libui to Android.bp"

Convert libui to Android.bp

Along with the libinput tests, which depend on libui. For more
information on Soong, see build/soong/README.md

Change-Id: Id4fbeb114d03e37556dca59a2c9f5c9abb0a1825
Merged-In: Id4fbeb114d03e37556dca59a2c9f5c9abb0a1825
(cherry picked from commit 871dda3d6485617bf3a1ea6c8236071212234433)

Merge "Sensor: Make switching from data injection mode to normal mode more productive"
am: e6a4e9b04e

Change-Id: Ic5b3c3bb1856d70a0e12878f8ecdd2cf81868152

Merge "Sensor: Make switching from data injection mode to normal mode more productive"

Merge changes I5041c541,I0eeafc99,Ie982d375
am: 30fcc9fc7f

Change-Id: If83104d4b621a6e29782f45f62d40d2899e480b5

Squashed: gralloc1/binder/requestor
am: f62eaf5994

Change-Id: Icc0f8ccf6ce1799ce83655cfc60eb5f5da0d9f12

libui: Add Gralloc1On0Adapter and C++ shim
am: eb03fd3a8b

Change-Id: I3bc1a4a54d4c9b7d8de0fb45140e7f3d8c194319

libagl: Switch from gralloc to GraphicBufferMapper
am: 204240a88b

Change-Id: I9f43911878549a281050b86af9b4430a35b15c04

Merge changes I5041c541,I0eeafc99,Ie982d375

* changes:
  Squashed: gralloc1/binder/requestor
  libui: Add Gralloc1On0Adapter and C++ shim
  libagl: Switch from gralloc to GraphicBufferMapper

Merge changes I1b74d58b,I38c6e8bc
am: 9f41107732

Change-Id: Ib1a09ccb5773c4f6c01ad1d39b43c6f85f16092d

ServiceManager: Allow system services running as secondary users to add services
am: 6b9c6d23f6

Change-Id: I556897757f0a1d019bbf269e8e32644d346119ba

ServiceManager: Restore basic uid check
am: 5fa90a063f

Change-Id: I64b3db319f75d1f07e8439e393e5c3c8d5b192e2

Merge changes I1b74d58b,I38c6e8bc

* changes:
  ServiceManager: Allow system services running as secondary users to add services
  ServiceManager: Restore basic uid check

ServiceManager: Allow system services running as secondary users to add services

This should be reverted when all system services have been cleaned up to not
do this. A process looking up a service while running in the background will
see the service registered by the active user (assuming the service is
registered on every user switch), not the service registered by the user that
the process itself belongs to.

BUG: 30795333
Change-Id: I1b74d58be38ed358f43c163692f9e704f8f31dbe

ServiceManager: Restore basic uid check

Prevent apps from registering services without relying on selinux checks.

Bug: 29431260

Change-Id: I38c6e8bc7f7cba1cbd3568e8fed1ae7ac2054a9b

Squashed: gralloc1/binder/requestor

This is a combination of the following three commits:

libui: Convert Allocator and Mapper to gralloc1

Converts GraphicBufferAllocator and GraphicBufferMapper to speak
gralloc 1.0 (via the C++ shim and optionally the 1On0 adapter) instead
of gralloc 0.x.

-----

Create graphic buffer using binder interfaces

Using binder interfaces rather than directly allocating the buffer prevents
SELinux warnings.

-----

Add requestor name to GraphicBuffer alloc metadata

Adds a requestor name (usually the BufferQueue consumer's name) to the
metadata that GraphicBufferAllocator stores on allocation so that
`dumpsys SurfaceFlinger` can attempt to attribute buffer usage to the
correct client.

Bug: 28401203
Bug: 29402015
Bug: 30776557

libui: Add Gralloc1On0Adapter and C++ shim

Adds an adapter which provides the gralloc1 interface on top of a
gralloc 0.x device as well as a C++ shim which wraps a gralloc1 device
and provides a somewhat nicer interface to the rest of the system.

This has also been squashed with a later commit that added mutexes to
protect both the buffer list and the descriptor list from concurrent
access.

Bug: 28401203
Bug: 29420918
Change-Id: I0eeafc998b56e2e2fc39de6fad41e3ed2e19658a

Merge "Fix google-explicit-constructor warnings in surfaceflinger."
am: 5e703a6ad0

Change-Id: I758fe34b9d89c5d2c343830d6521b7b2217cdbde