From ec6e1975dd1632c528130a57f2a8acaf2e73613a Mon Sep 17 00:00:00 2001 From: Stefan Andonian Date: Mon, 20 Mar 2023 23:44:20 +0000 Subject: [PATCH] Write ViewCapture data to WmTrace when WindowManager dumps. This allows apps like the Launcher to write ViewCapture data to the wmtrace directory so it can be shown in their UI. Normally, Launcher doesn't have the correct file permissions. Also, Launcher's dump method is called after the wmtrace dir is written to the bug report, so we need to dump sooner via a callback method invoked inside WindowManager. Bug: 224595733 Test: Latency tested this change and verified that a bugreport generated the file properly, moved it to the wmtrace directory, and was picked up properly by the go/web-hv tool. Change-Id: I5091208c618cb43e1b65adb8e78bfcddc92e79bf --- .../src/com/android/quickstep/TouchInteractionService.java | 5 ----- 1 file changed, 5 deletions(-) diff --git a/quickstep/src/com/android/quickstep/TouchInteractionService.java b/quickstep/src/com/android/quickstep/TouchInteractionService.java index 5e14d19fd0..522a9e109b 100644 --- a/quickstep/src/com/android/quickstep/TouchInteractionService.java +++ b/quickstep/src/com/android/quickstep/TouchInteractionService.java @@ -72,7 +72,6 @@ import androidx.annotation.NonNull; import androidx.annotation.Nullable; import androidx.annotation.UiThread; -import com.android.app.viewcapture.SettingsAwareViewCapture; import com.android.launcher3.BaseDraggingActivity; import com.android.launcher3.LauncherPrefs; import com.android.launcher3.R; @@ -1235,10 +1234,6 @@ public class TouchInteractionService extends Service createdOverviewActivity.getDeviceProfile().dump(this, "", pw); } mTaskbarManager.dumpLogs("", pw); - - if (FeatureFlags.CONTINUOUS_VIEW_TREE_CAPTURE.get()) { - SettingsAwareViewCapture.getInstance(this).dump(pw, fd, this); - } } }