Merge "Move sensord" into oc-dev
am: 903726f55b

Change-Id: I6c55dc0e0e24b418cf33ec5e469f4c00411b63ce

Merge "Move sensord" into oc-dev

Merge "bufferqueue: allow null stream in H2BGBP::setSidebandStream" into oc-dev
am: 7fe71f8b0f

Change-Id: I449c8e4100044d3cea0135f2a966c018ed5e03ec

Merge "Split libdvr_headers from libdvr: Step 1" into oc-dev
am: 2eb4922a7b

Change-Id: I84efc214e668bd17bfa7f59797fdaa7344772b61

Merge "bufferqueue: allow null stream in H2BGBP::setSidebandStream" into oc-dev

Merge "Split libdvr_headers from libdvr: Step 1" into oc-dev

docs: Fixed doc comments to enable Doxygen to generate code block correctly. Test: Verified by manually generating docs.
am: 5163357fc0

Change-Id: I209ee283991ca3e341923a52a132fc08a63b5398

Remove vr_wm service (try 2)
am: 007b00e775

Change-Id: Ib5794017f2c0ac8ef48db94d15a597b97b23246b

bufferqueue: allow null stream in H2BGBP::setSidebandStream

Bug: 38172844
Change-Id: I585e1dc01f2e7f955c39046c1592bda7bea45a23

docs: Fixed doc comments to enable Doxygen to generate code block correctly.
Test: Verified by manually generating docs.

Change-Id: Id79a77e71deb26e0dd5d22843c2f09d9000e587d

Split libdvr_headers from libdvr: Step 1

Converts libdvr's Android file into Android.bp as now we have no Java
deps anymore. Then we added header only library libdvr_headers for all
headers.

Next step is to figure out how to split out public headers and make them
easy to be copied into Google3.

Bug: 37578558
Test: dvr_api-test
Change-Id: Idf15cfd5aa61cd7d6f5cc15c4db7cce485b3b293

Remove vr_wm service (try 2)

VR Window Manager has moved to VrCore. Removing deprecated code.

Bug: 36051907
Test: Compiled and verified VR still works.
Change-Id: Iea0c7646df8a068c058099cf19ef7d76bbf89192

Merge "dvrapi: Pass layer_count down" into oc-dev
am: 9a2b294255

Change-Id: Ie420d2d8ef6202a56e156f34c760cafd37b5fd0e

Merge "dvrapi: Pass layer_count down" into oc-dev

Merge "Remove slices from buffers" into oc-dev
am: 5c91f12131

Change-Id: I7538f77b09b4931a5b4409f7317663aba17c40bd

Merge "Remove slices from buffers" into oc-dev

Merge "Remove framebuffer_target" into oc-dev
am: c6c0c9c730

Change-Id: I6d65d8fb4d0586228890327c478bfce57cf3c7f7

Merge "Remove framebuffer_target" into oc-dev

dvrapi: Pass layer_count down

* We were missing layer count, which would prevent multiview
  from working

Bug: 37245304
Test: MultiLayerBufferQueue
Change-Id: I88b41f1aa7665df01e89a7386cbc23b15c9a79b0

Remove slices from buffers

* code no longer required, as it will be handled in vrcore

Bug: 38076195
Test: Manually ran testing
Change-Id: Id28918a3bf73b99cbb57aad4c21251b7a9623074

Merge "remove vr/libeds" into oc-dev
am: db768fd8e5

Change-Id: I46f312ee39adb1239d1c97f676a730dad00bd10b

Merge "remove vr/libeds" into oc-dev

Merge "Fix BufferHubQueueProducer::dequeueBuffer" into oc-dev
am: 1134dd38d8

Change-Id: I73fc24cadc2c647b3c0a507b30b625ffaf8536ca

Merge "Fix BufferHubQueueProducer::dequeueBuffer" into oc-dev

Merge "Prevent buggy display orientation when exiting vr" into oc-dev
am: 95d4e52244

Change-Id: Ie146b08ded70a24bbdc4b681c72724807e26717e

Merge "Prevent buggy display orientation when exiting vr" into oc-dev

Merge "Return an error if apiVersion is 1.1+." into oc-dev
am: 8c109c638a

Change-Id: I1e4e17b02155a4095d28cfff8791f63b7dcd3d08

Merge "Return an error if apiVersion is 1.1+." into oc-dev

Fix BufferHubQueueProducer::dequeueBuffer

Seems this might just be a rebase glitch.

Bug: 38137230
Test: buffer_hub_queue_producer-test
Change-Id: I94ce059475fad4cf31cec52a9661eb6fc9cd2e00

Merge "Do not return an error if the dalvik-cache odex is missing" into oc-dev
am: d6ca10b76d

Change-Id: I0ff52940cbce928720fcdbb08ee0ad2da8a55be0

Merge "Do not return an error if the dalvik-cache odex is missing" into oc-dev

Return an error if apiVersion is 1.1+.

The Vulkan loader is supposed to check the requested apiVersion and return
VK_ERROR_INCOMPATIBLE_DRIVER if the requested version is not supported.  The
loader wasn't performing the check, nor returning the error.  A log message is
also issued, explaining why the application failed to create an instance.

Bug: 38040828
Test: Modify cube to use 1.0 or 1.1 and ensure proper return value.
Change-Id: I8cc792305a346c6e1db40f61d5235c301d8ae848

Remove framebuffer_target

* Remove unused variable

Bug: 38132845
Test: Compiled
Change-Id: I08afba75b72c3d27aca042755bf36584979c7f0c

Do not return an error if the dalvik-cache odex is missing

Test: adb shell /data/nativetest/installd_service_test/installd_service_test
      Tried uninstalling an updated version of prebuilt app.
      Didn't find the error message in the logcat.
Bug: 35804241
Change-Id: Iec045fa9c9ac8f036785fe5cbd0f7e43f2b680c1

Fix enforcement of sensor's slowest rate
am: 663720b29f

Change-Id: I31ff9da171baa83318bc794553c7057b18be2dec

Fix enforcement of sensor's slowest rate

This code calculates a sensor's maximum sample
period in nanoseconds. This is stored as a 64-bit
value, as required for periods greater than ~2.1
seconds.

The calculation was done with 32-bit arithmetic,
sometimes resulting in overflow. This caused the
sensor to run at its maximum rate.

(The requested period is first clipped to the maximum
period. When the maximum period appears negative, it
is always used. The now-negative period is then clipped
to the minimum period, resulting in the sensor's maximum
supported rate.)

Bug: 37465457

Test: Verified correct operation with Goldfish accelerometer,
which has a 60-second maximum period.

Change-Id: Ic75a9dc7c4e7c9ca690eafbfa51ee50540ca5aaf

Merge "libpdx_uds: Always create channel sockets in the server process" into oc-dev
am: 584bc3cebf

Change-Id: I3f96c9639bb192bf651db005d11afa1cdf4ebd43

Merge "libpdx_uds: Always create channel sockets in the server process" into oc-dev

remove vr/libeds

* libeds is no longer needed

Bug: 38075841
Test: compiled
Change-Id: I61b7edb284633ab8e960387c0479b1a924d68222

Merge "Correctly set minimum priority on nodes." into oc-dev
am: fc93a4bd89

Change-Id: If7cb26f43e85a11cb99ce9be511c44d35ddaca02

Merge "Correctly set minimum priority on nodes." into oc-dev

Merge "Clear cached files on external storage." into oc-dev
am: 4b567487b7

Change-Id: Ib979f876c37632b723e6a982dc48ec746aaef290

Merge "Clear cached files on external storage." into oc-dev

Merge "Fix lshal --init-vintf may have empty <instance> tags." am: 4a9b7174ca am: 0a1b8a1128
am: bf578bcc61

Change-Id: Ibf2e5b42970b3d5a55e021225cf47af0c1fd94e8

Merge "Fix lshal --init-vintf may have empty <instance> tags." am: 4a9b7174ca
am: 0a1b8a1128

Change-Id: I007fb20df553c2cec9d9e88a9ae79a4f523acfc6

Merge "Fix lshal --init-vintf may have empty <instance> tags."
am: 4a9b7174ca

Change-Id: Ic6eeff994ab0fb8e2e30dd7c200d9eab512cf057

Merge "Fix lshal --init-vintf may have empty <instance> tags."

Clear cached files on external storage.

When clearing cached files belonging to an app, include any cached
files on external storage.  Since we need to keep sdcardfs in the
loop about any file deletions, we always mutate by going through the
sdcardfs layer instead of behind its back.

Test: cts-tradefed run commandAndExit cts-dev -m CtsAppSecurityHostTestCases -t android.appsecurity.cts.StorageHostTest
Bug: 37486230, 37566983, 37913442, 37914374
Change-Id: If174bf7eaf682da83cf0ab1b4938fe9a5956d464

Merge changes from topic 'o-compositor-unification' into oc-dev
am: 0e04fe4713

Change-Id: Ie819585288871279d6de56290c7939b1c9ab63aa

Merge "Revert "Deprecate producer/consumer usage: Step 3"" into oc-dev
am: cffef95674

Change-Id: I9bc11e913ec84acbbe90a255ed95851dc1d953d0

Merge changes from topic 'o-compositor-unification' into oc-dev

* changes:
  Remove the VR compositor from the framework.
  Remove unnecessary clients and rename files.
  Support multiple consumer queues.
  Return pdx::Status<T> from BufferHubQueue::Dequeue.
  Add support for consumer queue initial import and hangup.

Merge "Revert "Deprecate producer/consumer usage: Step 3"" into oc-dev

Merge "Switch touch input device on hover events." into oc-dev
am: 5a3361d2ed

Change-Id: Ic51df53ed3be731d52e18ee0e95e9f7e87509cf7

Merge "Switch touch input device on hover events." into oc-dev

Correctly set minimum priority on nodes.

It looks like libbinder has always set the minimum priority
for a node to 0x7f (127). This is an invalid value as far
as the kernel is concerned, since the current driver only
accepts positive nice values (0..19). The effect of using
0xf7 is that the nice value was clamped to MAX_NICE (19) -
so effectively the kernel never raised the priority
based on the min_priority setting of a node. Correct this
by just using MAX_NICE directly.

Additionally, a recent change removed the use of the
gDisableBackgroundScheduling flag, which system_server
used to make sure that incoming transactions were not
executed at a lower priority. That behavior is actually
still desired, so this change sets the min_priorirty
value to 0 for processes that have requested background
scheduling to be disabled. The effect of that is that
all transactions into those nodes will be executed with
a priority of at least nice 0 (and potentially higher,
depending on the priority of the caller).

Bug: 37677242
Test: verified /d/binder output
Change-Id: I7cbfd309d04cbd052f868fbfe0930529ff21a48a

Merge "Remove redundant dependencies" into oc-dev-plus-aosp

Revert "Deprecate producer/consumer usage: Step 3"

These changes are effectively handled as part of another, higher priority CL.

This reverts commit 7cab28591d5cdc26913a0882df1b28e3f5393066.

Change-Id: I4cf47eee219870b2758997fe5bd6019d83b82451

Merge "Deprecate producer/consumer usage: Step 3" into oc-dev
am: 944be994f8

Change-Id: I63b35ceb9fe4adaaf1bdbc52eb14d003182346bf

Merge "Deprecate producer/consumer usage: Step 3" into oc-dev

Merge "Revert "Remove vr_wm service"" into oc-dev
am: ef2d2a0d15

Change-Id: Id8a2bd3c93c38b466d87c1c9f09f63bf949efff2

Merge "Revert "Remove vr_wm service"" into oc-dev

Revert "Remove vr_wm service"

This reverts commit efd3e0873e24f25e855bb64fae8485fc04690d56.

Reason for revert: IVrWindowManager.aidl is still used in frameworks/base

Change-Id: Id52fb8cf482ea5ab141ebf6b304c12b18a9d8070

Merge "Remove vr_wm service" into oc-dev
am: 3bfcfeb12e

Change-Id: Ie8937890f4a689073b78ac593c706674dcc366f2

Merge "Remove vr_wm service" into oc-dev

resolve merge conflicts of 7827bcc3e to oc-dev-plus-aosp

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

Merge "Handle safe mode in PackageManager." am: 7a45fd929a
am: de18423ff8

Change-Id: Ia6806efa144248a36114b6e65329d60ab960c698

Merge "Handle safe mode in PackageManager."
am: 7a45fd929a

Change-Id: Icd7bf11cb3bdc3acdf3414a3ba13cd7964e13f3b

Merge "Handle safe mode in PackageManager."

Remove the VR compositor from the framework.

Remove the VR compositor framework and enable out-of-process VR composition
in VrCore.

This CL seems large due to the ripple effect of changing the VrFlinger
API and protocol types. There are three major modules that require
concurrent changes:
  1. Protocol definitions and low-level VrFlinger API in libdisplay.
     * Additional changes needed to keep old interfaces working for
       a short time while replacing the dependent code (dvrGraphics*).
  2. VrFlinger service implementation changes to support VrCore compositor
     and the removal of the internal compositor.
  3. Changes to libdvr platform library API due to changes in #1 and #2.

Because of the nature of the interdependence of types and other defs it is
difficult to break this CL into smaller chunks. However, review of the three
major modules (libdisplay, libdvr, and libvrflinger) may be done separately
to ease the mental burden on reviewers.

Change Summary:
- Remove obsolete screenshot service. VR screenshots will be implemented
  by VrCore.
- Update display protocol definitions for changes in VrFlinger service
  requirements. The majority of the changes in libdisplay are a
  consequence of these protocol and service changes.
- Update VrFlinger to support two kinds of surfaces:
    1. Application - use by VR apps.
    2. Direct - used by VrCore (protected by permission check).
- Remove VrFlinger internal compositor and GL context.
- Remove obsolete debug console.
- Update VrFlinger hardware composer interface to handle direct
  surfaces only, removing the concept of GPU (compositor) layers.
- Update display manager to expose access to application surface info
  to VrCore (protected by permission check).
- Update libdvr platform library interfaces for changes to VrFlinger
  API / protocol.
- Clean up libdvr API struct setup using a common include.
- Add C++ header-only helpers for DVR platform library opaque types.

Bug: 36401174
Test: Build; run VrFlinger display test tool.
Change-Id: I15abfde5f72dbb3725a3f58621486afba6b64902

Merge "surfaceflinger: make EventThread FIFO" into oc-dev
am: 0ad5feff27

Change-Id: Id59b34906f6a79420c75a2377e7f86906b6e81f0

Merge "Revert "Set the atrace clock to boot when possible and mono otherwise."" into oc-dev
am: 7fccc2079e

Change-Id: I183fd416c85c85ea1549eae2a417ef0a95b9a5bf

Merge "surfaceflinger: make EventThread FIFO" into oc-dev

Merge "Revert "Set the atrace clock to boot when possible and mono otherwise."" into oc-dev

Fix typo introduced by manual merge

Manual merge from previous commit contains an incorrect copy-paste
error.

Test: Build
Change-Id: Icde12b12d06c12e9d89adec231ceee1d2bb01995

Remove unnecessary clients and rename files.

- Remove clients for obsolete services.
- Rename display protocol defs file to match purpose.
- Rename display manager client file. "impl" does not match naming
  conventions used by DVR libraries.
- Remove video mesh surface interface. This will be replaced by more
  general surface support in the unified O compositor.

Bug: 36401174
Test: Build; run cube sea.
Change-Id: Idb782623319329f6fa1b0aa45005d9fee3b9957d

Support multiple consumer queues.

- Add support for importing posted buffers when spawing a new
  consumer queue.
- Correctly handle adding signaled buffers to epoll with edge
  triggered mode set.
- Add test for multi-consumer behavior.

Bug: 36401174
Test: buffer_hub_queue-test passes.
Change-Id: Id09f01502a1b18bf80a0ae465c2941b548cde2e4

Merge "Re-enable the zipping of bluetooth logs" into oc-dev
am: cf2efe056b

Change-Id: Ic92f75300d6cf4ac928c778730e843e62f674dfc

Merge commit '3a75c70a06795ff2d563d3828351ff7c8656c75a' into manual_merge_3a75c70a0

Test: Build
Change-Id: If6409a707e1d76a56c445a6286ea5dbde29e7a71

Merge "Re-enable the zipping of bluetooth logs" into oc-dev

Return pdx::Status<T> from BufferHubQueue::Dequeue.

Switch to using Status<T> to return buffers or meaningful errors
from BufferHubQueue::Dequeue. This enables determining whether an
error is normal (e.g. timeout) or abnormal (e.g. disconnect).

Bug: 36401174
Test: buffer_hub_queue-test passes.
Change-Id: Ifef5f737a5e737b70d19bdbffd7544a993438e1c

Add support for consumer queue initial import and hangup.

- Add support for consumer queues to import buffers that are created
  before the consumer queue is created, making multi-consumer queue
  patterns possible. This is essential for VrFlinger operation.
- Add support for notifying consumer queues when the producer queue
  hangs up.
- Correct the epoll event loop to check for hangups even when buffers
  are available.
- Add method to retrieve the event fd from a queue.
- Add trace logging and minor cleanup.
- Improve bufferhubd dump state output.

Bug: 36401174
Test: build; bufferhub tests pass.
Change-Id: Idd6f38a3341c048192062734e288d11de48bc4d4

Merge "Deprecate producer/consumer usage: Step 2" into oc-dev
am: 6f413c5f54

Change-Id: I3402c0cb46110c710c19a82af1cba044419df20e

Merge "Add 'override' specifier to overridden destructor methods" am: 44c9a3766f
am: dcb3ee039d

Change-Id: I4041248d20ad6a811761058f9bf5faeab3f8d6bd

Merge "Add 'override' specifier to overridden destructor methods"
am: 44c9a3766f

Change-Id: Ib41b7fef7b07cf7df6125cf1f09a58eb223d714c

Merge "Deprecate producer/consumer usage: Step 2" into oc-dev

Merge "Add 'override' specifier to overridden destructor methods"

Revert "Set the atrace clock to boot when possible and mono otherwise."

This change broke CTS tests.

This reverts commit 56a2ba0b0009b6a8db7c385489e5fef4c0d32d0a.

Bug: 37906221
Test: cts-tradefed run singleCommand cts-dev --module
CtsAtraceHostTestCases passes locally.
Change-Id: Ia82732eac33fbde8bddc1e8de7a68c20befe760e

Remove redundant dependencies

android.hidl.base@1.0 and android.hidl.manager@1.0 are built into libhidltransport.

Test: links
Bug: 33276472
Change-Id: Iaf818c23d1929c81f83978fcf605a8f80e741294
(cherry picked from commit cf58afb7fccd7a073e0d34b08ccddc45ee1c1be9)

surfaceflinger: make EventThread FIFO

Jitter in EventThread scheduling was contributing to SystemUI jank.

Test: thread is FIFO
bug 36631902

Change-Id: I930c729b27403ef8941e9a1e0ef8ec57a3d323a0

Switch touch input device on hover events.

On a phone in a Mirage headset, two touch nubs are in continuous
contact with the screen while in VR. In order to pass virtual
touchpad events to embedded 2D content that is confined to the
physical display (e.g. permission dialogues), allow a hover event
(as generated for the user's controller pointer) to initiate a
new event.

Bug: 36686408
Test: manual on device
Change-Id: I868472721599843b183b2615aac8e6cd87a58e8e

Merge "Correct sensor handle for operation parameter" into oc-dev
am: 8bba03d323

Change-Id: I4cfbf1d9924e3574054ba26567f3b959e657f35e

Merge "Correct sensor handle for operation parameter" into oc-dev

Add 'override' specifier to overridden destructor methods

Test: Build
Bug: 37752547
Change-Id: I157e1c2aa5d9d08fcab6055a3ca48c6ed8a050fc

Merge "Add const specifier to SortLayersByZ comparator" am: 40f0088f45 am: f2995e72f4
am: 221869c797  -s ours

Change-Id: Ied0cdd6d215d76160418eb32913081d4b1611dba

Merge "Add const specifier to SortLayersByZ comparator" am: 40f0088f45
am: f2995e72f4

Change-Id: I0eeb4aa44ae0ee2072da9b6b5cad173f7009d9b0

Merge "Add const specifier to SortLayersByZ comparator"
am: 40f0088f45

Change-Id: Ia7316e218e388b6f3760374751b32bb992c6441a

Merge "Add const specifier to SortLayersByZ comparator"

Merge "installd create_data_user_ce_path uses dir instead of symlink" am: f8af02fffa am: 21ea7683bc
am: 906a243b4b

Change-Id: I31741b3f6ead4a9468b56f779e459d76b3029537

Merge "installd create_data_user_ce_path uses dir instead of symlink" am: f8af02fffa
am: 21ea7683bc

Change-Id: Ie8b0042f30066c59e22c2da99abc2adff400e79a

Merge "installd create_data_user_ce_path uses dir instead of symlink"
am: f8af02fffa

Change-Id: I22a693a5bf288876413bbb3b1dfe2c364944deaa