SurfaceFlinger: tune infrequent detection logic more
A layer will be considered frequent unless proven otherwise, in order
to make sure animations will change the refresh rate as soon as possible.
In addition, if a layer is classified infrequent, then we will keep it
infrequent until we get a burst of frames. These two policies combined
allow us to differentiate between an animating layer and a layer that
usually doesn't animate.
Bug: 157096772
Test: Play 24fps video in YouTube PIP mode and rotate the device - no jank
Test: Chrome playing video - no refresh rate switching
Test: Cursor blinking on Messages - switch to lowest refresh rate quickly
Change-Id: I3629b6e4786cd43919f51465e347f2abb52234d9
diff --git a/services/surfaceflinger/Scheduler/LayerInfoV2.h b/services/surfaceflinger/Scheduler/LayerInfoV2.h
index 48bddf5..82da7e3 100644
--- a/services/surfaceflinger/Scheduler/LayerInfoV2.h
+++ b/services/surfaceflinger/Scheduler/LayerInfoV2.h
@@ -94,7 +94,7 @@
bool pendingConfigChange;
};
- bool isFrequent(nsecs_t now) const;
+ bool isFrequent(nsecs_t now);
bool hasEnoughDataForHeuristic() const;
std::optional<float> calculateRefreshRateIfPossible();
std::pair<nsecs_t, bool> calculateAverageFrameTime() const;
@@ -110,6 +110,13 @@
float mLastReportedRefreshRate = 0.0f;
+ // Used to determine whether a layer should be considered frequent or
+ // not when we don't have enough frames. This member will not be cleared
+ // as part of clearHistory() to remember whether this layer was frequent
+ // or not before we processed touch boost (or anything else that would
+ // clear layer history).
+ bool mLastReportedIsFrequent = true;
+
// Holds information about the layer vote
struct {
LayerHistory::LayerVoteType type;