Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use blast sync only for overlay decor when resizing display

Browse Source 
Only the windows above the screenshot need to be synced.
This avoids requesting several windows in the same process
to sync at the same time, which may occupy buffers and
block RenderThread.

Bug: 277214890
Test: CompatChangeTests
Test: Loop "wm size 800x1600" and "wm size reset".
      SystemUI still works normally.

Change-Id: I611586ee2edea1dfca3e9b59497fe3e76395589f
This commit is contained in:
Riddle Hsu
2023-04-10 23:53:25 +08:00
parent 7dd18dd348
commit c55dbcd4ec
1 changed files with 20 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

30
services/core/java/com/android/server/wm/TransitionController.java
View File

@@ -443,26 +443,36 @@ class TransitionController {
return type == TRANSIT_OPEN || type == TRANSIT_CLOSE;
}
/** Whether the display change should run with blast sync. */
private static boolean shouldSync(@NonNull TransitionRequestInfo.DisplayChange displayChange) {
if ((displayChange.getStartRotation() + displayChange.getEndRotation()) % 2 == 0) {
/** Sets the sync method for the display change. */
private void setDisplaySyncMethod(@NonNull TransitionRequestInfo.DisplayChange displayChange,
@NonNull Transition displayTransition, @NonNull DisplayContent displayContent) {
final int startRotation = displayChange.getStartRotation();
final int endRotation = displayChange.getEndRotation();
if (startRotation != endRotation && (startRotation + endRotation) % 2 == 0) {
// 180 degrees rotation change may not change screen size. So the clients may draw
// some frames before and after the display projection transaction is applied by the
// remote player. That may cause some buffers to show in different rotation. So use
// sync method to pause clients drawing until the projection transaction is applied.
return true;
mAtm.mWindowManager.mSyncEngine.setSyncMethod(displayTransition.getSyncId(),
BLASTSyncEngine.METHOD_BLAST);
}
final Rect startBounds = displayChange.getStartAbsBounds();
final Rect endBounds = displayChange.getEndAbsBounds();
if (startBounds == null || endBounds == null) return false;
if (startBounds == null || endBounds == null) return;
final int startWidth = startBounds.width();
final int startHeight = startBounds.height();
final int endWidth = endBounds.width();
final int endHeight = endBounds.height();
// This is changing screen resolution. Because the screen decor layers are excluded from
// screenshot, their draw transactions need to run with the start transaction.
return (endWidth > startWidth) == (endHeight > startHeight)
&& (endWidth != startWidth || endHeight != startHeight);
if ((endWidth > startWidth) == (endHeight > startHeight)
&& (endWidth != startWidth || endHeight != startHeight)) {
displayContent.forAllWindows(w -> {
if (w.mToken.mRoundedCornerOverlay && w.mHasSurface) {
w.mSyncMethodOverride = BLASTSyncEngine.METHOD_BLAST;
}
}, true /* traverseTopToBottom */);
}
}
/**
@@ -494,9 +504,9 @@ class TransitionController {
} else {
newTransition = requestStartTransition(createTransition(type, flags),
trigger != null ? trigger.asTask() : null, remoteTransition, displayChange);
if (newTransition != null && displayChange != null && shouldSync(displayChange)) {
mAtm.mWindowManager.mSyncEngine.setSyncMethod(newTransition.getSyncId(),
BLASTSyncEngine.METHOD_BLAST);
if (newTransition != null && displayChange != null && trigger != null
&& trigger.asDisplayContent() != null) {
setDisplaySyncMethod(displayChange, newTransition, trigger.asDisplayContent());
}
}
if (trigger != null) {