From cc13d405990d9fb35e849d59f805f5afcf2fd053 Mon Sep 17 00:00:00 2001 From: Riddle Hsu Date: Wed, 28 Jul 2021 14:55:49 +0000 Subject: [PATCH] Revert "Add interval for relayout perf test" This reverts commit 9ba705609f019eafe1ca16fe5c02d50006416f9b. Reason for revert: The metrics become unstable Bug: 129527548 Change-Id: If1b9368eb55b124a8d66f06c82de193e4ddb9318 --- .../windowmanager/src/android/wm/RelayoutPerfTest.java | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/apct-tests/perftests/windowmanager/src/android/wm/RelayoutPerfTest.java b/apct-tests/perftests/windowmanager/src/android/wm/RelayoutPerfTest.java index 44e6c702fd90e..1be68f5d53a44 100644 --- a/apct-tests/perftests/windowmanager/src/android/wm/RelayoutPerfTest.java +++ b/apct-tests/perftests/windowmanager/src/android/wm/RelayoutPerfTest.java @@ -22,7 +22,6 @@ import android.app.Activity; import android.content.Context; import android.graphics.Point; import android.os.RemoteException; -import android.os.SystemClock; import android.perftests.utils.BenchmarkState; import android.perftests.utils.PerfStatusReporter; import android.perftests.utils.PerfTestActivity; @@ -125,12 +124,6 @@ public class RelayoutPerfTest extends WindowManagerPerfTestBase } private static class RelayoutRunner { - /** - * There may be some messages post to other threads with holding WM lock after relayout. - * Let it take a break to avoid lock contention that isn't the scope of this test. - */ - private static final long INTERVAL_MS = 10; - final ClientWindowFrames mOutFrames = new ClientWindowFrames(); final MergedConfiguration mOutMergedConfiguration = new MergedConfiguration(); final InsetsState mOutInsetsState = new InsetsState(); @@ -165,9 +158,6 @@ public class RelayoutPerfTest extends WindowManagerPerfTestBase mViewVisibility.getAsInt(), mFlags, mFrameNumber, mOutFrames, mOutMergedConfiguration, mOutSurfaceControl, mOutInsetsState, mOutControls, mOutSurfaceSize); - state.pauseTiming(); - SystemClock.sleep(INTERVAL_MS); - state.resumeTiming(); } } }