Refactored DummyConsumer
DummyConsumer no longer uses a ProxyConsumerListener Change-Id: I0a99428b3f9dae82cb47d5da3589de77653f3d7c
This commit is contained in:
parent
bb67556871
commit
9d61a28d72
|
@ -25,14 +25,11 @@ namespace android {
|
||||||
|
|
||||||
// The DummyConsumer does not keep a reference to BufferQueue
|
// The DummyConsumer does not keep a reference to BufferQueue
|
||||||
// unlike SurfaceTexture. This prevents a circular reference from
|
// unlike SurfaceTexture. This prevents a circular reference from
|
||||||
// forming. The BufferQueue indirectly keeps a DummyConsumer alive
|
// forming without having to use a ProxyConsumerListener
|
||||||
// through the proxy ConsumerListener.
|
class DummyConsumer : public BufferQueue::ConsumerListener {
|
||||||
class DummyConsumer : public virtual RefBase,
|
|
||||||
protected BufferQueue::ConsumerListener {
|
|
||||||
public:
|
public:
|
||||||
DummyConsumer(sp<BufferQueue> bufferQueue);
|
DummyConsumer();
|
||||||
virtual ~DummyConsumer();
|
virtual ~DummyConsumer();
|
||||||
|
|
||||||
protected:
|
protected:
|
||||||
|
|
||||||
// Implementation of the BufferQueue::ConsumerListener interface. These
|
// Implementation of the BufferQueue::ConsumerListener interface. These
|
||||||
|
|
|
@ -24,17 +24,8 @@
|
||||||
|
|
||||||
namespace android {
|
namespace android {
|
||||||
|
|
||||||
DummyConsumer::DummyConsumer(sp<BufferQueue> bufferQueue) {
|
DummyConsumer::DummyConsumer() {
|
||||||
wp<BufferQueue::ConsumerListener> listener;
|
ALOGV("DummyConsumer");
|
||||||
sp<BufferQueue::ConsumerListener> proxy;
|
|
||||||
listener = static_cast<BufferQueue::ConsumerListener*>(this);
|
|
||||||
proxy = new BufferQueue::ProxyConsumerListener(listener);
|
|
||||||
|
|
||||||
status_t err = bufferQueue->consumerConnect(proxy);
|
|
||||||
if (err != NO_ERROR) {
|
|
||||||
ALOGE("DummyConsumer: error connecting to BufferQueue: %s (%d)",
|
|
||||||
strerror(-err), err);
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
DummyConsumer::~DummyConsumer() {
|
DummyConsumer::~DummyConsumer() {
|
||||||
|
|
Loading…
Reference in New Issue