| commit | 33902557ec935b49d187af99abb4ad02a8582b34 | [log] [tgz] |
|---|---|---|
| author | Brandon Dayauon <brdayauon@google.com> | Mon Oct 14 12:04:17 2024 -0700 |
| committer | Brandon Dayauon <brdayauon@google.com> | Mon Oct 14 12:05:41 2024 -0700 |
| tree | def469a29f3be60b4d282c26474e3c5c8c2f0e84 | |
| parent | e710e70bc84316d0aeede17ef91913bfd80abe57 [diff] |
Set psDivider to gone in collapse due to updateAdapterItem suppression in test. Set PSDivider to gone during collapse animation. This is to help with https://android-build.corp.google.com/test_hub/search/?filter=&invfilter=unhealthyInvocations&invfilter=staleInvocations&query=test%3Acom.android.quickstep.TaplPrivateSpaceTest%23testPrivateSpaceLockingBehaviour&tab=clusterView&clusterId=12644122704855694316 tapl issue saying divider is there. This is because we suppress updateAdapterItem call and ps container contents aren't completely refreshed. This change is safe to do so anyways since floatingMaskView and decorator takes care of how the end result looks like. bug: 373427348 test: presubmit/tapl/manually Flag: NONE bugfixing for test flakiness Change-Id: Ia422096e022ab8de6745071707620192f03661a5
diff --git a/src/com/android/launcher3/allapps/PrivateProfileManager.java b/src/com/android/launcher3/allapps/PrivateProfileManager.java index e215cab..e7881c4 100644 --- a/src/com/android/launcher3/allapps/PrivateProfileManager.java +++ b/src/com/android/launcher3/allapps/PrivateProfileManager.java
@@ -473,7 +473,8 @@ break; } // Make the private space apps gone to "collapse". - if (mFloatingMaskView == null && isPrivateSpaceItem(currentItem)) { + if ((mFloatingMaskView == null && isPrivateSpaceItem(currentItem)) || + currentItem.viewType == VIEW_TYPE_PRIVATE_SPACE_SYS_APPS_DIVIDER) { RecyclerView.ViewHolder viewHolder = allAppsRecyclerView.findViewHolderForAdapterPosition(i); if (viewHolder != null) {