Merge changes from topic 'libcutils_private_headers' into oc-mr1-dev

* changes:
  Use getpwnam()/getgrnam() instead of AID_* macros (dumpstate)
  Use getpwnam()/getgrnam() instead of AID_* macros (libgui)

Use getpwnam()/getgrnam() instead of AID_* macros (dumpstate)

AID_* values are defined in libcutils private headers and hence
is not exported. Use getpwnam() instead to get these values.

Test: device boots
Test: bugreport
Test: dumpstate_test
Bug: 63135587
Change-Id: Ib35fe4fc4524eb1885616a9a1403ceb09f713777

Use getpwnam()/getgrnam() instead of AID_* macros (libgui)

AID_* values are defined in libcutils private headers and hence
is not exported. Use getpwnam() instead to get these values.

Test: device boots
Bug: 63135587
Change-Id: I417d0472df2170ab190c1c1622117df58292125c

Initialize native Sensor object correctly
am: 27cc5f9634

Change-Id: I744d375ea103c77ae4bf9d65c368d157cb6aca5b

Initialize native Sensor object correctly

This CL gives default values for all fields in Sensor native
object.

Bug: 37640915
Test: added local logging to verify all native sensor objects are
      initialized correctly.

Change-Id: I4076756249788f56960c74629cb800f1b2bae79f

Merge "Add dvrReadBufferQueueGetEventFd" into oc-dr1-dev
am: 42d6d5c805

Change-Id: I608dac89a7a3076a9df7201e2defd948e700a3b9

Merge "Add dvrReadBufferQueueGetEventFd" into oc-dr1-dev

Postpone color mode change until after boot animation
am: c53d3558d5

Change-Id: I96c0f0fb297be1ff9c1b5df2b33ffeacc39888b9

Add dvrReadBufferQueueGetEventFd

We need to expose the read queue's event fd, so that a caller can epoll
on a set of queues' fd for efficient buffer acquisition.

Bug: 62359843
Test: dvrapi-test

Change-Id: I1b9271afed9ee6c6675551402cd3f5a66e98ebe2

Postpone color mode change until after boot animation

sRGB is only set by SurfaceFlinger when the default display is
wide color capable. Since we compute the best color mode on
every frame anyway for wide color capable displays, we can
simply move the sRGB set to the end of the boot animation.

Bug: 63823274
Test: Manual
Change-Id: I11bb6095acb63e66bcf7dff9e3f7b36588e371c1

Merge "surfaceflinger: fix initial crop computation" into oc-dr1-dev
am: 349abaf55f

Change-Id: Ied5b49f6089e2cd78ef53c9918d4e57c5231d504

Merge "surfaceflinger: fix initial crop computation" into oc-dr1-dev

Merge "Properly applies the selected saturation boost (vivid mode)" into oc-dr1-dev
am: 3eba4064ef

Change-Id: I9ee74bf11f728524501b7003dbcd5fcf6707be2d

Merge "Properly applies the selected saturation boost (vivid mode)" into oc-dr1-dev

SurfaceFlinger: Set property in StartPropertySetThread during init
am: f9b05eeb5f

Change-Id: I3433d57d6e929f57202c4d9a68a6a09845474166

Properly applies the selected saturation boost (vivid mode)

The saturation boost setting is read by SurfaceFlinger as a
persistent system property. Unfortunately, persistent props
are only available after Vold is up and /data is decrypted,
which may happen before or after SF attempts to read the
property.

This CL moves the propery lookup to the end of the boot
animation. This solves two issues:
- The saturation boost will not be applied to the boot animation
- The vivid colors user setting is now reliably applied

Bug: 63823274
Test: Manual

Change-Id: Icb8e30c799c30cf674f0fc0bab0369f4c99367ed

SurfaceFlinger: Set property in StartPropertySetThread during init

This is similar to ag/1849505/ (see b/34499826), which by setting
property in a separate thread, that CL aims to avoid slow initialization
in SurfaceFlinger::init where SurfaceFlinger is waiting on
property_service.

There is new property_set() call added, and this CL is to move it to the
StartPropertySetThread.

Bug: 63844978
Test: on taimen with simulated delay ag/2562492/
Change-Id: I31547cb5e75f44eac635386b3cf345a44931c78f

surfaceflinger: fix initial crop computation

This function computes the crop of the layer, that is, the region of
the layer to be drawn.  We must consider the size of the layer when
applying s.active.crop.

Bug: 63786936
Test: manual
Change-Id: I8000413beba54a054db8d7ad28e868d23b0167ab

Use appendFormat instead of append when passing arguments am: 53e5aa93fa
am: fab65a0d34

Change-Id: Ic78a7fff83919089330cbdebeffd5a9aebb0243b

Use appendFormat instead of append when passing arguments
am: 53e5aa93fa

Change-Id: Ia3ad9c8ddbef9937542feca396d05f8dca136125

Replace dexopt shared libraries with class loader context

Pass the class loader context to dex2oat with --class-loader-context
instead of using the runtime -cp argument.

Test: adb shell cmd package compile a_split_test_apk
Bug: 38138251
Change-Id: I2e483969af8adf97291aa94f42e6da9ee5f1716b

Merge "sf: Defer DispSync initialization"

sf: Defer DispSync initialization

Some DispSync members are initialized based on uninitialized static
members of sf, that are in turn initialized in sf constructor. Fix
the sequence by deferring DispSync initialization.

Current sequence:
sf constructor|-> DispSync constructor -> Access static sf members
              |-> Initialize sf static members

New sequence:
sf constructor|-> DispSync constructor
              |-> Initialize sf static members
              |-> DispSync init -> Access static sf members

Bug: 63671437
Test: "present fences are ignored" not present in SF dumpsys
Change-Id: I618d2bbbbd4e39fc382e67f85dd8d637dd82cf38

Use appendFormat instead of append when passing arguments

Using append crashes the system.

Bug: 63804323
Test: On a contrived build with no sensors, dumpsys does not crash the
      sensorservice.

Change-Id: I4ee942d58af8347388f4f5f71c48da339c1cf7e7

Merge "_FILE_OFFSET_BITS=64 support for asset_manager.h." am: 47c479d9b9 am: 0f169dac29 am: 1cb4f68592
am: 6468e8c874

Change-Id: I297ddaf5f21a84c00104c16b70e727b3f4b1f9fa

Merge "_FILE_OFFSET_BITS=64 support for asset_manager.h." am: 47c479d9b9 am: 0f169dac29
am: 1cb4f68592

Change-Id: I70e186b1f6e3b4dee6901435587c991b0307a48c

Merge "_FILE_OFFSET_BITS=64 support for asset_manager.h." am: 47c479d9b9
am: 0f169dac29

Change-Id: I52a2b05119df7ebda3084074aa4f6584fe53e9d1

Merge "_FILE_OFFSET_BITS=64 support for asset_manager.h."
am: 47c479d9b9

Change-Id: I8fcdb3ff9680b1fb7cc41a7bb5e91b71fd16abc5

Merge "_FILE_OFFSET_BITS=64 support for asset_manager.h."

Merge "Merge "Add target binderDriverInterfaceTest_IPC_32" into oc-dev am: 72cce2f70a am: de1ad9de0c" into oc-dr1-dev-plus-aosp
am: 67b2424559

Change-Id: Icab64a7718c9f6d6b57a422a26736bab7a7dd3c6

Merge "Add binderLibTest_IPC_32 am: 75be4a1000 am: 973ff48d4e" into oc-dr1-dev-plus-aosp
am: fcff4bb16d

Change-Id: I9c531beeca730cd5ce1219f1aea42cd0f1344059

Merge "Add target binderDriverInterfaceTest_IPC_32" into oc-dev am: 72cce2f70a am: e97e5d9556
am: fb8eff13e9

Change-Id: Ic189e0685e95fa8658b913ea47ca8944360cc96a

Add binderLibTest_IPC_32 am: 75be4a1000 am: 3274de62d4
am: 3ba7fd6ea8

Change-Id: Iac14e1a40a7d9b244cd9d8160a26ac5cbeaa9893

Merge "Merge "Add target binderDriverInterfaceTest_IPC_32" into oc-dev am: 72cce2f70a am: de1ad9de0c" into oc-dr1-dev-plus-aosp

Merge "Add target binderDriverInterfaceTest_IPC_32" into oc-dev am: 72cce2f70a
am: de1ad9de0c

Change-Id: Ib974b6a33a5d3c3fb2bafdd5e718a0911ddaed1a

Merge "Add binderLibTest_IPC_32 am: 75be4a1000 am: 973ff48d4e" into oc-dr1-dev-plus-aosp

Add binderLibTest_IPC_32 am: 75be4a1000
am: 973ff48d4e

Change-Id: I0c59ad1f2fdab199ed541dd58be1822c9dd2f858

Merge "Add target binderDriverInterfaceTest_IPC_32" into oc-dev am: 72cce2f70a
am: e97e5d9556

Change-Id: Ia2573b6c9b78413fad4be1b7ddb3ea3841b9a426

Add binderLibTest_IPC_32 am: 75be4a1000
am: 3274de62d4

Change-Id: Iaf348e731bc397460a2e8d8354b62b327d70c722

Merge "Add target binderDriverInterfaceTest_IPC_32" into oc-dev
am: 72cce2f70a

Change-Id: I32f4bf8987679def643f2a4d848878f545dbf130

Merge "Add target binderDriverInterfaceTest_IPC_32" into oc-dev
am: 72cce2f70a

Change-Id: I7c81227c25df80581ef40b40da84915abcb3b7e9

Add binderLibTest_IPC_32
am: 75be4a1000

Change-Id: I1ad068367e6e3c0ddbf44825bbe4e1aadee1948a

Add binderLibTest_IPC_32
am: 75be4a1000

Change-Id: I47b598881baf7cb4520b0fad05a68ec741201a27

Merge "Add target binderDriverInterfaceTest_IPC_32" into oc-dev

Merge "Fix clang static analyzer warnings." am: 411414c916 am: c95b45c72f am: 37146b7b02
am: ef5e1bb46e

Change-Id: Ia9ca864825904017a05c502a15ce970802ea4dc3

Merge "Fix clang static analyzer warnings." am: 411414c916 am: c95b45c72f
am: 37146b7b02

Change-Id: I80d0a661cfa8da9d19609dffe7beb9688d82535a

Merge "Fix clang static analyzer warnings." am: 411414c916
am: c95b45c72f

Change-Id: Iaf350d2172258a8c0425cb8970b197a1eacb86c4

Merge "Fix clang static analyzer warnings."
am: 411414c916

Change-Id: Icb0a337935cdcdccd5d66702188e7e53cf319c35

Merge "Fix clang static analyzer warnings."

Fix clang static analyzer warnings.

Fix the following warning:
Potential leak of memory pointed to by 'b'
[clang-analyzer-cplusplus.NewDeleteLeaks]

Bug: b/27101951
Test:Warning is gone.
Change-Id: If898bae71771f325c97777a19af0c0cf2e8a0400

Merge "Fix clang static anaylzer warnings." am: a97d58a21b am: 747d847ff1 am: 802529f46d
am: e7c634a4e5

Change-Id: I17ac8c3b889c78234315b176230a66d650bad733

Merge "Fix clang static anaylzer warnings." am: a97d58a21b am: 747d847ff1
am: 802529f46d

Change-Id: I497397026dafe63a912927cbd1d8a0ca327399f4

Merge "Reference nativehelper/jni.h as jni.h" am: 28c9177f66 am: 6ac4d2a42b am: 9cd659c8dc
am: 7ec584b142

Change-Id: Id80fe1c03e327153a8ecb73cb9953f43b3ed8cfb

Merge "Reference nativehelper/jni.h as jni.h" am: 28c9177f66 am: 6ac4d2a42b
am: 9cd659c8dc

Change-Id: Iaf417e768c22335df1720b13e6583006a9a6c6c4

Merge "resolve merge conflicts of f8187f4e1 to oc-dr1-dev-plus-aosp" into oc-dr1-dev-plus-aosp
am: 72f1824c46  -s ours

Change-Id: Id0de85cb32bfc7d90cf048639c4d3a6a21d63a54

surfaceflinger: fix a nullptr dereference am: 0a68b461d3
am: b111e153d9

Change-Id: I31ce55db7b94082405d54b730f98c7875c1c85e5

Merge "resolve merge conflicts of f8187f4e1 to oc-dr1-dev-plus-aosp" into oc-dr1-dev-plus-aosp

surfaceflinger: fix a nullptr dereference
am: 0a68b461d3

Change-Id: I4f449e515a8f18335d711b5e145537435dabe27f

resolve merge conflicts of f8187f4e1 to oc-dr1-dev-plus-aosp

Test: I solemnly swear I tested this conflict resolution.
Merged-In: I5f908f88923378e4d121dfd5fbc11e4f210cbb2a
Change-Id: I29912e8c2d61f7711b201deaf0e35cebb0bbf28a

Merge "Fix clang static anaylzer warnings." am: a97d58a21b
am: 747d847ff1

Change-Id: Ic74cc49d92fc6a661be461bca19b77b45e9a3a4e

Merge "Fix clang static anaylzer warnings."
am: a97d58a21b

Change-Id: Id66bd914d41190df5c3dc29968fab5b7f942851d

Merge "Reference nativehelper/jni.h as jni.h" am: 28c9177f66
am: 6ac4d2a42b

Change-Id: Id1fd8abd06cfb80ccae1304d2a3005b27ba95008

Merge "Reference nativehelper/jni.h as jni.h"
am: 28c9177f66

Change-Id: If460edb57fbd4678cfe932eb76b0931024f17f80

resolve merge conflicts of 8a094116b to stage-aosp-master
am: bb7e1b68e6

Change-Id: I453a7aaf28c20634337ac70ea6586d5a10d4736c

surfaceflinger: fix a nullptr dereference

When the child layer latched a buffer but the fixed-size parent
layer never did (for reasons such as no buffer queued or buffer got
rejected), we could end up with p->mActiveBuffer being nullptr.

Bug: 62996512
Test: manual (I could never repro anyway)
Change-Id: Id7e4c7037633b8a37039baa6e8a306e55170b894

_FILE_OFFSET_BITS=64 support for asset_manager.h.

Rename off_t based functions to their 64-bit equivalents if we're in
LP32 with a 64-bit off_t.

We can safely do this for all supported API levels now since API 13 is
no longer supported. Clean up all the guards for that API level while
I'm here.

Test: make checkbuild
Bug: https://github.com/android-ndk/ndk/issues/459
Change-Id: I7032e94c7288f87b4895411079216f0ac2cab6bc

Merge "Check dexopt needed for secondary dex while downgrading"

Merge "Fix clang static anaylzer warnings."

Merge "Add android version guard to ASensorManager_getInstanceForPackage"

Merge "Make vulkan build with the VNDK."

Add android version guard to ASensorManager_getInstanceForPackage

This function is added to header file since 2015 but the corresponding
function is not exported through libandroid.map file until O. Thus,
it make sense to gate the usage to ANDROID_O and forward.

ASensorManager_getInstance is deprecated accordingly.

Bug: 63639195
Test: compiles
Change-Id: Ib2fdb635ffca9ec629dd487bd377aaa98afa4359

Merge "Reference nativehelper/jni.h as jni.h"

Merge "Fix concurrent writes to the hardware composer" into oc-dr1-dev am: 87cd11513f
am: ad59d3ad0b

Change-Id: Ib0e1c3a2804d2b77c7f41adcca65b4de926ae24b

Merge "Fix concurrent writes to the hardware composer" into oc-dr1-dev
am: 87cd11513f

Change-Id: I00d0a5267913c284eddd97bfe7eefb15b9ee403e

Merge "Fix concurrent writes to the hardware composer" into oc-dr1-dev

Fix concurrent writes to the hardware composer

Stale hardware composer layers were causing concurrent writes to the
Composer object from the surface flinger and vr flinger threads, a big
no-no. The concurrent writes would sometimes stomp on each other,
causing the hardware composer service to fail to read the command buffer
containing surface flinger's composer commands, leading to all sorts of
issues.

Bug: 62925812

Test: Locally added logs to surface flinger to catch the concurrent
writes, and confirmed the logs are no longer present with this patch
applied. Went through a bunch of sleep/wake cycles and confirmed the
device continues to function normally.

Change-Id: I70929c4a3c71142f5e9083cac294c122d127aa27

Merge "Fix memory leak." into oc-dr1-dev am: 02b37f7594
am: 5b7045ee2d

Change-Id: I1960f76a9d237bed8b0cb9d3be109aedac7b697c

Merge "Fix memory leak." into oc-dr1-dev
am: 02b37f7594

Change-Id: If382353979a9a31c2d46fd7398ec59b086d97ac6

Merge "Fix memory leak." into oc-dr1-dev

resolve merge conflicts of 8a094116b to stage-aosp-master

Test: I solemnly swear I tested this conflict resolution.
Change-Id: I87e2955a7e44aa28eafacd44d404a23a1ae2e4a0

Merge "Update the Khronos EGL and GLES* header files."

Merge "Update the Khronos EGL and GLES* header files."

Reference nativehelper/jni.h as jni.h

nativehelper will no longer export nativehelper/jni.h so that everywhere
can reference this file with the same name.

Bug: 63762847
Change-Id: Ifc3a3feb7578fd1352c25a3f55884ab75f337039

Make vulkan build with the VNDK.

When the vndk is enabled (BOARD_VNDK_VERSION=current),
vendor libraries must only link against vendor variants
in the build system. This also means that they receive
restricted sets of headers and that automatic global
headers are restricted.

Test: build vulkan with BOARD_VNDK_VERSION=current
Bug: 37342627
Change-Id: I9690635e4654e053cf35a3941aff14f4f7b46491

Update the Khronos EGL and GLES* header files.

The latest Khronos header file contents are used.  Some
not-yet-registered Android extensions are added in, along with a few
data types that were accidentally deleted from "GLES/gl.h".  Note that
Khronos rearranged some of the content.

In addition, a repeat of a 2012 fix was required to avoid ANativeWindow
being declared twice during the compilation of source files which also
include "window.h".  Both "window.h" and "native_window.h" contain the
following line:  typedef struct ANativeWindow ANativeWindow;

I tested this by building Android as well as other source that includes
"window.h", and carefully compared diff's of the old and new content.

Bug: None
Test: Built Android

Merged-In: I5f908f88923378e4d121dfd5fbc11e4f210cbb2a
Change-Id: Ic40276fc8bc0b7120a11db34c3e59ebeec9c9f97

Update the Khronos EGL and GLES* header files.

The latest Khronos header file contents are used.  Some
not-yet-registered Android extensions are added in, along with a few
data types that were accidentally deleted from "GLES/gl.h".  Note that
Khronos rearranged some of the content.

In addition, a repeat of a 2012 fix was required to avoid ANativeWindow
being declared twice during the compilation of source files which also
include "window.h".  Both "window.h" and "native_window.h" contain the
following line:  typedef struct ANativeWindow ANativeWindow;

I tested this by building Android as well as other source that includes
"window.h", and carefully compared diff's of the old and new content.

Bug: None
Test: Built Android
Change-Id: I5f908f88923378e4d121dfd5fbc11e4f210cbb2a

Add target binderDriverInterfaceTest_IPC_32

Add the target binderDriverInterfaceTest_IPC_32 which tests 32-bit
kernel and is independent from product_variables.

Bug: 62124459
Test: vts-tradefed run commandAndExit vts -m VtsKernelBinderTest
Change-Id: Ifad9ef24c35060179650955e24d1f046af7b7385
(cherry picked from commit 6929575affc7d0037c79a45b2cb0ef7c6020581a)

Add binderLibTest_IPC_32

1) Add product_variables to binderLibTest because the test case
   CheckHandleZeroBinderHighBitsZeroCookie uses binder data structure.
2) Add binderLibTest_IPC_32 which always compiles the test program for
   32-bit binder.

Bug: 63307240
Bug: 62366937
Test: vts-tradefed run commandAndExit vts -m VtsKernelBinderTest
Change-Id: I40c738a039459fbfece9689012249ae7ea93df9d
(cherry picked from commit f6edfc1b096cba544227d85b0ea3f38a4dd11af5)

Check dexopt needed for secondary dex while downgrading

Test: make otapreopt
Test: manual
* Remove the check in the code that allows downgrade only when
  the space is low on the device.
* adb root
* Set pm.dexopt_unopt_after_inactive_days to 600
* Make sure the current time of the device is correctly set
* Install 2 non system apps - B, C
* Downgrade B to extract
* Upgrade a system apps to speed-profile - E
* Downgrade a system app to quicken - G
* adb shell cmd package bg-dexopt-job

Expected Results:
* Extract - B
* Verify - C
* There should not be any entries for apps E an G
  in dalvik_cache

For secondary dex:
* compile googlequicksearchbox to everything.
* run background dexopt service for secondary dex.
* verify the compiler filter for the corresponding odex files.

Bug: 36598475
Change-Id: I6ad40e2c53433326f83c2589646d94ba555400b2

Merge changes I23e6f088,I800208e8,I55123a7a into oc-dr1-dev am: 607b5d1b40
am: 1042a22877

Change-Id: I4c5e092f7119fb9a90e791dd31fbdb4f56b811cd

Merge changes I23e6f088,I800208e8,I55123a7a into oc-dr1-dev
am: 607b5d1b40

Change-Id: I51b25a25bae1517f64edf9ce072244addd1b3544

Merge changes I23e6f088,I800208e8,I55123a7a into oc-dr1-dev

* changes:
  SF: Move screenshot Surface ops off main thread
  SF: Add ImageHolder for screenshot code
  SF: Add WindowDisconnector for screenshot code

Merge "Add GraphicBufferMapper::preloadHal" into oc-dr1-dev am: 2f83627449
am: 6645d88787

Change-Id: If7a5617abdf33ad055dec1a46586e50813a8c151

Merge "Add GraphicBufferMapper::preloadHal" into oc-dr1-dev
am: 2f83627449

Change-Id: I0462d321276c9315fba82d8cc97b9889c0e5a88c

Merge "Add GraphicBufferMapper::preloadHal" into oc-dr1-dev

Fix memory leak.

A missing destructor caused a memory leak in the UDS backend for PDX.
Add the destructor and use std namespace for realloc/malloc/free.

Bug: 63026993
Test: Ran pdx_benchmarks for 400M iterations before and after fix.
      Confirmed that heap grows before fix and is stable after fix.
      Confirmed that VR service reported in the bug now have stable
      memory use.

Change-Id: If870a5db50dfaf5f212285824a1803f34acdabc7

SF: Move screenshot Surface ops off main thread

This change rearranges the various operations that correspond to
capturing a screenshot such that all of the Surface-related ones
(connect, dequeue, queue, disconnect) are performed on the incoming
Binder thread rather than on SurfaceFlinger's main thread. This has two
major benefits:

  1) It reduces the amount of time that the SurfaceFlinger main thread
     is blocked while performing a screenshot, often by a considerable
     amount. This should reduce the risk of jank when screenshots are
     taken, such as for task snapshots during window transitions.
  2) It means that the SurfaceFlinger main thread is not susceptible to
     being blocked by a badly-performing BufferQueue consumer. This
     also enables us to remove the GraphicProducerWrapper class, which
     was previously performing a similar role.

Finally, this change also adds a mechanism that detects if the
screenshot would have been performed between the two phases of normal
SurfaceFlinger operation (invalidate and refresh), and defers it if
this condition is detected. This should further reduce the risk of jank
as a screenshot will only occur between frames rather than in the
middle of a frame.

Bug: 62257775
Test: SurfaceFlinger_test and manual verification that screenshots
      still work
Change-Id: I23e6f088b4d6e477472dfc2a6c36ef3dd930c047

Merge "Fix ALOGE_IF log in bufferhubqueue" into oc-dr1-dev am: c3e0cdb28a
am: 05a8133967

Change-Id: I8fec5db7cc3248c8142f08f9fd896d6bd16dd5b2

Merge "Fix ALOGE_IF log in bufferhubqueue" into oc-dr1-dev
am: c3e0cdb28a

Change-Id: I5ed228cd0f98a6324b799b506377df750238e549

Merge "Fix ALOGE_IF log in bufferhubqueue" into oc-dr1-dev