diff options
author | Mathias Agopian <mathias@google.com> | 2010-05-25 17:51:34 -0700 |
---|---|---|
committer | Mathias Agopian <mathias@google.com> | 2010-05-25 17:51:34 -0700 |
commit | 2ce19af45bf4c5e311a73df474ffe88d96d118c5 (patch) | |
tree | 3df75d3e86308a618a3ddc47d301731af30769a8 /include/surfaceflinger | |
parent | d6ddcb7f00a7af95b452233d965b922632f78f21 (diff) | |
download | frameworks_base-2ce19af45bf4c5e311a73df474ffe88d96d118c5.zip frameworks_base-2ce19af45bf4c5e311a73df474ffe88d96d118c5.tar.gz frameworks_base-2ce19af45bf4c5e311a73df474ffe88d96d118c5.tar.bz2 |
fix [2712278] The preview buffer left some black borders in left and bottom edges
we were incorrectly flagging push_buffer surfaces as invalid
Change-Id: I4dfd4ffbbe8a71f7e23e835db8d71966416c29bb
Diffstat (limited to 'include/surfaceflinger')
-rw-r--r-- | include/surfaceflinger/Surface.h | 3 | ||||
-rw-r--r-- | include/surfaceflinger/SurfaceComposerClient.h | 23 |
2 files changed, 12 insertions, 14 deletions
diff --git a/include/surfaceflinger/Surface.h b/include/surfaceflinger/Surface.h index 77e4a61..e561fb9 100644 --- a/include/surfaceflinger/Surface.h +++ b/include/surfaceflinger/Surface.h @@ -228,7 +228,7 @@ private: */ void init(); status_t validate() const; - sp<SurfaceComposerClient> getClient() const; + status_t initCheck() const; sp<ISurface> getISurface() const; inline const GraphicBufferMapper& getBufferMapper() const { return mBufferMapper; } @@ -271,6 +271,7 @@ private: uint32_t mFlags; GraphicBufferMapper& mBufferMapper; SharedBufferClient* mSharedBufferClient; + status_t mInitCheck; // protected by mSurfaceLock Rect mSwapRectangle; diff --git a/include/surfaceflinger/SurfaceComposerClient.h b/include/surfaceflinger/SurfaceComposerClient.h index 9d0f0cb..8396038 100644 --- a/include/surfaceflinger/SurfaceComposerClient.h +++ b/include/surfaceflinger/SurfaceComposerClient.h @@ -123,13 +123,6 @@ public: status_t linkToComposerDeath(const sp<IBinder::DeathRecipient>& recipient, void* cookie = NULL, uint32_t flags = 0); -private: - friend class Surface; - friend class SurfaceControl; - - SurfaceComposerClient(const sp<ISurfaceComposer>& sm, - const sp<IBinder>& conn); - status_t hide(SurfaceID id); status_t show(SurfaceID id, int32_t layer = -1); status_t freeze(SurfaceID id); @@ -142,17 +135,21 @@ private: status_t setMatrix(SurfaceID id, float dsdx, float dtdx, float dsdy, float dtdy); status_t setPosition(SurfaceID id, int32_t x, int32_t y); status_t setSize(SurfaceID id, uint32_t w, uint32_t h); - void signalServer(); - status_t destroySurface(SurfaceID sid); - void _init(const sp<ISurfaceComposer>& sm, + SharedClient* getSharedClient() const; + +private: + SurfaceComposerClient(const sp<ISurfaceComposer>& sm, + const sp<IBinder>& conn); + + void init(const sp<ISurfaceComposer>& sm, const sp<ISurfaceFlingerClient>& conn); - inline layer_state_t* _get_state_l(SurfaceID id); - layer_state_t* _lockLayerState(SurfaceID id); - inline void _unlockLayerState(); + inline layer_state_t* get_state_l(SurfaceID id); + layer_state_t* lockLayerState(SurfaceID id); + inline void unlockLayerState(); mutable Mutex mLock; layer_state_t* mPrebuiltLayerState; |