Merge "ACodec: defer setSurface at OutputPortSettingsChangedState" (a16b76a3) · Commits · e / os / android_frameworks_av

media/libstagefright/ACodec.cpp

+25 −6

Original line number	Diff line number	Diff line
		@@ -5804,17 +5804,19 @@ bool ACodec::BaseState::onMessageReceived(const sp<AMessage> &msg) {

		case ACodec::kWhatSetSurface:
		{
		sp<AReplyToken> replyID;
		CHECK(msg->senderAwaitsResponse(&replyID));

		sp<RefBase> obj;
		CHECK(msg->findObject("surface", &obj));

		status_t err = mCodec->handleSetSurface(static_cast<Surface *>(obj.get()));

		sp<AReplyToken> replyID;
		if (msg->senderAwaitsResponse(&replyID)) {
		sp<AMessage> response = new AMessage;
		response->setInt32("err", err);
		response->postReply(replyID);
		} else if (err != OK) {
		mCodec->signalError(OMX_ErrorUndefined, err);
		}
		break;
		}

		@@ -8310,6 +8312,23 @@ bool ACodec::OutputPortSettingsChangedState::onMessageReceived(
		break;
		}

		case kWhatSetSurface:
		{
		ALOGV("[%s] Deferring setSurface", mCodec->mComponentName.c_str());

		sp<AReplyToken> replyID;
		CHECK(msg->senderAwaitsResponse(&replyID));

		mCodec->deferMessage(msg);

		sp<AMessage> response = new AMessage;
		response->setInt32("err", OK);
		response->postReply(replyID);

		handled = true;
		break;
		}

		case kWhatCheckIfStuck:
		{
		int32_t generation = 0;