commit 067fcd36bc53a5f884108df5634cb1ca5549a30c
author Shuzhen Wang <shuzhenwang@google.com> Wed Aug 14 10:41:12 2019 -0700
committer Shuzhen Wang <shuzhenwang@google.com> Tue Sep 03 16:48:02 2019 -0700
tree 6676a28615ae2d7fe754a2de60b4ad38b1175233
parent 5488416631c0a23ccef61975860f2755e8e8d834

libgui: Add discardFreeBuffers callback to producer

This enables buffer producer to clear its buffer caches.

Test: Camera CTS, libgui_test
Bug: 136677409
Change-Id: I7f46e977f3edc3b08177654531745d8ca0b86889

libs/gui/BufferQueueConsumer.cpp

diff --git a/libs/gui/BufferQueueConsumer.cpp b/libs/gui/BufferQueueConsumer.cpp
index 528bfb1..3a7cb44 100644
--- a/libs/gui/BufferQueueConsumer.cpp
+++ b/libs/gui/BufferQueueConsumer.cpp
@@ -166,7 +166,9 @@
                         mCore->mFreeBuffers.push_back(front->mSlot);
                     }
 
-                    listener = mCore->mConnectedProducerListener;
+                    if (mCore->mBufferReleasedCbEnabled) {
+                        listener = mCore->mConnectedProducerListener;
+                    }
                     ++numDroppedBuffers;
                 }
 
@@ -457,7 +459,9 @@
             mCore->mFreeBuffers.push_back(slot);
         }
 
-        listener = mCore->mConnectedProducerListener;
+        if (mCore->mBufferReleasedCbEnabled) {
+            listener = mCore->mConnectedProducerListener;
+        }
         BQ_LOGV("releaseBuffer: releasing slot %d", slot);
 
         mCore->mDequeueCondition.notify_all();
@@ -668,7 +672,7 @@
         BQ_LOGV("setMaxAcquiredBufferCount: %d", maxAcquiredBuffers);
         mCore->mMaxAcquiredBufferCount = maxAcquiredBuffers;
         VALIDATE_CONSISTENCY();
-        if (delta < 0) {
+        if (delta < 0 && mCore->mBufferReleasedCbEnabled) {
             listener = mCore->mConsumerListener;
         }
     }