am d5776414: make sure to call compositionComplete after taking a screenshot

* commit 'd577641411f067c93aabcd2acf7ce06862fb26d3':
  make sure to call compositionComplete after taking a screenshot

am 7589b2a7: add two libgui tests

* commit '7589b2a7975cc3bc2a0659ab6503ceb3e4d94048':
  add two libgui tests

make sure to call compositionComplete after taking a screenshot

older drivers which are doing implicit synchronization need this
or they could deadlock.

Bug: 8341885
Change-Id: Icd980a6be16071678d6151e34725b3c1c547d7ee

add two libgui tests

we check that the order in which we destroy GLConsumer wrt.
releasing the corresponding EGLSurface via eglMake(Un)Current
doesn't leak a buffer.

On at least 2 devices this test doesn't pass.

Change-Id: I63ab83951b4b0a977f38571158f948cbd9dc7cec

am 6b2b7009: Merge changes Ibb3b6ff0,I2341e20c into jb-mr2-dev

* commit '6b2b7009560964ed25cde06602872dff763b09f4':
  rename binder services main thread to Binder_*
  limit number of extra binder threads in SF to 4

Merge changes Ibb3b6ff0,I2341e20c into jb-mr2-dev

* changes:
  rename binder services main thread to Binder_*
  limit number of extra binder threads in SF to 4

am 49385ec7: Merge "When disconnecting a surface, cancel don\'t queue the buffer" into jb-mr2-dev

* commit '49385ec7c361347a69245c8a0838f6d03343a1de':
  When disconnecting a surface, cancel don't queue the buffer

Merge "When disconnecting a surface, cancel don't queue the buffer" into jb-mr2-dev

am 86306b58: Merge "Defer destroying surfaces until not current" into jb-mr2-dev

* commit '86306b587644e939c5b6d4a53d67c9b74ce24ba5':
  Defer destroying surfaces until not current

Merge "Defer destroying surfaces until not current" into jb-mr2-dev

rename binder services main thread to Binder_*

When a binder service's main thread joins the thread pool
it retains its name (whatever the exec name was), which is
very confusing in systrace.

we now rename that thread just like its friends in the
thread pool.

Change-Id: Ibb3b6ff07304b247cfc6fb1694e72350c579513e

limit number of extra binder threads in SF to 4

it should be enough and it makes debugging easier.

Bug: 6609290
Change-Id: I2341e20cfcfde184fe1ed9b925215f8fa3481ced

When disconnecting a surface, cancel don't queue the buffer

This isn't really right either, but avoids having an extra buffer that
the consumer has to drain which it might not be expecting.

To be correct, disconnecting a surface from a context should retain
the current buffer and continue using it when reconnected. The buffer
should only be canceled when the surface is destroyed. That will wait
for a later change.

Bug: 8320762
Change-Id: I5efa39c741193ca4f5612ea9de001ccbb683b345

Defer destroying surfaces until not current

Bug: 8320762
Change-Id: I1320cf87923bcc5b795a86a13193363a49e29653

am 6c337e7a: resolved conflicts for merge of bfb2aa46 to jb-mr2-dev

* commit '6c337e7a663c1049e58e0051bd7bfba61614ef37':
  Tweaks for forward compatibility

resolved conflicts for merge of bfb2aa46 to jb-mr2-dev

Merge "Tweaks for forward compatibility"

am 3cd153a0: another attempt to fix the build

* commit '3cd153a0330d4234a536fc5f4e4346413bde5a44':
  another attempt to fix the build

am 84b470d7: attempt to fix build

* commit '84b470d78f10b6fbfbf84edba749e5f7f315fc3a':
  attempt to fix build

am 1f758231: Merge changes I7f696e42,I51c6df3c into jb-mr2-dev

* commit '1f758231c26261bd6ef3ec869ce230c06ed5b596':
  Make GraphicBuffer a RefBase
  Fix RefBase debugging. O_CREAT must specify the mode.

am 2be4e8ff: workaround to fix screenshot leak on N4

* commit '2be4e8ff23f24285f4c195ba5537c7a51c13be4d':
  workaround to fix screenshot leak on N4

another attempt to fix the build

Change-Id: Ie0245db97170cca9a2cd8783b0fd512daf0d30a6

attempt to fix build

Change-Id: Ifb186d2f157ad02f593bf9b681aab53575f183ce

Merge changes I7f696e42,I51c6df3c into jb-mr2-dev

* changes:
  Make GraphicBuffer a RefBase
  Fix RefBase debugging. O_CREAT must specify the mode.

workaround to fix screenshot leak on N4

Bug: 8322020

Change-Id: Ie60af0eb431866b8d64b2674ae7bd8b5ee05f5d6

Make GraphicBuffer a RefBase

- this gives us access to RefBase's refcounting debugging
- it doesn't cost much because GraphicBuffer already has a vtable

Change-Id: I7f696e421fea14b14bfaeb83880689b83e96af4d

Fix RefBase debugging. O_CREAT must specify the mode.

Change-Id: I51c6df3cfd59b20ca73c3edee86bc2f74dbde1b1

am 438ca07b: Merge changes I66511c08,Ia051949f,Ic7451365,I5b571a4c into jb-mr2-dev

* commit '438ca07b6ba74235e87bfbd78c94874d8bbde391':
  Get rid of LayerBase.
  Make LayerDim a regular Layer instead of a LayerBase
  fold LayerBaseClient into LayerBase
  Remove support for ScreenshotLayer

Merge changes I66511c08,Ia051949f,Ic7451365,I5b571a4c into jb-mr2-dev

* changes:
  Get rid of LayerBase.
  Make LayerDim a regular Layer instead of a LayerBase
  fold LayerBaseClient into LayerBase
  Remove support for ScreenshotLayer

Tweaks for forward compatibility

Change-Id: I136fcbbc14072ee5f5281d09445f28d083ed3ce1

am 175264b0: Return NO_MEMORY when glReadPixels() fails.

* commit '175264b09c6080b29a23fc9f545d4b99445714bd':
  Return NO_MEMORY when glReadPixels() fails.

Get rid of LayerBase.

The functionality of LayerBase and Layer is folded
into Layer. There wasn't a need for this abstraction
anymore.

Change-Id: I66511c08cc3d89009ba4deabf47e26cd4cfeaefb

Make LayerDim a regular Layer instead of a LayerBase

this is in preparation to get rid of LayerBase entirely

Change-Id: Ia051949fc5205fd87371331145356ee11598a597

fold LayerBaseClient into LayerBase

Change-Id: Ic745136522df59c42f0885fd969e75ea55d09f01

Remove support for ScreenshotLayer

Change-Id: I5b571a4cf3faa77d2c4aca916fa4bd00a1065bb9

Return NO_MEMORY when glReadPixels() fails.

Change-Id: Ic66134ef457e8442ae9135e9ec50e3d02932a253

am f082323c: Merge "CpuConsumer_test: Extend test with new formats RGBA8888 and optional Y8/Y16" into jb-mr2-dev

* commit 'f082323c0dbd915d2ad7b6f4d9c732e230fcda46':
  CpuConsumer_test: Extend test with new formats RGBA8888 and optional Y8/Y16

Merge "CpuConsumer_test: Extend test with new formats RGBA8888 and optional Y8/Y16" into jb-mr2-dev

CpuConsumer_test: Extend test with new formats RGBA8888 and optional Y8/Y16

Change-Id: I0a0f6ce73516883bf50c749a1759abbdb3748e08

am 7284145d: Add eglPresentationTimeANDROID

* commit '7284145d564fa8a422a8e564a38c730fb4a2962b':
  Add eglPresentationTimeANDROID

Add eglPresentationTimeANDROID

Added EGL extension to set a timestamp on a surface.

Also, fix JNI encoding of "long" in glgen.

Bug 8191230

Change-Id: I38b7334bade3f8ff02bffe600bb74469ef22c164

Reconcile with jb-mr2-release - do not merge

Change-Id: I7bb6b77ca42a715a6eb69081512c43a43158e1e3

merge in jb-mr2-release history after reset to jb-mr2-dev

am 306f18c5: Merge "rework screenshot API and implementation" into jb-mr2-dev

* commit '306f18c5fb15ac05db09ece7241af02b9713a23d':
  rework screenshot API and implementation

Merge "rework screenshot API and implementation" into jb-mr2-dev

am eabe3140: Merge "Init displays to null layer stack" into jb-mr2-dev

* commit 'eabe3140f11e515639e7a70a1286dd6af7352c9e':
  Init displays to null layer stack

Merge "Init displays to null layer stack" into jb-mr2-dev

am f59d2f90: Merge changes I0e90b3f3,Ib7769bde,I4c25f34f,I1ec6400a into jb-mr2-dev

* commit 'f59d2f9070621e8f36b2ded3ec1d07bd3aec6150':
  implement display projection clipping in h/w composer
  refactor the crop region for hwc is calculated/set
  apply the projection's viewport to the visibleregion passed to hwc
  set correct crop rectangle in LayerBase::setCrop

Merge changes I0e90b3f3,Ib7769bde,I4c25f34f,I1ec6400a into jb-mr2-dev

* changes:
  implement display projection clipping in h/w composer
  refactor the crop region for hwc is calculated/set
  apply the projection's viewport to the visibleregion passed to hwc
  set correct crop rectangle in LayerBase::setCrop

am 006a601f: Fix reading NULL Surfaces from Parcels

* commit '006a601f22b275bcabf608f630b321df027d56d1':
  Fix reading NULL Surfaces from Parcels

Reconcile with jb-mr2-release - do not merge

Change-Id: I8176eb9302e58477ef1bd9b394a974d85fa67792

merge in jb-mr2-release history after reset to jb-mr2-dev

Fix reading NULL Surfaces from Parcels

Writing a NULL Surface was being read as a non-NULL Surface with NULL
mGraphicBufferProducer. Before the SurfaceTextureClient -> Surface
refactoring, you'd get a NULL Surface, and some code relies on that.

Bug: 8291161
Change-Id: I477bfe8882693e53a5f604a3d2c9e3cfe24473b4

rework screenshot API and implementation

- SurfaceFlinger now supports to take a screenshot
  directly into an IGraphicBufferProducer

- reimplement the IMemoryHeap screenshot on top
  of the above

- reimplement LayerScreenshot such that its
  BufferQueue is directly used as the destination
  of the screenshot. LayerScreenshot is now a thin
  wrapper around Layer

Bug: 6940974

Change-Id: I69a2096b44b91acbb99eba16f83a9c78d94e0d10

Revert "Change SurfaceControl setPosition to take floats"

Temporary, to fix weekend build, until we get Nvidia code drop.

This reverts commit 9a867a8798fa6ea21f6341db31e38ea64fde6c83

DO NOT MERGE

Change-Id: I7b5dbc4db46ef3d97dc8598057d5487d6971178b

Revert "Change SurfaceControl setPosition to take floats"

Temporary, to fix weekend build, until we get Nvidia code drop.

This reverts commit 9a867a8798fa6ea21f6341db31e38ea64fde6c83

DO NOT MERGE

Change-Id: I7b5dbc4db46ef3d97dc8598057d5487d6971178b

am 22641851: Merge "dumpstate: use correct header file."

* commit '22641851c48fee6e79d01adcfab01c0b625cfb3c':
  dumpstate: use correct header file.

Merge "installd: use correct header file." into jb-mr2-dev

Merge "dumpstate: use correct header file."

Init displays to null layer stack

When a display is added, initialize it to use an empty layer stack, so
if it is somehow visible it will show black. It will be assigned the
real layer stack -- along with a projection and other properties -- by
window manager soon. Normally a display remains blanked until window
manager has decided what to show on it, but for HDMI connected at boot
that isn't currently the case.

Bug: 7258935
Change-Id: Ic9bb25f7a9b8d9d3772b097ab1d6fa03bc8780a1

implement display projection clipping in h/w composer

- cropping to the projection's "viewport" is "simply"
  accomplished by intersecting it with the window crop
  expressed in layerstack space.

Bug: 7149437
Change-Id: I0e90b3f37945292314b5d78a8f134935967e8053

refactor the crop region for hwc is calculated/set

- the crop region is now always calculated and set
  in LayerBase::setGeometry which uses new virtuals to
  access the "content" crop and transform (which are
  provided by the Layer subclass)

Change-Id: Ib7769bdec0917dd248f926600c14ddf9ea84897a

apply the projection's viewport to the visibleregion passed to hwc

each desplay's projection's viewport essentially clips each layer,
so this should be reflected in the visibleregion passed to
h/w composer.

DisplayDevice getViewport and getFrame are now guaranteed to return
valid Rects.

Change-Id: I4c25f34fb26af10179eb26d429ca6c384c671e91

set correct crop rectangle in LayerBase::setCrop

The crop always had left=top=0, because the crop position
and the layer's transform were merged together in
computeBounds() (which really used to compute the
bounds in screen space, which we usually call the
"frame" elsewhere in the code)

Note: in practice this crop value is not used by
hwc, because it's overridden in Layer::setGeometry(), which
is why this bug was never apparent.

Change-Id: I1ec6400a8fc8314408e4252708f43ea98c2fe64e

installd: use correct header file.

This is a port of https://android-review.googlesource.com/52540 ,
which doesn't apply cleanly to master.

Change-Id: I2824bb7afd4e343b5dc8bed9f24002ec1dcbf07f

dumpstate: use correct header file.

Change-Id: I006d7a219ce6309259f271beff6011cc266f1a61

CpuConsumer: Add optional asynchronous mode

Bug: 8290146
Bug: 8291751

Change-Id: I9c8ac4bff38b0411e987a204e540d018dba6d0b4

CpuConsumer: Don't unlock buffers on producer disconnect

Bug: 8291751

Change-Id: I062a3d34b41183d07fb6b9109cdb6bf0c0c75672

merge in jb-mr2-release history after reset to jb-mr2-dev

Initialize DisplayData fences to NO_FENCE, not NULL

Also fix another place that was checking for NULL fence rather than
Fence::isValid().

Bug 8283950

Change-Id: Ie06db327eb416828d8dac139171d96d4470b2e35

Add createTJunctionFreeRegion

T-junction free regions are useful for rendering regions with various
geometric transformations, and the Region's span-ordered, sorted rect
list supports T-junction free storage without modification.

This approach creates a T-junction free region by splitting each
rectangle that is part of a vertical T-junction. This approach is two
pass (up and down) so that divisions can trickle up/down to other
adjacent spans.

Change-Id: Ifcf5e6fe0034c96b00ef09a4433b2b0fce8f4300

merge in jb-mr2-release history after reset to jb-mr2-dev

Merge "GraphicBuffer.h: Remove redundant friend class declaration causing warnings" into jb-mr2-dev

GraphicBuffer.h: Remove redundant friend class declaration causing warnings

Change-Id: I8a4b0e0b4d0a1cf472de820553c31f2d42044543

merge in jb-mr2-release history after reset to jb-mr2-dev

SurfaceFlinger: fix a couple NULL fence checks

This change replaces checks for a NULL fence pointer with calls to
Fence::isValid.  There should no longer be NULL fences.

Change-Id: If17c9c132fcb1801531bf7588f8ba53476c57dad

Merge "Added a test that checks the error behavior of ConsumerBase::abandon" into jb-mr2-dev

Merge "add some missing parameter validation in GLES java bindings" into jb-mr2-dev

Added a test that checks the error behavior of ConsumerBase::abandon

We check that calling eglSwapBuffers() on an abandonned BufferQueue
return EGL_BAD_SURFACE -- this is to ensure consistancy between
drivers.

Change-Id: Ibb548e0cf767ceee69f2fc4a85811d15a6522277

add some missing parameter validation in GLES java bindings

- added support for comments in checks.spec
- added most missing checks
- added and commented with // special-cased functions
- added and commented with # functions that are still missing validation checks
- moved glGet* to a special case and updated all the "pnames" from the khronos spec
- changed ifcheck to default to 1 value. this allows us to simplify the checks.spec file
  and handle unknown pnames automatically (they'll be validated against 1 value, if
  it happens to need more, the call will go through but the validation will not
  happen).
- refactored the cpp headers in to a common header + GLES version specific
  header

Bug: 7402895
Change-Id: Ib5c68ca0ca416407b4cfa36e3a21901b2d6263ab

SurfaceFlinger: fix a couple NULL fence checks

This change replaces checks for a NULL fence pointer with calls to
Fence::isValid.  There should no longer be NULL fences.

Change-Id: If17c9c132fcb1801531bf7588f8ba53476c57dad

Merge changes Ic344ef63,If9bb02be,Ic8f94634

* changes:
  Symlink /system/lib/libGLESv3.so -> libGLESv2.so
  Add ES3 support to libGLESv2 and tracing tools
  Import OpenGL ES 3.0 headers from Khronos SVN

Merge changes I45836219,I70a4075e

* changes:
  Regenerate API/trace files for constness change
  Import constness fix from gl3.h into gl2.h.

Merge "sort the check file in alphabetical order"

Merge "update glgen to take into account recent changes"

sort the check file in alphabetical order

this simplifies a lot comparing it to the spec files.

Change-Id: I4b9b5e7469338a626100472d360ea26a902017bc

update glgen to take into account recent changes

- it looks like UserId has been renamed to UserHandle in the
framework.
- also for some reason I don't understand glgen didn't seem
to be up to date with respect to the generated bindings in the tree.
It's like the bindings were generated with a more recent version of
glgen (maybe it was never checked in). So we fix that here.

Change-Id: Ie49522ebf67fcab9213246b4d93500e37a3cbc05

Remove obsolte 'su' from 'ping' call (to work on user build)

Change-Id: I2d391bbd39fc591eef33b9c9e4b4cdbda29e41e7
Signed-off-by: Dmitry Shmidt <dimitrysh@google.com>

Change SurfaceControl setPosition to take floats

Bug: 8153162
Change-Id: If3c4b7a3e124ace25999eb9a569dc9a806a1fbc2

Merge "get rid of Surface::getISurfaceTexture()"

Merge remote-tracking branch 'goog/jb-mr2-release'

get rid of Surface::getISurfaceTexture()

this was there just for legacy reasons.

Change-Id: I0a48d6e04d7efa65033f9f10c91bba557f16ae8b

Fixes to native app ops services

Bug: 8181262
Change-Id: I3d3f20453f6c6f2756c988363c5b5453ef309cb6

merge in jb-mr2-release history after reset to master

fix build: SurfaceTextureClient to Surface rename

Change-Id: I0041e6f49bc637bc52161c83f0bd44101246f480

fix build: SurfaceTextureClient to Surface rename

Change-Id: I54d68241f1080eef640aefe71f33912ad79345ef

fix build: SurfaceTextureClient to Surface rename

Change-Id: Ibcf98337940f9195bcff7b5e8bddb6e912e65163

Refactoring: Rename SurfaceTextureClient to Surface

Change-Id: Ibed34175ae273608393aaa5f0a7df207dc40d709

Merge "remove access ANDROID_VIEW_SURFACE_JNI_ID"

Merge "get rid of Surface identity and token"