Remove obsolete DEXOPT_SAFEMODE.
am: 2520d44e66

Change-Id: I2f053a09a4eeb48bcee07bb0c9509709bb8dc50f

Merge "Remove obsolete DEXOPT_SAFEMODE." am: 4fa2b96d21 am: d1058fb90c
am: 422eab43cc

Change-Id: I5547f465814d5cd3dca05e0a4c8e0f2473febc75

Merge "Remove obsolete DEXOPT_SAFEMODE." am: 4fa2b96d21
am: d1058fb90c

Change-Id: Ic9dd064d8eb66f1dd99302c0cfed83a5e4ca2233

Merge "Handle safe mode in PackageManager." into oc-dev
am: 89726a1dea

Change-Id: I71dc793457b9f04d91d1863e2a33c70936c63ebd

Merge "Remove obsolete DEXOPT_SAFEMODE."
am: 4fa2b96d21

Change-Id: I652a433fb0e042b79e4289faf74b8b4c6cd84f91

Remove obsolete DEXOPT_SAFEMODE.

installd side of the change.

bug:37929796
Test: build
Change-Id: I358c0784a7b563cbd6ceaecdb80710971f483d02
(cherry picked from commit bad623a225d0b3d94f30f8654bd4684969e90d4c)

Merge "Handle safe mode in PackageManager." into oc-dev

Merge "Remove obsolete DEXOPT_SAFEMODE."

Merge "Add uniqueId to Virtual Display and pass through to inputflinger (2/2)" into oc-dev
am: 02207005fa  -s ours

Change-Id: I94e6f135f5e49d1514354ae326075ea4bca496f3

Merge "Logging for catching memory issue in Parcel" into oc-dev
am: c8ddb0171f

Change-Id: Ifc85054fa35bfc89722c8b37337b2d540164d122

Merge "Surfaceflinger: Propagate type & UID from parent layers" into oc-dev
am: 4ba6f99070

Change-Id: Iefbb9bd3c578fe8d8662f7d35422f6f20b8110a1

Merge "Add uniqueId to Virtual Display and pass through to inputflinger (2/2)" into oc-dev

Add uniqueId to Virtual Display and pass through to inputflinger (2/2)

This CL adds:
1) Adds uniqueId (protected via system/sig permission) to virtual
displays.
2) Add support for N virtual display viewports into inputflinger.
3) Set the virtual display's viewports in inputflinger if it has the
uniqueId value set to non-null. (a) Moving the new viewport from java to
native inputflinger and (b) adding "uniqueId" value to viewports makes
up the great majority of this change.
4) From the inputflinger side, we also read in a new value from the
input device configuration files called 'touch.displayId'.
5) When touch.displayId and the virtual display's uniqueId match,
inputflinger links the two.

Test: Start VR and ensure that the virtual viewport shows up when running
'adb shell dump input".  Run a VR app, and ensure that the virtual input
device is associated with the new virtual viewport.

Bug: 36051620
Change-Id: I662f09f863a3dc94c570e7e7f30b83888b3f514c
Merged-In: I662f09f863a3dc94c570e7e7f30b83888b3f514c

Merge "Logging for catching memory issue in Parcel" into oc-dev

Merge "Surfaceflinger: Propagate type & UID from parent layers" into oc-dev

Merge "Revert "atrace: Poke PDX services using utility class."" into oc-dev
am: 17cc37dc15

Change-Id: Ic31751da5e2699c9a9e11fd0df716847c0685d7e

Merge "docs: Fixed comments to generate API refs correctly. Test: Verified by manually generating the API refs." into oc-dev
am: a68ae7232b

Change-Id: I15968454d899a4c3f676527b9724b6fbb49fc0bd

Merge "SF: Use last call time to rate limit resyncs" into oc-dev
am: 2e252c93ac

Change-Id: Id9ecf8b6aeb8ce3b084c1aa21096a0dc03e3fb7c

Merge "Otapreopt: Work around bug 38186355" into oc-dev
am: 4c8cc1c86a

Change-Id: Ic8b695f766165402fd2f66f54efcf30b76b498fe

Merge "Correct PowerManager transaction IDs." into oc-dev
am: 5aab417693

Change-Id: I0c9897e77cbd35991dae21aaec06466f8c071316

Merge "lshal: remove "// static" comments" am: 8d2f788a05 am: 7d6944d40a
am: 45bf90c166

Change-Id: I3fd98e82715890ed1a83862dfe369a15b316c3ad

Merge "lshal: remove "// static" comments" am: 8d2f788a05
am: 7d6944d40a

Change-Id: I36367aa4f5bd756d31855186066cd2e930e32414

Merge "lshal: remove "// static" comments"
am: 8d2f788a05

Change-Id: Id2eacee2cd265922954cc3b480015c6fe9efa6d1

Merge "lshal: remove "// static" comments"

Merge "Add explicit default copy constructor" am: c9810011e9 am: 3bf99597b4
am: 992fa650be

Change-Id: I536ae9a50f5b263c1a60011fdbc1bba269096d1e

Merge "Add explicit default copy constructor" am: c9810011e9
am: 3bf99597b4

Change-Id: I1866cb30496b1ad3d49171e1b2b44dcfe44575a3

Merge "Add explicit default copy constructor"
am: c9810011e9

Change-Id: I4d6198e65c54f3f620c031da3d9fa395d6d3762f

Merge "Add explicit default copy constructor"

lshal: remove "// static" comments

Test: pass
Change-Id: I4d710c8aaad87cfcd7896800a5ab24f812775703

Surfaceflinger: Propagate type & UID from parent layers

Bug: 36589137
Test: Launch chrome in VR (surfaceview has a parent)
Change-Id: I9320a8df249d1d5790323b22f0ba88a792f9468f

Add explicit default copy constructor

Definition of implicit copy constructor is deprecated if it has a
user-declared destructor. This triggers -Wdeprecated compiler warning
and breaks the build in latest Clang update.

Test: Build
Bug: 37752547
Change-Id: I467d5ff3b4cca4644c7ec8c99ae35556029ce509

Merge "Add uniqueId to Virtual Display and pass through to inputflinger (2/2)" into oc-dev-plus-aosp

Merge "Revert "atrace: Poke PDX services using utility class."" into oc-dev

Merge "Switch from libcrypto_static to libcrypto target" am: 612c12b293 am: 4c246fe5df
am: 25eacb0b5e  -s ours

Change-Id: I0142405d334c5615fbc10201716d1d312b3686a9

Merge "docs: Fixed comments to generate API refs correctly. Test: Verified by manually generating the API refs." into oc-dev

Merge "Switch from libcrypto_static to libcrypto target" am: 612c12b293
am: 4c246fe5df

Change-Id: I6f49fd3560dc03de06cba7e06e039e98e55d697e

Merge "Switch from libcrypto_static to libcrypto target"
am: 612c12b293

Change-Id: I12980dcc8ea7b809b87199e9d7959202a7abc385

Merge "Switch from libcrypto_static to libcrypto target"

Merge "SF: Use last call time to rate limit resyncs" into oc-dev

Logging for catching memory issue in Parcel

Added a check to catch a bug that is suspected to lead to a segfault
during GC. Actual fix for the bug is commented and will be checked in
once the flow that is hitting the bug is uncoverered

Bug: 37298089
Test: manual

Change-Id: I68dea02566a9e9375e3a09d4cc19e39379d84747
Signed-off-by: Michael Wachenschwanz <mwachens@google.com>

Revert "atrace: Poke PDX services using utility class."

This reverts commit f70680e995d6df4604c9b34f6a5afebe613d7feb.

There appears to be a bug in the current build system that does not
cleanly rebuild all targets using static libraries built across
different build systems (Soong/make). With the recent change to PDX
connection handshake, it is important that all clients and services are
rebuilt to include the new protocol changes. However this is not the
case when doing incremental builds. Some services remain untouched
during global builds and hence there is a disparity in clients' and
services' implementations of the protocol.

atrace being one of the clients is affected (the symptom is that when
started atrace just hangs waiting for connection to a PDX service).

While there is nothing wrong with the current state of code, but the
issue of incomplete incremental builds is causing headaches for people
trying to chase performance issues in O with systrace (since it is
unclear that they need to do a clean build).

Reverting the dependency on PDX in atrace for now until the build system
issue is identified and fixed property. This change can be re-landed then.

Bug: 38201226
Change-Id: Ica0efc38f600295ef6f64b9f9584ac41cb84f424

Add uniqueId to Virtual Display and pass through to inputflinger (2/2)

This CL adds:
1) Adds uniqueId (protected via system/sig permission) to virtual
displays.
2) Add support for N virtual display viewports into inputflinger.
3) Set the virtual display's viewports in inputflinger if it has the
uniqueId value set to non-null. (a) Moving the new viewport from java to
native inputflinger and (b) adding "uniqueId" value to viewports makes
up the great majority of this change.
4) From the inputflinger side, we also read in a new value from the
input device configuration files called 'touch.displayId'.
5) When touch.displayId and the virtual display's uniqueId match,
inputflinger links the two.

Test: Start VR and ensure that the virtual viewport shows up when running
'adb shell dump input".  Run a VR app, and ensure that the virtual input
device is associated with the new virtual viewport.

Bug: 36051620
Change-Id: I662f09f863a3dc94c570e7e7f30b83888b3f514c

Merge "Otapreopt: Work around bug 38186355" into oc-dev

docs: Fixed comments to generate API refs correctly.
Test: Verified by manually generating the API refs.

Change-Id: I77f2072d2aa83a6c71e11b216e206c4e6563ab88

Remove obsolete DEXOPT_SAFEMODE.

installd side of the change.

bug:37929796
Test: build
(cherry picked from commit bad623a225d0b3d94f30f8654bd4684969e90d4c)

Change-Id: Ice855d20184443a641e081efa4ec943326ccefdd

Otapreopt: Work around bug 38186355

Temporarily also accept a malformed string for null.

Bug: 38186355
Test: m
Test: manual OTA
Change-Id: I19d527d04b86596551f7bed54221c67be06c34c7

Merge "Correct PowerManager transaction IDs." into oc-dev

Merge "egl: recover from failed eglMakeCurrent" into oc-dev
am: 35c63d456a

Change-Id: If62e2d89620bc7f464779a3db2c6f977bf34f7de

Merge "egl: recover from failed eglMakeCurrent" into oc-dev

Handle safe mode in PackageManager.

installd side of the change.

bug:37929796
Test: manual OTA with a safemode app.

(cherry picked from commit be6ecd6f6141de1ecd4ec6eb5885bdd0f655fafa)

Change-Id: I43dcdd3fea3f62398f16f89348c1a2f1b4b1bb6d

Merge "Always unlink the out_vdex_file." into oc-dev
am: 6949ec68e7

Change-Id: I2e9bf76d52567191bdceccbadc95defbb7a0fd0f

Merge "Always unlink the out_vdex_file." into oc-dev

Merge "Always unlink the out_vdex_file." am: fbb09a7192 am: e5a1bc620b
am: 2087818222

Change-Id: Iab9661cb66924b200263acaac4ce4f2876481a52

Merge "Always unlink the out_vdex_file." am: fbb09a7192
am: e5a1bc620b

Change-Id: Ib4ecf1c85ec826edee1de3cd11e565d0ecaf7114

Merge "Always unlink the out_vdex_file."
am: fbb09a7192

Change-Id: If1eed0f380172f05b4fbd7a8f853a9d7db6f4aa7

Always unlink the out_vdex_file.

Even if we pass the same vdex file as input and output to
dex2oat, make sure we unlink it if dex2oat failed.

bug:38113240
Test: manual, killing dex2oat invocation

(cherry picked from commit 7bb165779f59bfa1324ca991f64ab98331a410b5)

Change-Id: I157b3d33179b5bff1ca38750bee24aab01c5a19b

Merge "Always unlink the out_vdex_file."

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

Always unlink the out_vdex_file.

Even if we pass the same vdex file as input and output to
dex2oat, make sure we unlink it if dex2oat failed.

bug:38113240
Test: manual, killing dex2oat invocation
Change-Id: I406014f97a2c3b2bd646e13b5d86dfa0eaa2692f

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

Switch from libcrypto_static to libcrypto target

libcrypto target offers the same static library as libcrypto_static.
Thus, there's no need to use libcrypto_static.

Test: make dumpstate
      dumpstate binary unchanged by this commit
Bug: 26160319

Change-Id: I62ca775870482386f0bb7ce1e2f8c94cf6f0342a

SF: Use last call time to rate limit resyncs

SurfaceFlinger has a method which tells it to resync to hardware vsync
as long as it hasn't resynced too recently. This is used when we
receive a request for a Choreographer wakeup since if it has been a
while since we animated, we have likely drifted.

To determine whether we should actually resync or whether we should
instead rate-limit the call, we compare the current time to the last
time we drew, but this is problematic. When we first start animating
after a period of inactivity, this triggers a resync every time the
method is called until we receive the first frame and draw it, even if
we are already mid-resync, which delays the time until we can lock onto
the hardware vsync phase and has the side-effect of causing weird wakeup
times in both Choreographer and SurfaceFlinger.

This change instead keeps a local timestamp which is updated every time
the method is called and therefore effectively rate limits even before
the first frame is received.

Bug: 38117777
Test: ApiDemos/ListView + manual inspection of systraces
Change-Id: I7210594f9ed6ed5397bb0f3f14a8966503454643

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

Change-Id: I31ff9da171baa83318bc794553c7057b18be2dec

egl: recover from failed eglMakeCurrent

Some drivers don't actually restore current context when
eglMakeCurrent fails.  Restore the current context directly.

Bug: 37244059
Change-Id: I857da78078dd212b523d690415cf525ecd463961
Fixes: 37244059
Test: android.server.cts.ActivityManagerDisplayTests

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