Merge "[Composer3] Update notifyExpectedPresent parameter names" into main

This commit is contained in:
Ram Indani
2023-12-21 23:24:36 +00:00
committed by Android (Google) Code Review
4 changed files with 11 additions and 10 deletions

View File

@@ -42,7 +42,7 @@ parcelable VrrConfig {
int averageRefreshPeriodNs;
}
parcelable NotifyExpectedPresentConfig {
int notifyExpectedPresentHeadsUpNs;
int notifyExpectedPresentTimeoutNs;
int headsUpNs;
int timeoutNs;
}
}

View File

@@ -895,9 +895,9 @@ interface IComposerClient {
*
* The framework will call this function based on the parameters specified in
* DisplayConfiguration.VrrConfig:
* - notifyExpectedPresentTimeoutNs specifies the idle time from the previous present command
* where the framework must send the early hint for the next frame.
* - notifyExpectedPresentHeadsUpNs specifies minimal time that framework must send
* - notifyExpectedPresentConfig.timeoutNs specifies the idle time from the previous
* present command where the framework must send the early hint for the next frame.
* - notifyExpectedPresentConfig.headsUpNs specifies minimal time that framework must send
* the early hint before the next frame.
*
* The framework can omit calling this API when the next present command matches

View File

@@ -45,15 +45,16 @@ parcelable VrrConfig {
* The minimal time in nanoseconds that IComposerClient.notifyExpectedPresent needs to be
* called ahead of an expectedPresentTime provided on a presentDisplay command.
*/
int notifyExpectedPresentHeadsUpNs;
int headsUpNs;
/**
* The time in nanoseconds that represents a timeout from the previous presentDisplay, which
* after this point the display needs a call to IComposerClient.notifyExpectedPresent before
* sending the next frame. If set to 0, there is no need to call
* sending the next frame.
* If set to 0, hint is sent for every frame.
* IComposerClient.notifyExpectedPresent for timeout.
*/
int notifyExpectedPresentTimeoutNs;
int timeoutNs;
}
/**

View File

@@ -1275,8 +1275,8 @@ TEST_P(GraphicsComposerAidlV3Test, GetDisplayConfigurations) {
if (vrrConfig.notifyExpectedPresentConfig) {
const auto& notifyExpectedPresentConfig =
*vrrConfig.notifyExpectedPresentConfig;
EXPECT_GT(0, notifyExpectedPresentConfig.notifyExpectedPresentHeadsUpNs);
EXPECT_GE(0, notifyExpectedPresentConfig.notifyExpectedPresentTimeoutNs);
EXPECT_GE(notifyExpectedPresentConfig.headsUpNs, 0);
EXPECT_GE(notifyExpectedPresentConfig.timeoutNs, 0);
}
}
}