diff options
author | 2020-03-02 15:40:02 -0800 | |
---|---|---|
committer | 2020-03-02 15:58:39 -0800 | |
commit | b13d750d9497f0f3f05955955f34231635ce2931 (patch) | |
tree | 856a389bafb99dc81acb78bacda549f741e9f980 /libs/gui/ConsumerBase.cpp | |
parent | 7e221c91cd753edff6c59e61fea7619842a9addb (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.cpp | 18 |
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, ""); } |