Skip to content

Commit 673f3aa

Browse files
Ram IndaniAndroid (Google) Code Review
authored andcommitted
Merge "[Native] FRAME_RATE_COMPATIBILITY_GTE renamed to FRAME_RATE_COMPATIBILITY_AT_LEAST" into main
2 parents 10fcdbe + bcde3ec commit 673f3aa

5 files changed

Lines changed: 9 additions & 9 deletions

File tree

libs/gui/FrameRateUtils.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ bool ValidateFrameRate(float frameRate, int8_t compatibility, int8_t changeFrame
4242

4343
if (compatibility != ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_DEFAULT &&
4444
compatibility != ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_FIXED_SOURCE &&
45-
compatibility != ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_GTE &&
45+
compatibility != ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_AT_LEAST &&
4646
(!privileged ||
4747
(compatibility != ANATIVEWINDOW_FRAME_RATE_EXACT &&
4848
compatibility != ANATIVEWINDOW_FRAME_RATE_NO_VOTE))) {

libs/gui/tests/FrameRateUtilsTest.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ TEST(FrameRateUtilsTest, ValidateFrameRate) {
3434
ANATIVEWINDOW_CHANGE_FRAME_RATE_ALWAYS, ""));
3535
EXPECT_TRUE(ValidateFrameRate(60.0f, ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_FIXED_SOURCE,
3636
ANATIVEWINDOW_CHANGE_FRAME_RATE_ONLY_IF_SEAMLESS, ""));
37-
EXPECT_TRUE(ValidateFrameRate(60.0f, ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_GTE,
37+
EXPECT_TRUE(ValidateFrameRate(60.0f, ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_AT_LEAST,
3838
ANATIVEWINDOW_CHANGE_FRAME_RATE_ONLY_IF_SEAMLESS, ""));
3939

4040
// Privileged APIs.

libs/nativewindow/include/android/native_window.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -258,11 +258,11 @@ enum ANativeWindow_FrameRateCompatibility {
258258
ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_FIXED_SOURCE = 1,
259259

260260
/**
261-
* The window requests a frame rate that is greater than or equal to the specified frame rate.
261+
* The window requests a frame rate that is at least the specified frame rate.
262262
* This value should be used for UIs, animations, scrolling, and anything that is not a game
263263
* or video.
264264
*/
265-
ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_GTE = 2
265+
ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_AT_LEAST = 2
266266
};
267267

268268
/**

services/surfaceflinger/Scheduler/LayerInfo.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -504,7 +504,7 @@ FrameRateCompatibility LayerInfo::FrameRate::convertCompatibility(int8_t compati
504504
return FrameRateCompatibility::Exact;
505505
case ANATIVEWINDOW_FRAME_RATE_MIN:
506506
return FrameRateCompatibility::Min;
507-
case ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_GTE:
507+
case ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_AT_LEAST:
508508
return FrameRateCompatibility::Gte;
509509
case ANATIVEWINDOW_FRAME_RATE_NO_VOTE:
510510
return FrameRateCompatibility::NoVote;

services/surfaceflinger/tests/unittests/LayerHistoryIntegrationTest.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -584,7 +584,7 @@ TEST_F(LayerHistoryIntegrationTest, oneLayerExplicitGte_vrr) {
584584

585585
auto layer = createLegacyAndFrontedEndLayer(1);
586586
showLayer(1);
587-
setFrameRate(1, (33_Hz).getValue(), ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_GTE,
587+
setFrameRate(1, (33_Hz).getValue(), ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_AT_LEAST,
588588
ANATIVEWINDOW_CHANGE_FRAME_RATE_ONLY_IF_SEAMLESS);
589589
setFrameRateCategory(1, 0);
590590

@@ -623,7 +623,7 @@ TEST_F(LayerHistoryIntegrationTest, oneLayerExplicitGte_nonVrr) {
623623

624624
auto layer = createLegacyAndFrontedEndLayer(1);
625625
showLayer(1);
626-
setFrameRate(1, (33_Hz).getValue(), ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_GTE,
626+
setFrameRate(1, (33_Hz).getValue(), ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_AT_LEAST,
627627
ANATIVEWINDOW_CHANGE_FRAME_RATE_ONLY_IF_SEAMLESS);
628628
setFrameRateCategory(1, 0);
629629

@@ -662,7 +662,7 @@ TEST_F(LayerHistoryIntegrationTest, oneLayerGteNoVote_arr) {
662662

663663
auto layer = createLegacyAndFrontedEndLayer(1);
664664
showLayer(1);
665-
setFrameRate(1, (0_Hz).getValue(), ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_GTE,
665+
setFrameRate(1, (0_Hz).getValue(), ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_AT_LEAST,
666666
ANATIVEWINDOW_CHANGE_FRAME_RATE_ONLY_IF_SEAMLESS);
667667

668668
EXPECT_EQ(1u, layerCount());
@@ -694,7 +694,7 @@ TEST_F(LayerHistoryIntegrationTest, oneLayerGteNoVote_mrr) {
694694

695695
auto layer = createLegacyAndFrontedEndLayer(1);
696696
showLayer(1);
697-
setFrameRate(1, (0_Hz).getValue(), ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_GTE,
697+
setFrameRate(1, (0_Hz).getValue(), ANATIVEWINDOW_FRAME_RATE_COMPATIBILITY_AT_LEAST,
698698
ANATIVEWINDOW_CHANGE_FRAME_RATE_ONLY_IF_SEAMLESS);
699699
setFrameRateCategory(1, 0);
700700

0 commit comments

Comments
 (0)