Merge "Moving BrightnessTracker dump outside of syncRoot lock" into udc-dev am: b179431214

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/22200443

Change-Id: I1edc6102b49132e8a67c00bcbc8dd84681ee6f3d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Oleg Petšjonkin
2023-03-23 14:51:31 +00:00
committed by Automerger Merge Worker

View File

@@ -2589,9 +2589,7 @@ public final class DisplayManagerService extends SystemService {
void setDisplayModeDirectorLoggingEnabled(boolean enabled) {
synchronized (mSyncRoot) {
if (mDisplayModeDirector != null) {
mDisplayModeDirector.setLoggingEnabled(enabled);
}
mDisplayModeDirector.setLoggingEnabled(enabled);
}
}
@@ -2852,8 +2850,11 @@ public final class DisplayManagerService extends SystemService {
private void dumpInternal(PrintWriter pw) {
pw.println("DISPLAY MANAGER (dumpsys display)");
BrightnessTracker brightnessTrackerLocal;
synchronized (mSyncRoot) {
brightnessTrackerLocal = mBrightnessTracker;
pw.println(" mSafeMode=" + mSafeMode);
pw.println(" mPendingTraversal=" + mPendingTraversal);
pw.println(" mViewports=" + mViewports);
@@ -2926,10 +2927,6 @@ public final class DisplayManagerService extends SystemService {
for (int i = 0; i < displayPowerControllerCount; i++) {
mDisplayPowerControllers.valueAt(i).dump(pw);
}
if (mBrightnessTracker != null) {
pw.println();
mBrightnessTracker.dump(pw);
}
pw.println();
mPersistentDataStore.dump(pw);
@@ -2942,6 +2939,10 @@ public final class DisplayManagerService extends SystemService {
mDisplayWindowPolicyControllers.valueAt(i).second.dump(" ", pw);
}
}
if (brightnessTrackerLocal != null) {
pw.println();
brightnessTrackerLocal.dump(pw);
}
pw.println();
mDisplayModeDirector.dump(pw);
mBrightnessSynchronizer.dump(pw);