Fix seamless rotation with SurfaceView WM bypass.

First a brief review on seamless rotation. In seamless rotation
the WM will do the following atomically:

1. Update the display transform.
2. Set window transforms to make the surfaces at their old size appear
   at the correct location in the new coordinate space.
3. Set a flag to undo the transforms when the surfaces resize.

However, windows with NATIVE_WINDOW_TRANSFORM_DISPLAY needed to be special cased.
The flag will perform step 2 in terms of buffer mapping, but not in terms of bounds
computation. So to compensate the WM would apply the transform to the bounds,
set these bounds, and then skip applying the transform matrix (as the buffer is
transformed by the flag).

Now that the WM can't see the Camera view, it can't implement this special logic
and the Camera view inherits the parent transform. We compensate for this in
SurfaceFlinger by omitting parent rotation transform components for
Surfaces with NATIVE_WINDOW_TRANSFORM_INVERSE_DISPLAY. To some extent this
breaks arbitrary rotation for surfaces with NATIVE_WINDOW_TRANSFORM_INVERSE_DISPLAY
but this is essentially ok:

1. We lived with that bug until N-MR1
2. We fixed it in N-MR1 because the camera was falling back to ROTATE
   a lot which shouldn't be happening anymore, following introduction
   of specifying rotation animation in manifest.

Test: Rotate camera, disable HWC, rotate camera some more. Switch to front camera, repeat.
Bug: 36230754
Bug: 36727915
Change-Id: Ied390c9cb3968fcce32a84ee7947f699746fdc81

libgui: Make ISurfaceComposerClient a SafeInterface

Converts ISurfaceComposerClient (specifically its Bp/Bn children) to be
SafeInterfaces such that all parceling/unparceling is done
automatically.

Test: Manual testing
Change-Id: Ie5eee51fc893b0e4d7c3b498fe756306f9a029b5

libbinder: Add SafeInterface

Adds SafeInterface, a mechanism which will automatically translate
IInterface method calls into remote procedure calls, avoiding the need
to manually write parceling and unparceling code for each method.

Test: binderSafeInterfaceTest
Change-Id: I0fc7a6eee5528371f40b90d056404cb024166b23

Merge "vndk/window.h must be C compatible" into oc-dev

Merge "Add 180 rotation to sensord orientation" into oc-dev

Merge "Add basic dvrSurface C API" into oc-dev

vndk/window.h must be C compatible

Test: added a c compatibility test
Bug: 36725813
Change-Id: Ic7e1c8b50e1818707a9a457d84ed74e7e6c816c8

Add 180 rotation to sensord orientation

Bug: None
Test: Manually tested locally
Change-Id: I135416a2c5be68172ff466bad503261adf29c5a9

Merge "vrwm: Fix controller input/rendering mismatch" into oc-dev

Add basic dvrSurface C API

Test: Build and flash system
Bug: 36563654
Change-Id: Id6c79d33d80e719b1d1d7f43aec99aadfad40c03

Merge "Make sure dvr functions are prepended with dvr" into oc-dev

vrwm: Fix controller input/rendering mismatch

This works for portrait mode only for now. Landscape phones present
some weird buffer sizes (1080x1080) that get scaled incorrectly for
input

This also removes all of our older hacks, such as swapping
width/height in SurfaceFlingerView and the swap + invert of touch
input. A global 90 degree rotation is applied to present the portrait
buffers in the correct orientation

vr_wm_ctl has a command to stack another rotation on top of the
existing ones in order to facilitate development for now. Apps will
likely show up rotated, "vr_wm_ctl rotate 1" will fix it, or -1 if
it needs to be rotated the other way.

Bug: None
Test: Manual with permissions and calculator

Change-Id: Ifd31b444b944fbf3085613349caae37e817538f6
(cherry picked from commit eb49c451cbc5de7bbe5ba440dc8e43291daa6d4e)

Merge "Recenter upon notification from VrCore" into oc-dev

Make sure dvr functions are prepended with dvr

* Fix some inconsistencies

Bug: None
Test: Compiled
Change-Id: I9d67d2335f543700bd1a90d329e4b155f4c22c69

Recenter upon notification from VrCore

Bug: 36201389
Test: manual on device
Change-Id: I690b70130e45732f052d136bedd587ddbf9493b4

Merge "Return jobject when getting android Java Surface" into oc-dev

Temporarily disable 2 unratified Khronos extensions.

Test: Ran the following CTS test cases, which now say "NotSupported":
dEQP-VK.wsi.android.surface.query_capabilities2
dEQP-VK.wsi.android.surface.query_formats2
dEQP-VK.wsi.android.shared_presentable_image.scale_none.demand
dEQP-VK.wsi.android.shared_presentable_image.scale_none.continuous
dEQP-VK.wsi.android.shared_presentable_image.scale_up.demand
dEQP-VK.wsi.android.shared_presentable_image.scale_up.continuous
dEQP-VK.wsi.android.shared_presentable_image.scale_down.demand
dEQP-VK.wsi.android.shared_presentable_image.scale_down.continuous

Disable the Khronos/Vulkan VK_KHR_get_surface_capabilities2 and
VK_KHR_shared_presentable_image extensions until the May 5th
ratification of these two extensions.  A macro/ifdef's are used to allow
easy enablement of the extensions for internal testing and bug fixes.

V2: convert preprocessor flag to constant [chrisforbes]

Change-Id: Ic41360e42561a822d51e1d7fc3172c76c12e5bde

Return jobject when getting android Java Surface

This logic is originally borrowed from similar API from shim_gvr API
where we cannot explicity deps on JNI.h in the public header. This
constrain no longer exists as libdvr is an Android platform library.

Bug: 2013449
Test: Built libdvr
Change-Id: Ide72f5fb706cd24e13507749216d7708f1f40fcc

Merge "Use lshal's new --debug option when emitting bugreports." into oc-dev

Merge "Use gralloc usage conversion library" into oc-dev

Temporarily skipping dump of bluetooth logs.

Bug: 35998031
Test: manual verification
Change-Id: I0bdf6eb1546c0850b301441f48753b65fa7b1e5e

Use lshal's new --debug option when emitting bugreports.

This calls into live binderized services and logs their "IBase::debug(...)"
output.

Bug: 36520620
Test: make, adb shell lshal, adb shell bugreport
Change-Id: I421e30e74c82817adf276b5750ce40038852df83

Use gralloc usage conversion library

Converting from ..graphics.allocator@2.0 usage to gralloc0 usage isn't
quite as simple as ORing and truncating, which is what the default
implementation was doing. Switch to using library functions that do it
correctly.

Test: boot bullhead
Change-Id: Ibe644b53655e191b9bca9d619825a8918e8fc365

Merge "Remove legacy cache clearing logic."

Merge "Add stub for SensorManager::createEventQueue"

fixed bug undefining LIKELY/UNLIKELY in math lib

Bug: 36275627
Test: Built and check the surfaceflinger compiled

Change-Id: I062b8c0db540443635ac746824edb295190235d0

Merge "Disable BufferQueueInAnotherProcess test"

Remove legacy cache clearing logic.

It was riddled with funky memory access, and it's been fully replaced
by the new V2 logic.

Test: builds, boots
Bug: 27948817, 36655947
Change-Id: Ic5f0636c414ccf5ccf56eb50fea321b6688a83a0

Merge changes Iad7bf035,I870e08c6

* changes:
  VNDK ANativeWindow API - step 2
  VNDK ANativeWindow API

Disable BufferQueueInAnotherProcess test

The BufferQueueInAnotherProcess relies on executing before any test that
tries to open a BufferQueue in the local process. A recent test addition
broke the condition. Upcoming SF & BQ changes are likely to obsolete the
test, so won't spend time on fixing it. Disabling for now and the test
will be removed when the infrastructure changes land.

Test: libgui_test
Bug: 36592665
Change-Id: I648e197cab6bab3aed445390069265589a717bf1

Merge "Enable --debug option for "lshal" command." am: 16175ca725 am: 674d3acd18
am: 829f3c6d85

Change-Id: I6751fec855783983b96672bec82ae9ba0c737a8b

Merge "Enable --debug option for "lshal" command." am: 16175ca725
am: 674d3acd18

Change-Id: If87550eb68a9c8e280d8863063b4311a44ff6896

Merge "Enable --debug option for "lshal" command."
am: 16175ca725

Change-Id: Ifa29afcb7677a3ec76a03d5f0365ba4de7ee4c9b

Merge "Enable --debug option for "lshal" command."

Merge "libgfx: Remove SafeLog"

Merge changes from topic 'libdvr'

* changes:
  Add dvr_buffer_queue C API
  Move DVR platform C API into libdvr

VNDK ANativeWindow API - step 2

The vndk API is a superset of the NDK API.
Prior to this, vendor would have access to
system/window.h which itself is a superset of
what they need, exposes too much of the internals.

With this change, system/window.h depends on
vndk/window.h which depends on
android/native_window.h

Test: compiled, booted Nexus 6P
Bug: 34453351
Change-Id: Iad7bf035143541843309437fe630cc603d96a76e

VNDK ANativeWindow API

The vndk API is a superset of the NDK API.
Prior to this, vendor would have access to
system/window.h which itself is a superset of
what they need, exposes too much of the internals.

With this change, system/window.h depends on
vndk/window.h which depends on
android/native_window.h

Test: compiled, booted Nexus 6P
Bug: 34453351
Change-Id: I870e08c652d90b3f2cf4ed7c7ca541ce2893f689

Merge "VR: Rename HIDL service name from vr_hwcomposer to vr"

Merge "Properly recover vr_wm from surface vr_flinger crashes."

Merge "Update native ColorSpace API for libhwui"

Update native ColorSpace API for libhwui

Test: colorspace_test
Bug: 32984164
Change-Id: Id0728bfa90192173eb45c2d26a8b81ce970398d7

Properly recover vr_wm from surface vr_flinger crashes.

This is addressing a number of issues:

1) surface_flinger_view_ was incorrectly reset
on de-allocation, which renders vr_wm incapable of seeing
new frames after a crash.

2) dvrGraphicsWaitNextFrame will return -1 if vr_flinger
crashes, which means our graphics context is no longer valid.
We need to de-allocate resources and wait for SF to
reconnect.

3) Displays needed be be de-allocated on SF detach because
they will return incorrect fences on SF re-connect.

4) display_client_ needs to be allocated on demand instead
of at initialization time because vr_flinger crash
invalidates it.

Bug: 36589793
Test: Crash vr_flinger, observe vr_wm working after SF restarts.

Change-Id: I72a79690ee5224e3326c7c571ae4e23797103e6d

This test is used to check if the kernel scheduler

1. supports priority inheritance.
2. utilize the sync flag.

It also make statistic on average, best and worst latency
as well as number of deadline miss with a configurable
deadline specified in us.

The output is in json format hopefully to make it easy
to integrated with present test framework.

usage:

./schd-dbg                   # test and output json
./schd-dbg -i 1              # number of iterations
./schd-dbg -pair 4           # number of process pairs
./schd-dbg -deadline_us 2500 # deadline in us
./schd-dbg -v                # debug

Test: pixel
Change-Id: Idccf0d402dcd7b81c639020364b5ee32075e18cb

Merge "libdvrcommon: Remove revision code."

libdvrcommon: Remove revision code.

Move revision to libdvrplatform.

Bug: 36040117
Test: N/A.
Change-Id: I3f70509f9a694383bd7eb94173d5230bd8c66e51

Checks hidl service and remove hidl call at dump

Two things:
  * check for hidl service availability before calling any hidl
    functions.
  * remove hidl dependency on dump to ensure dump will not cause
    crash due to hidl failure.

Bug: 36073404
Test: check sensor running after reboot
Test: check dumpsys sensorservice works normally.
Change-Id: I8626635a143a9f07f263f85d95ab16749ea1e452

Merge "Revert "Quick fix to correctly handle empty file/channel handles.""

Enable --debug option for "lshal" command.

This calls into live binderized services and logs their "IBase::debug(...)"
output.

Bug: 36520620
Test: make, adb shell lshal, adb shell bugreport
Change-Id: Ida3b0c61eafd9f3de562a5df3af24bbaf383de22

VR: Rename HIDL service name from vr_hwcomposer to vr

This is part of an effort to simplify service names.

Bug: 36564122
Test: Ran on device and verified SurfaceFlinger connects to vr_wm
Change-Id: I3e2a2b778f3f0d2066bf5b756263de4b238b2f83

Add dvr_buffer_queue C API

Bug: 36401767
Bug: 32213274
Bug: 36266201
Test: build and ran dvr_buffer_queue-test
Change-Id: Icd8d063bd8b7adb15bf67f7e543b87dad1c32a68

Move DVR platform C API into libdvr

Introduce a new static library build with Android.mk. The reasoning
behind this is:
1/ This library is going to consolidate all C headers and implementation
of the DVR platform library.
2/ No other internal system components need to depend on this library.
3/ This library is build with Android.mk so that it has depend on other
system components (such as libandroid_runtime) without worrying about
that Android.bp-based libs cannot depent and Androib.mk-based libraries.
4/ This library defines dvr_api.h header as the official definition of
DVR platform library headers. The header is moved from
'vendor/unbundled_google/packages/PrebuiltGoogleVr', which is not an
ideal place to hold the platform library anyway.
5/ If we ever decide to move these C wrappers out of frameworks/native,
this makes it easier to do so by bundling things now.

TODO: C APIs from libvrsensor and libvirtualtouchpadclient are not moved
into this bundle as some functions are still used by internal system
components. Ideally, these will be moved either to Google3 or vendor projects.

Test: Built and flash
Bug: 36563654
Change-Id: Ie5ce89faa70f86ac934a77d8f7d4fca7e94112ee

Revert "Quick fix to correctly handle empty file/channel handles."

This reverts commit e71fb7d2ab1118d30c5bec8a85114ac610844dc6.

This CL broke PDX unit tests

Bug: 36587839
Change-Id: I5b3cb857aacff337cb9a268a0415f8398303abe8

Merge "media: mechanism to set/get vendor-extension configs for OMX component"

Merge "Remove more EGL_ANDROID_create_native_client_buffer references"

Remove more EGL_ANDROID_create_native_client_buffer references

Bug: 35101209
Test: build
Change-Id: I8709b9881f0133756e2cfcfad3819d4032cfb03e

Add stub for SensorManager::createEventQueue

It is not yet implemented.

Test: pass
Bug: 35219747
Change-Id: I8ef1627376ac88f552c20a2809416115ba342b82

Merge "Implement an API for vrwm to query if the current vr app is active"

binder: remove cgroup interactions [ DO NOT MERGE ] am: 05f4679d03
am: 400e9a72fd  -s ours

Change-Id: Ic31695b9c7a981341ab601638b68f0bb7ef99f28

binder: remove cgroup interactions [ DO NOT MERGE ]
am: 05f4679d03

Change-Id: Idba2f747151fc50866fb470681e50ba64b28b2d0

Merge "Revert "atrace: Make atrace notify VR services of new trace sessions""

Merge "Fix typo in Region to avoid unexpected values" am: 2cbfb0cda6 am: 7fc294e10e
am: 5288e22d00

Change-Id: Ib8095bd01becfacbbcecd0f4ec369ed933eff173

Merge "Dumpstate: dump HAL native traces processes."

Merge "Fix typo in Region to avoid unexpected values" am: 2cbfb0cda6
am: 7fc294e10e

Change-Id: I7feff39e1be80c69a744c4eefad835ba50796f4c

Merge "Fix typo in Region to avoid unexpected values"
am: 2cbfb0cda6

Change-Id: I92a204fa492374c4c69a5d794d974b3cb8f454b3

Merge "Fix typo in Region to avoid unexpected values"

Implement an API for vrwm to query if the current vr app is active

The API returns if the client_order 0 layer is visible or not. This
aids in 2D app detection

Bug: None
Test: Manual with permissionsgen & calculator

Change-Id: Id988e61a9f93885e93c28ed83ffaffd84f3bdf15

Revert "atrace: Make atrace notify VR services of new trace sessions"

This reverts commit 213feb32c76bb4ed4262dad8c71f0377ee710fac.

Reason for revert: tracing & perf teams don't want to support PDX at this point in time.

Change-Id: I28b8a7c55c9d31c8469236fb60a6e034b1ef8af7

Merge "Enable Universal Bandwidth Compression for textures"

Enable Universal Bandwidth Compression for textures

We removed texture compression for VR display surfaces, which introduced memory
bandwidth issues on some QC platforms.

This patch reintroduces texture compression for QC devices compatible with
both gralloc0/gralloc1 implementations.

Bug: 36370337
Test: Validate VR comes up
Change-Id: Ieaaa56f67522c85913476ed0d8da04f8eedbce63

Dumpstate: dump HAL native traces processes.

Test: hal service dumps in data/anr/bugreport-traces.txt after bugreport
Bug: 36414311
Change-Id: Ifca491d02f77bf4fb26450f06c2ea02769475786

media: mechanism to set/get vendor-extension configs for OMX component

Add OMX_CONFIG_ANDROID_VENDOR_EXTENSIONTYPE and
OMX_CONFIG_ANDROID_VENDOR_PARAMTYPE.

Android vendor extensions for OMX components have a name and one or
more parameter values - each with a string key - that are set together.
Support for not-set values is provided for vendor extensions in order
to not suddenly confuse applications with format keys that they were
not expecting.

Bug: 32746650
Change-Id: I21aad4a82d509fe2466764f65ee9e3b2d32f0842

Merge "Re-add access of DisplayClient from vr_rm"

Re-add access of DisplayClient from vr_rm

This was removed in a previous commit because it relied on an in-flight
CL that had not yet landed. Now that it has landed, the change is back
in.

Bug: 36139334
Test: Compile and run VR app, confirm that touch region is correct
      through hierarchy viewer.

Change-Id: I4ecc5e050fd896ba7442eac8b3a4e0b793915ed7

binder: remove cgroup interactions [ DO NOT MERGE ]

Having binder change cgroups on the fly is too expensive in the common case to be useful.

Test: builds

bug 34825105

Change-Id: Ia692c9d2ab261d852cb2fac8f27d6779d0e2b057

Merge "Check board config for wide-color color spaces"

Merge "VR: Add dvr API for service apps connecting to VR HWC"

Check board config for wide-color color spaces

Check that board config allows wide-color color
spaces.
Test: make tests in libs/gui/tests/
Test: adb sync
Test: adb shell /data/nativetest/libgui_test/libgui_test --gtest_filter="SurfaceTest.GetWideColorSupport"

bug #35996305

Change-Id: Ia5b8116bc25befb6491e72e67cd8ed3c90ca5698

Merge "Quick fix to correctly handle empty file/channel handles."

Merge "libpdx: Remove superfluous DispatchRemoteMethod overloads."

Add notification_title and notification_description to dumpstate

Initiator of dumpstate can set dumpstate.options.title and
dumpstate.options.description properties to notify dumpstate the details
that will be used to file a bug. dumpstate will send this information
along with the BUGREPORT_FINISHED intent.

Bug: 33561517
Test: Use the new API to invoke bugreport from TelephonyMonitor and
      ensure that the bugreport is taken and title and description are
      updated properly in the notification. Run dumpstate and
      BugreportReceiver unit tests

Change-Id: Ia23a3ef4d5751be14f347ac38ef7c8a18f79799d

Merge "Follow "atomic" to "group" refactoring."

Merge "lshal: --init-vintf exclude system hals" am: 8db626d2f3 am: 34206360f2
am: d1b6b40adb

Change-Id: Ie2d28a374a3fb18d969783394b80b3a0d6fe0869

Merge "lshal: --init-vintf exclude system hals" am: 8db626d2f3
am: 34206360f2

Change-Id: I21d0b37f6942204c748017e03e8c9a0e892a39dc

Merge "lshal: --init-vintf exclude system hals"
am: 8db626d2f3

Change-Id: Ibe4636eb13c9439d09f51948ca49921a1e209747

Merge "lshal: --init-vintf exclude system hals"

Merge "Fix BlockUntilVSync() to actually block"

Follow "atomic" to "group" refactoring.

Remove noisy logging about UIDs that are relying on default cache
quota of 64MiB.

Move away from yucky old statfs() and use statvfs() instead.

Test: /data/nativetest/installd_cache_test/installd_cache_test
Bug: 35812899, 35684969, 36482620
Change-Id: I3d68da97eac2ebcda489bdf9d27061cac5b3f7cc

Merge "SurfaceFlinger: Missed changes in HWC1. Fix build error."

lshal: --init-vintf exclude system hals

--init-vintf is provided for convenience so that vendors can initialize
their vendor interface. Excluding system hals so that they aren't
included in the default vintf file that is generated. They are
maintained manually.

Test: Verified frameworks hals are no longer visible in init-vintf.
Bug: 34772739
Change-Id: Ie40c384334affcb3abee063d58000b808f04e234

Fix BlockUntilVSync() to actually block

In my refactoring in ag/2008740 I introduced a regression where blocking
until the next vsync would busy loop instead of actually blocking on an
event on the vsync fd. The problem was I changed the poll() call to
listen for POLLIN in addition to POLLPRI. While waiting for vsync events
we only want to listen for POLLPRI.

Bug: 36495351

Test: Confirmed with debug logs the busy loop bug and that we correctly
block with this patch applied.

Change-Id: Iec4951e014575a5d0ed3cfe3fc20ea91c67edf1b

SurfaceFlinger: Missed changes in HWC1. Fix build error.

Apparently HWC1 isn't in presubmit anymore.

Test: Manual
Change-Id: Id173d71be62b8a3c888943c57c0ae0ec511bc94b

Merge "dumpstate: Remove FWDUMP_bcmdhd support" am: 86388dbc9a am: 2d71f97dfb
am: b09149f6ec  -s ours

Change-Id: Iba7447b4f5eb32cbdd480b69fcc369d1ee1a3e3f

Merge "batteryservice: add scheduleUpdate binder method" am: e007716bc4 am: 799bb3501a
am: cd1d8c6b10

Change-Id: I6fcf1788701a7b3c9e481a2bdfbdd352cf24d598

Merge "dumpstate: Remove FWDUMP_bcmdhd support" am: 86388dbc9a
am: 2d71f97dfb

Change-Id: I1e3a7ae0e7855a8003f463586d368a81ad89be5f

Merge "batteryservice: add scheduleUpdate binder method" am: e007716bc4
am: 799bb3501a

Change-Id: Ia1ef3d769c453bc58cc52b93df5d6f03e601d23e

Merge "dumpstate: Remove FWDUMP_bcmdhd support"
am: 86388dbc9a

Change-Id: Iadf37acb012f762919567354a7aede95f81b9eff

Merge "batteryservice: add scheduleUpdate binder method"
am: e007716bc4

Change-Id: I8565c0502c3838d74fcd09ec75677e65cf85a0a9

Merge "dumpstate: Remove FWDUMP_bcmdhd support"

Merge "batteryservice: add scheduleUpdate binder method"