diff options
| author | The Android Open Source Project <initial-contribution@android.com> | 2010-05-14 15:15:36 -0700 |
|---|---|---|
| committer | Android Git Automerger <android-git-automerger@android.com> | 2010-05-14 15:15:36 -0700 |
| commit | aa8da3dc405dca80c6aff71a7c17f883edea2814 (patch) | |
| tree | 6756be0a6d844ab2fd8afb3c8427012165f41896 | |
| parent | 3e4fd9ea19c16050ef78ea55138a114798033f65 (diff) | |
| parent | a8e0f75b7fe99dd6672bfc958734f4558905c3aa (diff) | |
| download | frameworks_base-aa8da3dc405dca80c6aff71a7c17f883edea2814.zip frameworks_base-aa8da3dc405dca80c6aff71a7c17f883edea2814.tar.gz frameworks_base-aa8da3dc405dca80c6aff71a7c17f883edea2814.tar.bz2 | |
am a8e0f75b: am 7450c02a: am 900b6157: merge from open-source master
| -rw-r--r-- | libs/surfaceflinger_client/SurfaceComposerClient.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/libs/surfaceflinger_client/SurfaceComposerClient.cpp b/libs/surfaceflinger_client/SurfaceComposerClient.cpp index 3117495..85167da 100644 --- a/libs/surfaceflinger_client/SurfaceComposerClient.cpp +++ b/libs/surfaceflinger_client/SurfaceComposerClient.cpp @@ -58,7 +58,7 @@ namespace android { // Must not be holding SurfaceComposerClient::mLock when acquiring gLock here. static Mutex gLock; static sp<ISurfaceComposer> gSurfaceManager; -static DefaultKeyedVector< sp<IBinder>, sp<SurfaceComposerClient> > gActiveConnections; +static DefaultKeyedVector< sp<IBinder>, wp<SurfaceComposerClient> > gActiveConnections; static SortedVector<sp<SurfaceComposerClient> > gOpenTransactions; static sp<IMemoryHeap> gServerCblkMemory; static volatile surface_flinger_cblk_t* gServerCblk; @@ -195,7 +195,7 @@ SurfaceComposerClient::clientForConnection(const sp<IBinder>& conn) { // scope for lock Mutex::Autolock _l(gLock); - client = gActiveConnections.valueFor(conn); + client = gActiveConnections.valueFor(conn).promote(); } if (client == 0) { @@ -383,8 +383,8 @@ void SurfaceComposerClient::openGlobalTransaction() const size_t N = gActiveConnections.size(); VERBOSE("openGlobalTransaction (%ld clients)", N); for (size_t i=0; i<N; i++) { - sp<SurfaceComposerClient> client(gActiveConnections.valueAt(i)); - if (gOpenTransactions.indexOf(client) < 0) { + sp<SurfaceComposerClient> client(gActiveConnections.valueAt(i).promote()); + if (client != 0 && gOpenTransactions.indexOf(client) < 0) { if (client->openTransaction() == NO_ERROR) { if (gOpenTransactions.add(client) < 0) { // Ooops! |