From b13d750d9497f0f3f05955955f34231635ce2931 Mon Sep 17 00:00:00 2001 From: Steven Thomas Date: Mon, 2 Mar 2020 15:40:02 -0800 Subject: Remove frame rate functions from buffer queue Remove the frame rate functions that were recently added to buffer queue in ag/10028291. Modifying buffer queue is considered risky because we have prebuilt binaries that use the classes in unknown ways. We added an alternative mechanism for communicating frame rate info in ag/10221036, so this code is no longer used for anything. Bug: 150642189 Test: - Built and deployed to a Pixel 4, confirmed normal behavior. - Built all the surface flinger unit tests. - Ran the setFrameRate() cts test. Change-Id: I26a620e842e79a378f58a78a088cff872a49de0d --- libs/gui/ConsumerBase.cpp | 18 ------------------ 1 file changed, 18 deletions(-) (limited to 'libs/gui/ConsumerBase.cpp') diff --git a/libs/gui/ConsumerBase.cpp b/libs/gui/ConsumerBase.cpp index 515f45c546..9f91d9d3aa 100644 --- a/libs/gui/ConsumerBase.cpp +++ b/libs/gui/ConsumerBase.cpp @@ -363,24 +363,6 @@ status_t ConsumerBase::discardFreeBuffers() { return OK; } -status_t ConsumerBase::setFrameRate(float frameRate) { - Mutex::Autolock _l(mMutex); - if (mAbandoned) { - CB_LOGE("setFrameRate: ConsumerBase is abandoned!"); - return NO_INIT; - } - return mConsumer->setFrameRate(frameRate); -} - -status_t ConsumerBase::getFrameRate(float* frameRate) { - Mutex::Autolock _l(mMutex); - if (mAbandoned) { - CB_LOGE("getFrameRate: ConsumerBase is abandoned!"); - return NO_INIT; - } - return mConsumer->getFrameRate(frameRate); -} - void ConsumerBase::dumpState(String8& result) const { dumpState(result, ""); } -- cgit v1.2.3-59-g8ed1b