diff options
author | Rebecca Schultz Zavin <rebecca@android.com> | 2009-07-21 16:17:59 -0700 |
---|---|---|
committer | Rebecca Schultz Zavin <rebecca@android.com> | 2009-07-22 17:17:46 -0700 |
commit | 1000170033ec2fd7427cf72c81432384ad8a5286 (patch) | |
tree | 217f9b6edf452295c69a60c3d758a404ec825f8e /libs/surfaceflinger/LayerBuffer.cpp | |
parent | 96df49bb7deaa33168040084021ff142963a086b (diff) | |
download | frameworks_native-1000170033ec2fd7427cf72c81432384ad8a5286.zip frameworks_native-1000170033ec2fd7427cf72c81432384ad8a5286.tar.gz frameworks_native-1000170033ec2fd7427cf72c81432384ad8a5286.tar.bz2 |
Add a flag to set whether the overlay has been initialized. Commit needs to be called at least once on each overlay, and it appears that sometimes this
doesn't happen because the visibility never changes. With this change
the overlay parameter and position will be committed when either the visibility
of the window changes, or on the first call to visibility resolved, if it
hasn't already been done.
Signed-off-by: Rebecca Schultz Zavin <rebecca@android.com>
Diffstat (limited to 'libs/surfaceflinger/LayerBuffer.cpp')
-rw-r--r-- | libs/surfaceflinger/LayerBuffer.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/libs/surfaceflinger/LayerBuffer.cpp b/libs/surfaceflinger/LayerBuffer.cpp index 2d949a0..90e7f50 100644 --- a/libs/surfaceflinger/LayerBuffer.cpp +++ b/libs/surfaceflinger/LayerBuffer.cpp @@ -575,6 +575,7 @@ LayerBuffer::OverlaySource::OverlaySource(LayerBuffer& layer, mFormat = overlay->format; mWidthStride = overlay->w_stride; mHeightStride = overlay->h_stride; + mInitialized = false; mOverlayHandle = overlay->getHandleRef(overlay); @@ -614,8 +615,9 @@ void LayerBuffer::OverlaySource::onVisibilityResolved( // this code-path must be as tight as possible, it's called each time // the screen is composited. if (UNLIKELY(mOverlay != 0)) { - if (mVisibilityChanged) { + if (mVisibilityChanged || !mInitialized) { mVisibilityChanged = false; + mInitialized = true; const Rect& bounds = mLayer.getTransformedBounds(); int x = bounds.left; int y = bounds.top; @@ -627,7 +629,7 @@ void LayerBuffer::OverlaySource::onVisibilityResolved( if (mOverlay) { overlay_control_device_t* overlay_dev = mOverlayDevice; overlay_dev->setPosition(overlay_dev, mOverlay, x,y,w,h); - overlay_dev->setParameter(overlay_dev, mOverlay, + overlay_dev->setParameter(overlay_dev, mOverlay, OVERLAY_TRANSFORM, mLayer.getOrientation()); overlay_dev->commit(overlay_dev, mOverlay); } |