SurfaceFlinger: add a crop to the layer state
This change adds a crop rectangle specified in window coordinates to the layer
state. The all window pixels outside this crop rectangle are treated as though
they were fully transparent. This change also adds the plumbing necessary for
WindowManager to set that crop.
Change-Id: I582bc445dc8c97d4c943d4db8d582a6ef5a66081
diff --git a/services/surfaceflinger/Layer.cpp b/services/surfaceflinger/Layer.cpp
index 6bc80ee..2899db7 100644
--- a/services/surfaceflinger/Layer.cpp
+++ b/services/surfaceflinger/Layer.cpp
@@ -141,47 +141,6 @@
void Layer::validateVisibility(const Transform& globalTransform) {
LayerBase::validateVisibility(globalTransform);
- if (mCurrentScalingMode == NATIVE_WINDOW_SCALING_MODE_FREEZE &&
- !mCurrentCrop.isEmpty()) {
- // We need to shrink the window size to match the buffer crop
- // rectangle.
- const Layer::State& s(drawingState());
- const Transform tr(globalTransform * s.transform);
- float windowWidth = s.w;
- float windowHeight = s.h;
- float bufferWidth = mActiveBuffer->getWidth();
- float bufferHeight = mActiveBuffer->getHeight();
- if (mCurrentTransform & NATIVE_WINDOW_TRANSFORM_ROT_90) {
- float tmp = bufferWidth;
- bufferWidth = bufferHeight;
- bufferHeight = tmp;
- }
- float xScale = float(windowWidth) / float(bufferWidth);
- float yScale = float(windowHeight) / float(bufferHeight);
-
- // Compute the crop in post-transform coordinates.
- Rect crop(mCurrentCrop.transform(mCurrentTransform,
- mActiveBuffer->getWidth(), mActiveBuffer->getHeight()));
-
- float left = ceil(xScale * float(crop.left));
- float right = floor(xScale * float(crop.right));
- float top = ceil(yScale * float(crop.top));
- float bottom = floor(yScale * float(crop.bottom));
-
- tr.transform(mVertices[0], left, top);
- tr.transform(mVertices[1], left, bottom);
- tr.transform(mVertices[2], right, bottom);
- tr.transform(mVertices[3], right, top);
-
- const DisplayHardware& hw(graphicPlane(0).displayHardware());
- const uint32_t hw_h = hw.getHeight();
- for (size_t i=0 ; i<4 ; i++)
- mVertices[i][1] = hw_h - mVertices[i][1];
-
- mTransformedBounds = tr.transform(
- Rect(int(left), int(top), int(right), int(bottom)));
- }
-
// This optimization allows the SurfaceTexture to bake in
// the rotation so hardware overlays can be used
mSurfaceTexture->setTransformHint(getTransformHint());
@@ -259,6 +218,46 @@
return NO_ERROR;
}
+Rect Layer::computeBufferCrop() const {
+ // Start with the SurfaceTexture's buffer crop...
+ Rect crop;
+ if (!mCurrentCrop.isEmpty()) {
+ crop = mCurrentCrop;
+ } else if (mActiveBuffer != NULL){
+ crop = Rect(mActiveBuffer->getWidth(), mActiveBuffer->getHeight());
+ } else {
+ crop = Rect(mTransformedBounds.width(), mTransformedBounds.height());
+ }
+
+ // ... then reduce that in the same proportions as the window crop reduces
+ // the window size.
+ const State& s(drawingState());
+ if (!s.crop.isEmpty()) {
+ // Transform the window crop to match the buffer coordinate system,
+ // which means using the inverse of the current transform set on the
+ // SurfaceTexture.
+ uint32_t invTransform = mCurrentTransform;
+ int winWidth = s.w;
+ int winHeight = s.h;
+ if (invTransform & NATIVE_WINDOW_TRANSFORM_ROT_90) {
+ invTransform ^= NATIVE_WINDOW_TRANSFORM_FLIP_V |
+ NATIVE_WINDOW_TRANSFORM_FLIP_H;
+ winWidth = s.h;
+ winHeight = s.w;
+ }
+ Rect winCrop = s.crop.transform(invTransform, s.w, s.h);
+
+ float xScale = float(crop.width()) / float(winWidth);
+ float yScale = float(crop.height()) / float(winHeight);
+ crop.left += int(ceil(float(winCrop.left) * xScale));
+ crop.top += int(ceil(float(winCrop.top) * yScale));
+ crop.right -= int(ceil(float(winWidth - winCrop.right) * xScale));
+ crop.bottom -= int(ceil(float(winHeight - winCrop.bottom) * yScale));
+ }
+
+ return crop;
+}
+
void Layer::setGeometry(hwc_layer_t* hwcl)
{
LayerBaseClient::setGeometry(hwcl);
@@ -293,23 +292,11 @@
hwcl->transform = finalTransform;
}
- if (isCropped()) {
- hwcl->sourceCrop.left = mCurrentCrop.left;
- hwcl->sourceCrop.top = mCurrentCrop.top;
- hwcl->sourceCrop.right = mCurrentCrop.right;
- hwcl->sourceCrop.bottom = mCurrentCrop.bottom;
- } else {
- const sp<GraphicBuffer>& buffer(mActiveBuffer);
- hwcl->sourceCrop.left = 0;
- hwcl->sourceCrop.top = 0;
- if (buffer != NULL) {
- hwcl->sourceCrop.right = buffer->width;
- hwcl->sourceCrop.bottom = buffer->height;
- } else {
- hwcl->sourceCrop.right = mTransformedBounds.width();
- hwcl->sourceCrop.bottom = mTransformedBounds.height();
- }
- }
+ Rect crop = computeBufferCrop();
+ hwcl->sourceCrop.left = crop.left;
+ hwcl->sourceCrop.top = crop.top;
+ hwcl->sourceCrop.right = crop.right;
+ hwcl->sourceCrop.bottom = crop.bottom;
}
void Layer::setPerFrameData(hwc_layer_t* hwcl) {
diff --git a/services/surfaceflinger/Layer.h b/services/surfaceflinger/Layer.h
index 1188621..393599f 100644
--- a/services/surfaceflinger/Layer.h
+++ b/services/surfaceflinger/Layer.h
@@ -101,6 +101,7 @@
uint32_t getEffectiveUsage(uint32_t usage) const;
uint32_t getTransformHint() const;
bool isCropped() const;
+ Rect computeBufferCrop() const;
static bool getOpacityForFormat(uint32_t format);
// -----------------------------------------------------------------------
diff --git a/services/surfaceflinger/LayerBase.cpp b/services/surfaceflinger/LayerBase.cpp
index 694ecde..81031b1 100644
--- a/services/surfaceflinger/LayerBase.cpp
+++ b/services/surfaceflinger/LayerBase.cpp
@@ -93,6 +93,7 @@
mCurrentState.flags = layerFlags;
mCurrentState.sequence = 0;
mCurrentState.transform.set(0, 0);
+ mCurrentState.crop.makeInvalid();
// drawing state & current state are identical
mDrawingState = mCurrentState;
@@ -172,6 +173,14 @@
requestTransaction();
return true;
}
+bool LayerBase::setCrop(const Rect& crop) {
+ if (mCurrentState.crop == crop)
+ return false;
+ mCurrentState.sequence++;
+ mCurrentState.crop = crop;
+ requestTransaction();
+ return true;
+}
Rect LayerBase::visibleBounds() const
{
@@ -229,15 +238,18 @@
const bool transformed = tr.transformed();
const DisplayHardware& hw(graphicPlane(0).displayHardware());
const uint32_t hw_h = hw.getHeight();
+ const Rect& crop(s.crop);
- uint32_t w = s.w;
- uint32_t h = s.h;
+ Rect win(s.w, s.h);
+ if (!crop.isEmpty()) {
+ win.intersect(crop, &win);
+ }
mNumVertices = 4;
- tr.transform(mVertices[0], 0, 0);
- tr.transform(mVertices[1], 0, h);
- tr.transform(mVertices[2], w, h);
- tr.transform(mVertices[3], w, 0);
+ tr.transform(mVertices[0], win.left, win.top);
+ tr.transform(mVertices[1], win.left, win.bottom);
+ tr.transform(mVertices[2], win.right, win.bottom);
+ tr.transform(mVertices[3], win.right, win.top);
for (size_t i=0 ; i<4 ; i++)
mVertices[i][1] = hw_h - mVertices[i][1];
@@ -260,7 +272,7 @@
mOrientation = tr.getOrientation();
mPlaneOrientation = planeTransform.getOrientation();
mTransform = tr;
- mTransformedBounds = tr.makeBounds(w, h);
+ mTransformedBounds = tr.transform(win);
}
void LayerBase::lockPageFlip(bool& recomputeVisibleRegions) {
@@ -391,15 +403,27 @@
GLfloat v;
};
+ Rect crop(s.w, s.h);
+ if (!s.crop.isEmpty()) {
+ crop = s.crop;
+ }
+ GLfloat left = GLfloat(crop.left) / GLfloat(s.w);
+ GLfloat top = GLfloat(crop.top) / GLfloat(s.h);
+ GLfloat right = GLfloat(crop.right) / GLfloat(s.w);
+ GLfloat bottom = GLfloat(crop.bottom) / GLfloat(s.h);
+
TexCoords texCoords[4];
- texCoords[0].u = 0;
- texCoords[0].v = 1;
- texCoords[1].u = 0;
- texCoords[1].v = 0;
- texCoords[2].u = 1;
- texCoords[2].v = 0;
- texCoords[3].u = 1;
- texCoords[3].v = 1;
+ texCoords[0].u = left;
+ texCoords[0].v = top;
+ texCoords[1].u = left;
+ texCoords[1].v = bottom;
+ texCoords[2].u = right;
+ texCoords[2].v = bottom;
+ texCoords[3].u = right;
+ texCoords[3].v = top;
+ for (int i = 0; i < 4; i++) {
+ texCoords[i].v = 1.0f - texCoords[i].v;
+ }
glEnableClientState(GL_TEXTURE_COORD_ARRAY);
glVertexPointer(2, GL_FLOAT, 0, mVertices);
diff --git a/services/surfaceflinger/LayerBase.h b/services/surfaceflinger/LayerBase.h
index d123d9b..31f6dfd 100644
--- a/services/surfaceflinger/LayerBase.h
+++ b/services/surfaceflinger/LayerBase.h
@@ -78,6 +78,7 @@
uint32_t tint;
Transform transform;
Region transparentRegion;
+ Rect crop;
};
virtual void setName(const String8& name);
@@ -91,6 +92,7 @@
bool setMatrix(const layer_state_t::matrix22_t& matrix);
bool setTransparentRegionHint(const Region& opaque);
bool setFlags(uint8_t flags, uint8_t mask);
+ bool setCrop(const Rect& crop);
void commitTransaction();
bool requestTransaction();
diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 0d4d2b9..35a7fd5 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -1352,6 +1352,10 @@
if (layer->setFlags(s.flags, s.mask))
flags |= eTraversalNeeded;
}
+ if (what & eCropChanged) {
+ if (layer->setCrop(s.crop))
+ flags |= eTraversalNeeded;
+ }
}
return flags;
}