bufferhubd: Fix an error on not closing post_fence explicitly.
am: 052f3aeb8c

Change-Id: I4d8f0951f31a62847b048be1db62603abe765c3a

libbufferhub: Fix compiler warning with printf format mismatch
am: a1336cf0de

Change-Id: Ie3b510a88b7ecdac194027d5761c0a9cc4a9a70b

libpdx_uds: Improve client connection logic
am: 4782814b34

Change-Id: I9b1653020b4f619d9761a66886e73a962fc26777

Remove unused dependency on libhardware from bufferhubd
am: 4ab4aa40eb

Change-Id: I8055ca59179a3097ec3b516b024800bc8c3acef8

libbufferhubqueue: Fix syntax in Android.bp
am: 734c7ec57e

Change-Id: Idb0c63c51ad757ca9bd803d89e70d985abe889df

bufferhubd: Fix an error on not closing post_fence explicitly.

There is a typo in the code that instead of calling close() on a Post
fence, actually did get_fd() which is really a no-op.

Fixed by changing the call to call() explicitly.

Bug: None
Test: `m -j32` succeeds. Sailfish boots, CubeSea works.
Change-Id: If788aa8ea291317f0618724ecc35a4b355575772

libbufferhub: Fix compiler warning with printf format mismatch

std::vector<bool>::at() does not return bool& but a special reference
proxy class. Passing this class on the stack for "%d" printf specifier
is unsafe.

Compiler throws a warning to indicate this problem:

    format specifies type 'int' but the argument has type 'reference'
    (aka '__bit_reference<std::__1::vector<bool, std::__1::allocator<bool>>>')

Fixed by explicitly casting to int before pushing the value onto the stack.

Bug: None
Test: `m -j32` = no more warning
Change-Id: I7b3bc4908e4932c25be60fc350e0e0b439842b66

libpdx_uds: Improve client connection logic

Handle the case where the service created a socket file but not bound the
socket to it, or not listening for incoming requests yet. Also, if the service
has crashed and left the socket file behind, need to make sure we reconnect
once the service is restarted.

Bug: None
Test: `m -j32` succeeds, device boots and CubeSea app works on Sailfish
Change-Id: I2039cfca6faccd5d1d4b725e454075669484b880
(cherry picked from commit d074fc416a7a90e76a0b28a870080c1dd30079a1)

Remove unused dependency on libhardware from bufferhubd

'libhardware' doesn't seem to be used anymore. Remove it.

Bug: None
Test: `m -j32` succeeds.
Change-Id: I2ff002ce17d920f44509347ca86ca5f030348cf5
(cherry picked from commit f9f18b6b5c413ffeb8cc3ac67dd49bc17364a76f)

libbufferhubqueue: Fix syntax in Android.bp

There was a spelling mistake for cflags property of libbufferhub
cc_library object.

Bug: None
Test: `m -j32` succeeds
Change-Id: I862c2266d86586469dc7299bf727f75bb3df6512
(cherry picked from commit 66fd7cb0f3b1a70121fb223b27a51f06534ebc46)

Merge "Add test config to dumpsys_test" am: cf617abcb2 am: c19f57f08e
am: e678a5ad0a

Change-Id: Ia879ceb72a4d8ba3763b026bf42cfa08ea81ca9c

Merge "Add test config to libgui_test" am: 7f1e92dc64 am: ca8cc51273
am: a485d63540

Change-Id: I03aab5fedd20ada0f10d6865aecd31e843b635eb

Merge "Add test config to dumpsys_test" am: cf617abcb2
am: c19f57f08e

Change-Id: Id7e21b0b4c5a182b24b229298a56f5508bedef31

Merge "Add test config to libgui_test" am: 7f1e92dc64
am: ca8cc51273

Change-Id: Ie4d7224ff3677ac9df8cf5c33a14aea993b2de29

Merge "Add test config to dumpsys_test"
am: cf617abcb2

Change-Id: I6257e48eeaf89e26ff4c46e69dd20941dae58392

Merge "Add test config to libgui_test"
am: 7f1e92dc64

Change-Id: Iaf39fdc5a6a9fb45cd0fb44504144d6eee66b909

Merge "Add test config to dumpsys_test"

Merge "Add test config to libgui_test"

Merge "Remove obsolete and unused PATCHOAT_FOR_RELOCATION." am: 804b145356 am: 55cfd8607f
am: 766381b582

Change-Id: Ia46753ab12f575ddff835b5aee30fa25fa0f3619

Merge "Remove obsolete and unused PATCHOAT_FOR_RELOCATION." am: 804b145356
am: 55cfd8607f

Change-Id: I473d7c32b9590d79a261c06b9616270737f58a8a

Merge "Remove obsolete and unused PATCHOAT_FOR_RELOCATION."
am: 804b145356

Change-Id: I0c9a38e9be249bcb81f7b04b436a00d60b779fc5

Merge "Remove obsolete and unused PATCHOAT_FOR_RELOCATION."

Merge "Add alpha channel for  the EGLConfig" into oc-dev
am: 34ca5f2ab8

Change-Id: Iaafc8ee8c34cf0806104bcf8e4c6a3f412083b55

Merge "Add alpha channel for  the EGLConfig" into oc-dev

Merge "fix the api style" into oc-dev
am: befdb0ec8c

Change-Id: I44d9e97467b99b156909126f061e941ba808b77b

Merge "fix the api style" into oc-dev

Merge "VR: Pass display size in the HWC observer callback" into oc-dev
am: 7e73970d7c

Change-Id: I7dc6be66cef28f166dad59e39751c7d819eb6cd2

Merge "VR: Pass display size in the HWC observer callback" into oc-dev

Fix seamless rotation with SurfaceView WM bypass.
am: cae605cd56

Change-Id: I3e8fbbfcad28b53505516e85706e64fd54cecea1

libgui: Make ISurfaceComposerClient a SafeInterface
am: a615e47ef0

Change-Id: I6fca16316016d08a9cf23e11ab1d7135ae07daf9

libbinder: Add SafeInterface
am: d630e520de

Change-Id: I6791a3395bd1441902313778a7a22d8150f208e9

Add alpha channel for  the EGLConfig

If the alpha channel is not assigned,the framebuffer will be Opaque.
And it will mask the underlying video and make the video invisible
when play video width mediaplayer

cherry-picked from AOSP master

Issue: b/36025630
Test: clean build and boot PixelC
Change-Id: If756ad2aaa789f6ddb178333b3e8854f52c8eac3

fix the api style

google3 c++ style says "no _ on struct members"

Happy to oblige

Bug: None
Test: Compiled
Change-Id: I74b65908339551b703125fe16e5a6d11cd3e9a89

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

VR: Pass display size in the HWC observer callback

1) Unify the places where primary display size is queried.
2) Allows virtual displays to have a different size than the primary
display and the size propagates properly to VR HWC and its observer.

Bug: None
Test: Unittest and manually on device
Change-Id: I7e026c4f9b018d47ca400247beed27841d71c806
(cherry picked from commit 04c73d23d5e366c9f0337cdaa71bd2d15ebf1f4e)

Remove obsolete and unused PATCHOAT_FOR_RELOCATION.

Bug: 33192586
Test: m
Change-Id: I7868402804935c27e896229371480c7a120e2477

Add test config to libgui_test

Design doc:
Generalized Suites & the Unification of APCT & CTS Workflows Design/Roadmap
https://docs.google.com/document/d/1eabK3srlBLouMiBMrNP3xJPiRRdcoCquNxC8gBWPvx8/edit#heading=h.78vup5eivwzo

Details about test configs changes are tracked in doc
https://docs.google.com/document/d/1EWUjJ7fjy8ge_Nk0YQbFdRp8DSHo3z6GU0R8jLgrAcw/edit#

Bug: 35882476
Test: local test
Change-Id: Iff1d3b7166dfc217a4d57eb211467f87c5bf4c34

Add test config to dumpsys_test

Design doc:
Generalized Suites & the Unification of APCT & CTS Workflows Design/Roadmap
https://docs.google.com/document/d/1eabK3srlBLouMiBMrNP3xJPiRRdcoCquNxC8gBWPvx8/edit#heading=h.78vup5eivwzo

Details about test configs changes are tracked in doc
https://docs.google.com/document/d/1EWUjJ7fjy8ge_Nk0YQbFdRp8DSHo3z6GU0R8jLgrAcw/edit#

Bug: 35882476
Test: local test
Change-Id: Icfbf76319b42959a0a65c5f8666744420fd5266c

Merge "vndk/window.h must be C compatible" into oc-dev
am: 0e7305cb0a

Change-Id: Ic135b9dba72dfd2c38f5ff63a363d5bcb3e6ac4d

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

Merge "Add 180 rotation to sensord orientation" into oc-dev
am: 61a5669b16

Change-Id: Ib62f7f8b1c722702c91626e7977902f14fbbef5b

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

Merge "Add basic dvrSurface C API" into oc-dev
am: b1e51745c2

Change-Id: Ia9ebb60de3c72911346ad5afa6d95fed55aaf584

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
am: 98c195251a

Change-Id: I9b32c79614b3e95a1374cb33ffd424acf1fa226a

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
am: ad0a84fd2a

Change-Id: Ib72c00def4a8d4e43a3873e63744f09854e237a8

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
am: 92e9112fb4

Change-Id: Ic66922da171331ae80611063fdae580cf9e1ebcf

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
am: 9d7bbd8045

Change-Id: Ib499fa6ea6249a942ba29d982a0b3efa00ce7bec

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

Temporarily disable 2 unratified Khronos extensions.
am: 34a327b710

Change-Id: I2f38cdac4de147423179dbe631acd7df4e2d84f3

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
am: 91cfa880a3

Change-Id: Iea0611f40d9a5fa4bf38d631a0b6565655af87d0

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

Merge "Remove references to property size limits." am: 8f8daca7ab am: 75500cc6fd
am: 624f84956b

Change-Id: Ibddf32a6fbf37988755d08eef31135749af29089

Merge "Remove references to property size limits." am: 8f8daca7ab
am: 75500cc6fd

Change-Id: Idb75e757f75f4f390a4ef596f37bbcc7c854e73e

Merge "Remove references to property size limits."
am: 8f8daca7ab

Change-Id: I146d719f51f6337abe13fc96b7e99fd7c4aebdcf

Merge "Remove references to property size limits."

Remove references to property size limits.

Bug: http://b/33926793
Test: "python systrace.py"
Change-Id: Ia11c160935b8601bc9c4d867abc57ccf37f84082

Merge "Use gralloc usage conversion library" into oc-dev
am: d787577ebf

Change-Id: I54219ad459b59ecd923814cf4ae7ea42dc6145c8

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

Temporarily skipping dump of bluetooth logs.
am: 3ae337c4a1

Change-Id: I93aa1aace4d2c23cd4de71455cb50bce4c0773ce

Temporarily skipping dump of bluetooth logs.

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

Merge changes If756ad2a,I19aeb7ac am: 15feba03ae am: 47609db709
am: e59f1b0bbe

Change-Id: Ic743e74ef2560e1657c8f73750dc747e91d336f2

Merge changes If756ad2a,I19aeb7ac am: 15feba03ae
am: 47609db709

Change-Id: I5a7ceb58874d4432d51342d275610dc46a539a61

Merge changes If756ad2a,I19aeb7ac
am: 15feba03ae

Change-Id: Ib1842ab89d245fb5bde291a26c4952e0fc262aba

Merge changes If756ad2a,I19aeb7ac

* changes:
  Add alpha channel for  the EGLConfig
  Change the conditions of layer as translucent

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