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

* commit '675427910b939e8b32545aa857e96c4066e3b525':
  rework screenshot API and implementation

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 6fbc098c: am eabe3140: Merge "Init displays to null layer stack" into jb-mr2-dev

* commit '6fbc098cfd730fb6dac3620caa720d8f4e620af0':
  Init displays to null layer stack

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 88138291: am f59d2f90: Merge changes I0e90b3f3,Ib7769bde,I4c25f34f,I1ec6400a into jb-mr2-dev

* commit '88138291611a2357fe0a2ee8cc32f17b2cd6e1be':
  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 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 54a1faa7: am 006a601f: Fix reading NULL Surfaces from Parcels

* commit '54a1faa7ca910b59b85e6afe969fde5d9a73a56f':
  Fix reading NULL Surfaces from Parcels

am 006a601f: Fix reading NULL Surfaces from Parcels

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

am a3b948e7: (-s ours) Reconcile with jb-mr2-release - do not merge

* commit 'a3b948e727a38582b88f1d3c7b604c0e3712f04f':
  Revert "Change SurfaceControl setPosition to take floats"
  SurfaceFlinger: fix a couple NULL fence checks

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

am bbb57f33: Revert "Change SurfaceControl setPosition to take floats"

* commit 'bbb57f3331c7182399ed82e9c4f93a965677dde3':
  Revert "Change SurfaceControl setPosition to take floats"

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

Merge "Revert "Change SurfaceControl setPosition to take floats""

Revert "Change SurfaceControl setPosition to take floats"

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

This reverts commit 9a867a8798fa6ea21f6341db31e38ea64fde6c83

Change-Id: I7b5dbc4db46ef3d97dc8598057d5487d6971178b

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

* commit '98ca63e6d15e47a9b62971f0db6424c13c226019':
  dumpstate: use correct header file.

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

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

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

* commit '5ec3e781c540d660a921a48575e241ae6839a26f':
  installd: 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

am eb0d1296: CpuConsumer: Add optional asynchronous mode

* commit 'eb0d12963d271052c24abb025d698504df9e7573':
  CpuConsumer: Add optional asynchronous mode

am 64d8b190: CpuConsumer: Don\'t unlock buffers on producer disconnect

* commit '64d8b1903e4b5f2838818eedcf4fef748b38709c':
  CpuConsumer: Don't unlock buffers on producer disconnect

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

am a9a1b006: Initialize DisplayData fences to NO_FENCE, not NULL

* commit 'a9a1b006e48320f5c501473e51e6c4a5f7a17b88':
  Initialize DisplayData fences to NO_FENCE, not NULL

CpuConsumer: Don't unlock buffers on producer disconnect

Bug: 8291751

Change-Id: I062a3d34b41183d07fb6b9109cdb6bf0c0c75672

am 3e010f31: Add createTJunctionFreeRegion

* commit '3e010f3138593cc6953039ee0e3db8ee31881296':
  Add createTJunctionFreeRegion

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

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

* commit '68a029edbbd162f2ca783733df37a44d5082ddee':
  GraphicBuffer.h: Remove redundant friend class declaration causing warnings

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

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

* commit '30055f1e96f5a04c6ff5fc7f7d5812fca38c2759':
  Added a test that checks the error behavior of ConsumerBase::abandon

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

* commit 'd6f549f92541d8ab947fe35e3b3c21133eed2a6f':
  add some missing parameter validation in GLES java bindings

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

am 789a6c3f: SurfaceFlinger: fix a couple NULL fence checks

* commit '789a6c3f1dfe22a1ffea7f39b2098d7842cd1f30':
  SurfaceFlinger: fix a couple NULL fence checks

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"

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

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

Change-Id: If84b082bc6bdea21673b16e5e717658c78edf405

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: Ie8fcc713188dd4819dfa55e5398e6b53ae9a14af

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"

am 45b0e435: Merge "Simplify OpenGL TLS access on ARM."

# Via Elliott Hughes (1) and Gerrit Code Review (1)
* commit '45b0e4350d3404bd194edc569e36ecca017c3d76':
  Simplify OpenGL TLS access on ARM.

Merge "Simplify OpenGL TLS access on ARM."

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

Simplify OpenGL TLS access on ARM.

bionic now assumes the TLS register is available, so OpenGL can too.

Change-Id: If2b56a4c08de9f887759b78f70022026a181dc47

Symlink /system/lib/libGLESv3.so -> libGLESv2.so

The platform has a unified GLES2/GLES3 wrapper in libGLESv2.so, which
is what bundled binaries should link against. The NDK will have
separate GLES2 and GLES3 libraries so that applications will get a
link error if they accidentally use GLES3 entry points in a GLES2
application. This symlink allows GLES3 NDK apps to load using the
unified wrapper.

Change-Id: Ic344ef63c334047fccebb55b7cb2fb23a4c2d61b

atrace: add HAL tag

Change-Id: Idfd4f2075e8cd82833ef06d4487423c62b59fe11

remove access ANDROID_VIEW_SURFACE_JNI_ID

Change-Id: I0ef474ed9be2cd26fa520ef871ffcf21a25e4994

get rid of Surface identity and token

we use the IBinder instead.

Change-Id: I4aa0b58869ba43f19980013620051e5a261b062d

Merge changes I50831d4e,I5360aebe

* changes:
  flatland: remove an unneeded #include
  libgui: disallow NULL Fence pointers

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

flatland: remove an unneeded #include

Change-Id: I50831d4efd543664ff7df7aaef35d842aebf1bd6

libgui: disallow NULL Fence pointers

This change eliminates the uses of a NULL sp<Fence> indicating that no waiting
is required.  Instead we use a non-NULL but invalid Fence object for which the
wait methods will return immediately.

Bug: 7892871
Change-Id: I5360aebe3090422ef6920d56c99fc4eedc642e48

Add ES3 support to libGLESv2 and tracing tools

Since ES3 is backwards compatible with ES2, a new wrapper isn't
necessary, and the Khronos implementation guidelines recommend
supporting both versions with the same library.

Change-Id: If9bb02be60ce01cc5fe25d1f40c4e7f37244ebf6

Import OpenGL ES 3.0 headers from Khronos SVN

Add MODULE_LICENSE_MIT: The GLES headers use an MIT license or a
slight variant of it (the "SGI Free Software License B version 2.0",
http://oss.sgi.com/projects/FreeB/).

Change-Id: Ic8f946341cdd0d579f51b6f7a3ee109cd31b1691

Merge "Implement issue #6646859: 4K!!!!  4K!!!!  4K!!!!"

Implement issue #6646859: 4K!!!!  4K!!!!  4K!!!!

Change-Id: I1bb630c58d0d41446b726de1b9d2c3875fcbbceb

Merge "App ops: new native access API."

App ops: new native access API.

This is to help implementation of bug #8181262 and maybe
bug #8181261

Note the current code has not yet been tested; it is only
known to compile at this point.

Change-Id: I489674c96d0d3fc0ddacc92611931a19a9ee5230

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

Regenerate API/trace files for constness change

Change-Id: I45836219fc0e97607d83d458de9babb44f4c7534

Import constness fix from gl3.h into gl2.h.

This change has been made in Khronos SVN for GLES2 already, however it
is mildly controversial and discussion is on-going.

This change is necessary for Android because otherwise applications
cannot include both GLES2/gl2.h and GLES3/gl3.h as the glShaderSource()
prototype mismatches. Native programs may include framework headers
(which now include GLES3 headers only) but already have included GLES2
headers, creating an incompatibility.

Change-Id: I70a4075e079ee063e76cef5a227abb0ae70aebb4

Force a specific locale for sorting

Avoids irrelevant output differences based on the user's locale. The
current output files were generated with a C or POSIX locale, and sort
differently under en_US.UTF-8.

Change-Id: I0dd85475e9ee2d58e7fa9fe2a4f86ad5c8142e00