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/BufferQueueConsumer.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/BufferQueueConsumer.cpp')
| -rw-r--r-- | libs/gui/BufferQueueConsumer.cpp | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/libs/gui/BufferQueueConsumer.cpp b/libs/gui/BufferQueueConsumer.cpp index 4435265bb2..da6143c59f 100644 --- a/libs/gui/BufferQueueConsumer.cpp +++ b/libs/gui/BufferQueueConsumer.cpp @@ -783,18 +783,6 @@ status_t BufferQueueConsumer::discardFreeBuffers() { return NO_ERROR; } -status_t BufferQueueConsumer::setFrameRate(float frameRate) { - std::lock_guard<std::mutex> lock(mCore->mMutex); - mCore->mFrameRate = frameRate; - return NO_ERROR; -} - -status_t BufferQueueConsumer::getFrameRate(float* frameRate) const { - std::lock_guard<std::mutex> lock(mCore->mMutex); - *frameRate = mCore->mFrameRate; - return NO_ERROR; -} - status_t BufferQueueConsumer::dumpState(const String8& prefix, String8* outResult) const { struct passwd* pwd = getpwnam("shell"); uid_t shellUid = pwd ? pwd->pw_uid : 0; |