summaryrefslogtreecommitdiff
path: root/libs/gui/ConsumerBase.cpp
diff options
context:
space:
mode:
author Steven Thomas <steventhomas@google.com> 2020-03-02 15:40:02 -0800
committer Steven Thomas <steventhomas@google.com> 2020-03-02 15:58:39 -0800
commitb13d750d9497f0f3f05955955f34231635ce2931 (patch)
tree856a389bafb99dc81acb78bacda549f741e9f980 /libs/gui/ConsumerBase.cpp
parent7e221c91cd753edff6c59e61fea7619842a9addb (diff)
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
Diffstat (limited to 'libs/gui/ConsumerBase.cpp')
-rw-r--r--libs/gui/ConsumerBase.cpp18
1 files changed, 0 insertions, 18 deletions
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, "");
}