Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "ALooperRoster no longer holds strong references to handlers and loopers." into gingerbread

Browse Source
This commit is contained in:
Andreas Huber
2010-07-07 09:41:05 -07:00
committed by Android (Google) Code Review
parent 6615defd4b 57576c1cae
commit 2e6aff8e19
2 changed files with 40 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
include/media/stagefright/foundation/ALooperRoster.h
View File

@@ -38,8 +38,8 @@ struct ALooperRoster {
private:
struct HandlerInfo {
sp<ALooper> mLooper;
sp<AHandler> mHandler;
wp<ALooper> mLooper;
wp<AHandler> mHandler;
};
Mutex mLock;

44
media/libstagefright/foundation/ALooperRoster.cpp
View File

@@ -54,10 +54,15 @@ void ALooperRoster::unregisterHandler(ALooper::handler_id handlerID) {
Mutex::Autolock autoLock(mLock);
ssize_t index = mHandlers.indexOfKey(handlerID);
CHECK(index >= 0);
CHECK_GE(index, 0);
const HandlerInfo &info = mHandlers.valueAt(index);
info.mHandler->setID(0);
sp<AHandler> handler = info.mHandler.promote();
if (handler != NULL) {
handler->setID(0);
}
mHandlers.removeItemsAt(index);
}
@@ -74,7 +79,18 @@ void ALooperRoster::postMessage(
}
const HandlerInfo &info = mHandlers.valueAt(index);
info.mLooper->post(msg, delayUs);
sp<ALooper> looper = info.mLooper.promote();
if (looper == NULL) {
LOG(WARNING) << "failed to post message. "
"Target handler still registered, but object gone.";
mHandlers.removeItemsAt(index);
return;
}
looper->post(msg, delayUs);
}
void ALooperRoster::deliverMessage(const sp<AMessage> &msg) {
@@ -86,12 +102,21 @@ void ALooperRoster::deliverMessage(const sp<AMessage> &msg) {
ssize_t index = mHandlers.indexOfKey(msg->target());
if (index < 0) {
LOG(WARNING) << "failed to deliver message. Target handler not registered.";
LOG(WARNING) << "failed to deliver message. "
<< "Target handler not registered.";
return;
}
const HandlerInfo &info = mHandlers.valueAt(index);
handler = info.mHandler;
handler = info.mHandler.promote();
if (handler == NULL) {
LOG(WARNING) << "failed to deliver message. "
"Target handler registered, but object gone.";
mHandlers.removeItemsAt(index);
return;
}
}
handler->onMessageReceived(msg);
@@ -106,7 +131,14 @@ sp<ALooper> ALooperRoster::findLooper(ALooper::handler_id handlerID) {
return NULL;
}
return mHandlers.valueAt(index).mLooper;
sp<ALooper> looper = mHandlers.valueAt(index).mLooper.promote();
if (looper == NULL) {
mHandlers.removeItemsAt(index);
return NULL;
}
return looper;
}
} // namespace android