Skip to content

Commit 42657b7

Browse files
Merge "Error fix from enabling Clang thread-safety checks in aosp_panther" into main am: 24abe44
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/3279597 Change-Id: I7b4f39063a684750257197e15696bef92355b6d7 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2 parents ec61f80 + 24abe44 commit 42657b7

1 file changed

Lines changed: 6 additions & 2 deletions

File tree

services/surfaceflinger/tests/unittests/RefreshRateSelectorTest.cpp

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,9 @@ struct TestableRefreshRateSelector : RefreshRateSelector {
100100
const std::vector<Fps>& knownFrameRates() const { return mKnownFrameRates; }
101101

102102
using RefreshRateSelector::GetRankedFrameRatesCache;
103-
auto& mutableGetRankedRefreshRatesCache() { return mGetRankedFrameRatesCache; }
103+
auto& mutableGetRankedRefreshRatesCache() NO_THREAD_SAFETY_ANALYSIS {
104+
return mGetRankedFrameRatesCache;
105+
}
104106

105107
auto getRankedFrameRates(const std::vector<LayerRequirement>& layers,
106108
GlobalSignals signals = {}, Fps pacesetterFps = {}) const {
@@ -138,7 +140,9 @@ struct TestableRefreshRateSelector : RefreshRateSelector {
138140
return setPolicy(policy);
139141
}
140142

141-
const auto& getPrimaryFrameRates() const { return mPrimaryFrameRates; }
143+
const auto& getPrimaryFrameRates() const NO_THREAD_SAFETY_ANALYSIS {
144+
return mPrimaryFrameRates;
145+
}
142146
};
143147

144148
class RefreshRateSelectorTest : public testing::TestWithParam<Config::FrameRateOverride> {

0 commit comments

Comments
 (0)