Add logging to help debug flake when swiping home to overview.
Bug: 238461210 Test: TaplTestsQuickstep#testSwitchToOverview Change-Id: I8ee9af4f4a331c43761ba3738e12acb376ababc2
This commit is contained in:
@@ -30,6 +30,7 @@ import static com.android.quickstep.views.TaskView.FLAG_UPDATE_ALL;
|
||||
import android.annotation.TargetApi;
|
||||
import android.os.Build;
|
||||
import android.util.FloatProperty;
|
||||
import android.util.Log;
|
||||
import android.util.Pair;
|
||||
|
||||
import androidx.annotation.NonNull;
|
||||
@@ -37,6 +38,7 @@ import androidx.annotation.Nullable;
|
||||
|
||||
import com.android.launcher3.BaseQuickstepLauncher;
|
||||
import com.android.launcher3.LauncherState;
|
||||
import com.android.launcher3.Utilities;
|
||||
import com.android.launcher3.anim.AnimatorListeners;
|
||||
import com.android.launcher3.anim.PendingAnimation;
|
||||
import com.android.launcher3.anim.PropertySetter;
|
||||
@@ -84,6 +86,13 @@ public final class RecentsViewStateController extends
|
||||
// While animating into recents, update the visible task data as needed
|
||||
builder.addOnFrameCallback(() -> mRecentsView.loadVisibleTaskData(FLAG_UPDATE_ALL));
|
||||
mRecentsView.updateEmptyMessage();
|
||||
// TODO(b/238461210): Remove logging once root cause of flake detected.
|
||||
if (Utilities.IS_RUNNING_IN_TEST_HARNESS) {
|
||||
Log.d("b/238461210", "RecentsView#setStateWithAnimationInternal getCurrentPage(): "
|
||||
+ mRecentsView.getCurrentPage()
|
||||
+ ", getScrollForPage(getCurrentPage())): "
|
||||
+ mRecentsView.getScrollForPage(mRecentsView.getCurrentPage()));
|
||||
}
|
||||
} else {
|
||||
builder.addListener(
|
||||
AnimatorListeners.forSuccessCallback(mRecentsView::resetTaskVisuals));
|
||||
|
||||
@@ -1521,6 +1521,13 @@ public abstract class RecentsView<ACTIVITY_TYPE extends StatefulActivity<STATE_T
|
||||
// Restore mCurrentPage but don't call setCurrentPage() as that clobbers the scroll.
|
||||
mCurrentPage = previousCurrentPage;
|
||||
} else {
|
||||
// TODO(b/238461210): Remove logging once root cause of flake detected.
|
||||
if (Utilities.IS_RUNNING_IN_TEST_HARNESS) {
|
||||
Log.d("b/238461210", "RecentsView#applyLoadPlan() -> !settlingOnNewTask -> "
|
||||
+ "previousCurrentPage: " + previousCurrentPage
|
||||
+ ", getScrollForPage(previousCurrentPage): "
|
||||
+ getScrollForPage(previousCurrentPage));
|
||||
}
|
||||
setCurrentPage(previousCurrentPage);
|
||||
}
|
||||
|
||||
|
||||
@@ -767,6 +767,13 @@ public abstract class PagedView<T extends View & PageIndicator> extends ViewGrou
|
||||
}
|
||||
|
||||
if (mScroller.isFinished() && pageScrollChanged) {
|
||||
// TODO(b/238461210): Remove logging once root cause of flake detected.
|
||||
if (Utilities.IS_RUNNING_IN_TEST_HARNESS && !(this instanceof Workspace)) {
|
||||
Log.d("b/238461210", this.getClass().getSimpleName() + "#onLayout() -> "
|
||||
+ "if(mScroller.isFinished() && pageScrollChanged) -> getNextPage(): "
|
||||
+ getNextPage() + ", getScrollForPage(getNextPage()): "
|
||||
+ getScrollForPage(getNextPage()));
|
||||
}
|
||||
setCurrentPage(getNextPage());
|
||||
}
|
||||
onPageScrollsInitialized();
|
||||
|
||||
Reference in New Issue
Block a user