Merge "Fix race condition clearing VSYNC enable on VrFlinger startup." into oc-dr1-dev
am: 8f09b6abec

Change-Id: I671c34252d7e0cfedeb2b22d77e6e635aa3d907b

Merge "Fix race condition clearing VSYNC enable on VrFlinger startup." into oc-dr1-dev

Merge "Mark libbinder and libdumpstateutil as VNDK in Android.bp" into oc-mr1-dev

Fix race condition clearing VSYNC enable on VrFlinger startup.

Fix VrFlinger to avoid touching the VSYNC enable setting in HWC
during startup by avoiding calling the cleanup code when entering
idle state for the first time.

Bug: 63626797
Test: Manual testing.
Change-Id: I5320bd5c1febb51367e759a8619d35f155789628

Merge "Use cutils/android_filesystem_config.h" into oc-mr1-dev

Merge "Add metadata in direct display surface to support ANativeWindow use cases." into oc-mr1-dev

Merge "Fix missing check on buffer import." into oc-dr1-dev
am: e018119bb3

Change-Id: I6916696a90822809768cdaf8795a85203abc4315

Merge "Fix missing check on buffer import." into oc-dr1-dev

Add metadata in direct display surface to support ANativeWindow use cases.

When direct display surface is used with metadata, ComsumerQueue dequeue
reports error because metadata size is hardcoded as 0. It breaks
ANativeWindow case because it has a metadata on the fly.

Create a metadata structure, which could be read and used in the future.

Bug: 64155181
Test: Manual
Change-Id: Ieb35a69d26213769497c6afc7151dd135150c795

Merge "libsurfaceflinger: handle WINDOW_TYPE_DONT_SCREENSHOT" into oc-dr1-dev
am: 1e9bf7c91b

Change-Id: Ie0d80934d0bf1d92ecfb2ab491cdf2b374132c17

Merge "libsurfaceflinger: handle WINDOW_TYPE_DONT_SCREENSHOT" into oc-dr1-dev

Merge "Remove constraint on touch location." into oc-dr1-dev
am: d8fb01fe77

Change-Id: Iebda7471bcff007c88f2cb67472c914988337e33

Merge changes from topic '62033391' into oc-dr1-dev
am: 97d1c4f234

Change-Id: I33111bae2e009087d979b5f83b0d76a6ba9328c1

Enable InputMonitors for non-default display.
am: 48aeb512e9

Change-Id: I8d7faebe9c57bc17ddeeb7d4df3a69afa908171c

Merge "Remove constraint on touch location." into oc-dr1-dev

libsurfaceflinger: handle WINDOW_TYPE_DONT_SCREENSHOT

When a layer has type WINDOW_TYPE_DONT_SCREENSHOT, hide it from
everywhere but the primary display.  This should be reverted when we
switch to use layer hierarchy properly.

Bug: 63311708
Test: screencap, screenrecord, android.view.cts.SurfaceViewSyncTest
Change-Id: I6a8d6b93399b0dc42832588f9a6c5e8879a8b754

Merge changes from topic '62033391' into oc-dr1-dev

* changes:
  Add displayId in InputDispatcher, InputTransport
  Enable InputMonitors for non-default display.

Merge changes If4271719,I9c9b5de9,I46a26a67 into oc-dr1-dev
am: ddd9c71272

Change-Id: Ie3868b140dfc675f450ac5f436f3533c1d2bdc93

Merge "EGL: Add EGL_GL_scrgb extension" into oc-dr1-dev
am: d5e8f47eff  -s ours

Change-Id: Ifa3a705354ca92a5cc091d872bd5fff5b2512e7c

Merge changes If4271719,I9c9b5de9,I46a26a67 into oc-dr1-dev

* changes:
  Communicate composition buffer dataspace to HWC
  Add VK_COLOR_SPACE_EXTENDED_SRGB_NONLINEAR_EXT.
  Check wide-color support before adding extensions

Merge "EGL: Add EGL_GL_scrgb extension" into oc-dr1-dev

Merge "Installd: Fix math overflow on quota calculation" into oc-mr1-dev

Mark libbinder and libdumpstateutil as VNDK in Android.bp

As a VNDK module, Android.bp must have 'vndk' tag as well as
'vendor_available: true'.

The 'vndk' tag for VNDK module is formated as below:
vndk: {
    enabled: true,
},

VNDK modules will be installed both in system/lib(64) as normal and
in system/lib(64)/vndk as a vendor variant.

Bug: 63866913
Test: build and boot with BOARD_VNDK_VERSION=current
Change-Id: Ib1d9469ad0aead4cbec7c2548438d44608f36819

Use cutils/android_filesystem_config.h

private/android_filesystem_config.h is not available for
vndservicemanager.

Test: m -j
Test: BOARD_VNDK_VERSION m -j

Bug: 63135587
Change-Id: Iaf9911a519ebbb798184e93b2911d0e813e6026c

Renamed 2nd device file back to dumpstate_board.bin.

Fixes: 62872793
Test: manual verification
Test: mmm -j32 frameworks/native/cmds/dumpstate/ && adb push $OUT/system/lib/libdumpstateutil.so /system/lib/ && adb push ${ANDROID_PRODUCT_OUT}/data/nativetest/dumpstate_test* /data/nativetest && adb shell /data/nativetest/dumpstate_test/dumpstate_test

Change-Id: I3574124be9d430b92a08531cff412e88f4469fa6

Add displayId in InputDispatcher, InputTransport

Bug: 62033391
Test: make StructLayout_test
make libinput_tests_InputEvent_test
make libinput_tests_InputPublisherAndConsumer_test
Run tests under /data/nativetest64/libinput_tests/

Change-Id: Id02cab83924d399eae0c1c233dc545fbb64945c2

Merge "Add new Dolby Vision Profile types" into oc-mr1-dev

Merge "Have the Surface class track the buffer age." into oc-mr1-dev

Communicate composition buffer dataspace to HWC

Test: adb shell dumpsys SurfaceFlinger
      look for dataspace info in DisplayDevice section
Bug: 63146977
Change-Id: If427171994fbc91faacf5bad9cc736ddfbd35ec3
(cherry picked from commit 79d272442ce13418c3ea81c95d7fea0159b4b481)

Add VK_COLOR_SPACE_EXTENDED_SRGB_NONLINEAR_EXT.

Bug: 63077212
Test: CTS basicExtensionTest

Change-Id: I9c9b5de99249a6fdd49df1995971c14915f6a7c8
(cherry picked from commit edc88e78a30ca412a0a085a21f1fc1be7be5081f)

Check wide-color support before adding extensions

Don't want applications seeing the wide-color EGL extensions
if the device or display cannot support wide-color.
Bug: 63170158
Test: adb shell /data/nativetest/test-opengl-gl2_basic/test-opengl-gl2_basic
Verify that EGL_EXT_gl_colorspace_scrgb, EGL_EXT_gl_colorspace_scrgb_linear,
EGL_EXT_gl_colorspace_display_p3_linear and EGL_EXT_gl_colorspace_display_p3
are not present on devices that do not support wide-color, e.g. Nexus 6P

Change-Id: I46a26a67f2d6da9c5aad50d884ef02a62ccb6945
(cherry picked from commit e5d6f994158d554c692afae0f547d89c75abde71)

EGL: Add EGL_GL_scrgb extension

Test: Android CTS
    adb -d shell am start \
    -n com.drawelements.deqp/android.app.NativeActivity \
    -e cmdLine '"deqp --deqp-case=dEQP-EGL.functional.wide_color.* \
    --deqp-log-filename=/sdcard/dEQP-Log.qpa"'
Bug: 62424735

Change-Id: I12a0bd64c01bcf314b4a73d8eb3151b09bf9f8ae
(cherry picked from commit 33e2b781a05f1caf274e95c143005a862223e9bf)

Merge "sf: Defer DispSync initialization" into oc-dr1-dev
am: 37d8c19a19

Change-Id: If075910e28f94462c1b24b650afa1b0fcad8d144

Merge "sf: Defer DispSync initialization" into oc-dr1-dev

Merge "Add TestStableBufferIdAndHardwareBuffer" into oc-mr1-dev

Enable InputMonitors for non-default display.

InputFlinger today doesn't deliver InputMonitors for non-deafult
display. In order to support focussing windows on virtual-displays
(like the Vr2dDisplay), we need to be able to send motion events
to them for keyboard to gain focus.

Bug: 62033391
Test: Manual by launching
com.google.vr.vrcore/.daydream.MetaworldActivity & then
com.google.android.apps.nexuslauncher/.NexusLauncherActivity.
Exact steps are mentioned in the bug.
Change-Id: I5954e54cef8c0a29f05bc964debc95ea81f16758

Installd: Fix math overflow on quota calculation

On 32-bit devices, the hard quota size calculation could overflow that
would cause the hard quota limit size to be much lower than intended.

b/64160395

Change-Id: If7b0f2a40f77bb5e5957c663999544f4ab2e69be

Add TestStableBufferIdAndHardwareBuffer

Verifies a Dvr{Read,Write}BufferQueue contains the same set of
Dvr{Read,Write}Buffer(s) during their lifecycles. And for the same
buffer_id, the corresponding AHardwareBuffer handle stays the
same. User of DVR API need to make sure buffers from a queue can be
cached by buffer_id.

Bug: 64208560
Test: DvrApi-test
Change-Id: I1f89c49061b8727bb6a321201b1556b5ceffc9a9

vulkan: Translate usage flags before passing to driver
am: 8e0c3f5f1c

Change-Id: Ia6f0c0a8c43b1e9cbe94f05e342f2c0cd92cdad1

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

(Cherry-picked from 4744291270cc6 in oc-dev)

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

Have the Surface class track the buffer age.

Have the Surface class track the buffer age, so that Surface::query()
can return the buffer age without having to use a binder call to
BufferQueueProducer::query().  The idea is for
BufferQueueProducer::dequeueBuffer() to return the value, which the
Surface class will cache for later use by Surface::query().

Bug: b/27903668
Test: Use systrace to no ensure query binder call after dequeueBuffer.
Change-Id: I106a7bd27461d381f0bd84df70d804de56a128ab
(cherry picked from commit d11b044864be525a1646f93106ab496195bb8239)

Fix missing check on buffer import.

Fix a missing check on the success of importing buffers into the
ConsumerQueue. There is a race condition where the producer can
invalidate its buffers, for example by resizing, before the consumer
has a chance to import the previous buffers. The missing check
causes a crash in SurfaceFlinger and VrCore, which are both
consumers of application VR surface buffers.

Also fix a missing lock around the consumer queues in VR surfaces
found during the analysis of this bug.

Bug: 64042620
Test: Ran test.apk before and after the fix. Observe stable operation
      after applying the fix.

Change-Id: I416df3ca47978404dcdb53599ddeec9b4bd6fb1a

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
(cherry picked from commit f41745301d5ecfa680dcef3a1948a8a321f80509)

dumpstate: Add historical ANRs as separate entries in the zip file.

ANRs now show up as separate entries in the bugreport zip file, like so:

-rw----     0.0 fat   516946 bl    48779 defN 17-Jul-27 10:47 FS/data/anr/anr_2017-07-27-10-47-45-798
-rw----     0.0 fat   520627 bl    48995 defN 17-Jul-27 10:47 FS/data/anr/anr_2017-07-27-10-47-30-328

Bug: 32470528
Test: adb bugreport ./bugreport.zip; zipinfo -l ./bugreport.zip | grep anr

Change-Id: Ibcde7bd35cab1d81e26c72e9247d63cf7bce1504

Merge "Add media/openmax as an include directory" into oc-mr1-dev

Prevent media keys from waking the device, even if they're external.
am: 58ba988476

Change-Id: Ieef908ac1b0a2cfbcad57d17356dddd1d787f09c

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

Prevent media keys from waking the device, even if they're external.

Bug: 38258736
Test: manual
Change-Id: I56deb9d99df17172ea063544334c1814a6f31509

Remove constraint on touch location.

InputFlinger does accept and handle 'off-screen' touch input,
and it is sometimes useful, for instance for drag scrolling.

Bug: 64063111
Test: manual on device
Change-Id: I62baf4e01c583c10853518de4b87769aad29dab7

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

Add new Dolby Vision Profile types

Test: build

Change-Id: Ie18b3ea536fc0b95ddb3c6c89b4f2643c94979fa
Signed-off-by: Previr Rangroo <prang@dolby.com>

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

Add media/openmax as an include directory

This will obviate changing Khronos's header files.

Note that all header files in media/openmax have OMX_ prefix, so the
chance of collision is already low.

Test: Compiles
Bug: 63821829
Change-Id: I40cc92409de5091a1e2ed7c89c0012000b20f68f

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