[automerger skipped] Merge "Persist destroyed staged sessions until they are cleaned up" into qt-dev am: 30348d5c am: ca4faab2 -s ours am: 18651732 -s ours am: 467ac2a3 -s ours am: 810c1dd5 -s ours am: 2bf268f0 -s ours

am skip reason: Merged-In I4ede6b7a4b5d861e5c73f13884c7aa86cf7633a2 with SHA-1 731bd965 is already in history

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

Change-Id: I35d983ccf93beae63e2b9578134009c25383e1ff

[automerger skipped] Merge "Don't abandon child sessions (1/n)" into qt-dev am: 5bf4a3f2 -s ours am: a0961661 -s ours am: 793e1271 -s ours am: bebd3ff2 -s ours am: c8a55e53 -s ours am: fd22133b -s ours

am skip reason: skipped by user schfan

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

Change-Id: I57549b872fe364a83ff800b6e5e829575128383b

[automerger skipped] Merge "Persist destroyed staged sessions until they are cleaned up" into qt-dev am: 30348d5c am: ca4faab2 -s ours am: 18651732 -s ours am: 467ac2a3 -s ours am: 810c1dd5 -s ours

am skip reason: Merged-In I4ede6b7a4b5d861e5c73f13884c7aa86cf7633a2 with SHA-1 731bd965 is already in history

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

Change-Id: I6886857954d3d0f9e2d90ad8cb48a4daf53d5f5b

[automerger skipped] Merge "Don't abandon child sessions (1/n)" into qt-dev am: 5bf4a3f2 -s ours am: a0961661 -s ours am: 793e1271 -s ours am: bebd3ff2 -s ours am: c8a55e53 -s ours

am skip reason: skipped by user schfan

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

Change-Id: I4448b452a89858c1c44a28da49e7e7f46fb76643

[automerger skipped] Merge "Persist destroyed staged sessions until they are cleaned up" into qt-dev am: 30348d5c am: ca4faab2 -s ours am: 18651732 -s ours am: 467ac2a3 -s ours

am skip reason: Merged-In I4ede6b7a4b5d861e5c73f13884c7aa86cf7633a2 with SHA-1 731bd965 is already in history

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

Change-Id: Ib52ce7ead476cb1e8409ef8ab418a0b5144fa618

[automerger skipped] Merge "Don't abandon child sessions (1/n)" into qt-dev am: 5bf4a3f2 -s ours am: a0961661 -s ours am: 793e1271 -s ours am: bebd3ff2 -s ours

am skip reason: skipped by user schfan

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

Change-Id: I6bf4e7d093b09b27a5a07bc733fdb5d8c21e04e4

[automerger skipped] Merge "Persist destroyed staged sessions until they are cleaned up" into qt-dev am: 30348d5c am: ca4faab2 -s ours am: 18651732 -s ours

am skip reason: Merged-In I4ede6b7a4b5d861e5c73f13884c7aa86cf7633a2 with SHA-1 731bd965 is already in history

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

Change-Id: I2f1c1e461710916489bc7a282a566b3e3f4c7dde

[automerger skipped] Merge "Don't abandon child sessions (1/n)" into qt-dev am: 5bf4a3f2 -s ours am: a0961661 -s ours am: 793e1271 -s ours

am skip reason: skipped by user schfan

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

Change-Id: I5dc25204fa53a6f77336c466adc92402f8d0e00e

[automerger skipped] Merge "[RESTRICT AUTOMERGE] Fix the inconsistency of protection level" into rvc-dev am: c9fc96e7 am: 6b5b8cd0 -s ours am: 94c78071 -s ours am: 03f25cb0 -s ours

am skip reason: subject contains skip directive

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

Change-Id: I30b183fc5dce963523aa65dcc75095e968d2b41a

[automerger skipped] [RESTRICT AUTOMERGE] Fix the inconsistency of protection level am: b5efdf72 am: c1826ef1 -s ours am: 6d3ad47c -s ours am: 46ace1dc -s ours

am skip reason: subject contains skip directive

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

Change-Id: I50fe8e72b015113966e7a4ecb1cd741c1c518558

[automerger skipped] Merge "[RESTRICT AUTOMERGE] Fix the inconsistency of protection level" into rvc-dev am: c9fc96e7 am: ccb1fef8 -s ours am: 2e684868 -s ours am: 536b639b -s ours

am skip reason: subject contains skip directive

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

Change-Id: I2df89c74f1e40c980b7e87d3da797281c82da3b4

[automerger skipped] [RESTRICT AUTOMERGE] Fix the inconsistency of protection level am: b5efdf72 am: 3142a08e -s ours am: 24dbab1a -s ours am: ba1c351e -s ours

am skip reason: subject contains skip directive

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

Change-Id: I8510c30cadfb1c78ca09f4d3ac3bf914fefac33b

[automerger skipped] Merge "Persist destroyed staged sessions until they are cleaned up" into qt-dev am: 30348d5c am: ca4faab2 -s ours

am skip reason: Merged-In I4ede6b7a4b5d861e5c73f13884c7aa86cf7633a2 with SHA-1 731bd965 is already in history

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

Change-Id: I02d6b14b5a55dc54931feb5ec1b08b41dfa95b92

[automerger skipped] Merge "Don't abandon child sessions (1/n)" into qt-dev am: 5bf4a3f2 -s ours am: a0961661 -s ours

am skip reason: skipped by user schfan

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

Change-Id: I4c52b159f68e5086d84edeb22080323ac205c96e

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

Change-Id: I4b67d39d39b3592139235e3375edf5c408795e79

am skip reason: skipped by user schfan

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

Change-Id: Ie9547a891f73ff858f29c144d4c4c55ddd7e4292

[automerger skipped] Merge "[RESTRICT AUTOMERGE] Fix the inconsistency of protection level" into rvc-dev am: c9fc96e7 am: 6b5b8cd0 -s ours am: 94c78071 -s ours

am skip reason: subject contains skip directive

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

Change-Id: I82eeaf435702c626bc5f519e81412b4c1d184565

[automerger skipped] [RESTRICT AUTOMERGE] Fix the inconsistency of protection level am: b5efdf72 am: c1826ef1 -s ours am: 6d3ad47c -s ours

am skip reason: subject contains skip directive

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

Change-Id: I61259d2f8a3408904434af6e25ad51af590a635b

[automerger skipped] Merge "[RESTRICT AUTOMERGE] Fix the inconsistency of protection level" into rvc-dev am: c9fc96e7 am: ccb1fef8 -s ours am: 2e684868 -s ours

am skip reason: subject contains skip directive

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

Change-Id: I370248b05d8ef29a850416f65174fbccab062279

[automerger skipped] [RESTRICT AUTOMERGE] Fix the inconsistency of protection level am: b5efdf72 am: 3142a08e -s ours am: 24dbab1a -s ours

am skip reason: subject contains skip directive

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

Change-Id: Id3fb0ca272226b3041be8bc43190f663c785acb4

[automerger skipped] Merge "[RESTRICT AUTOMERGE] Fix the inconsistency of protection level" into rvc-dev am: c9fc96e7 am: 6b5b8cd0 -s ours

am skip reason: subject contains skip directive

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

Change-Id: I83320d1cbeb7c0245caec8fd2ffd88ff00fb9446

[automerger skipped] [RESTRICT AUTOMERGE] Fix the inconsistency of protection level am: b5efdf72 am: c1826ef1 -s ours

am skip reason: subject contains skip directive

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

Change-Id: I59c09e837f149088215f515c249ecc5ca452236f

[automerger skipped] Merge "[RESTRICT AUTOMERGE] Fix the inconsistency of protection level" into rvc-dev am: c9fc96e7 am: ccb1fef8 -s ours

am skip reason: subject contains skip directive

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

Change-Id: I2339e1b4090aee194ea0300d02201a717ee4e6b3

[automerger skipped] [RESTRICT AUTOMERGE] Fix the inconsistency of protection level am: b5efdf72 am: 3142a08e -s ours

am skip reason: subject contains skip directive

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

Change-Id: If4dc703efd1a9d49801745c0a95d7aeda41bfcbd

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

Change-Id: Ic21462f65215d1da9791b1913972966e8245c3b5

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

Change-Id: I9c1ffc3bdba6e40ba32be04ada50ffc82892a62c

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

Change-Id: I7b9e808864a6a5a962a5e23a87f7a0e2a902f0f9

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

Change-Id: I5f62762f2fb31797b59849bf299b33e438b756af

[automerger skipped] Check group channels for FGSes am: 331b6179 -s ours am: d21f306f -s ours am: a250ac1c -s ours am: 120d0e7a -s ours am: cb3d58b1 -s ours am: 8143b818 -s ours

am skip reason: skipped by user juliacr

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

Change-Id: I333868287743aa23b3d592a983273d5f52512974

[automerger skipped] Merge "Don't abandon child sessions (1/n)" into rvc-dev am: 966fd5d6 am: 09acdc8e -s ours am: e0823161 -s ours am: 39415c57 -s ours

am skip reason: Merged-In Ib0ba9f3786dda2d3174f3ea8c65d1061a3fcb586 with SHA-1 c981d90b is already in history

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

Change-Id: I6b1053e9dab318c46bff3045242339c1d95c9b68

[automerger skipped] Check group channels for FGSes am: 331b6179 -s ours am: d21f306f -s ours am: a250ac1c -s ours am: 120d0e7a -s ours am: cb3d58b1 -s ours

am skip reason: skipped by user juliacr

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

Change-Id: I80a8c645ddb4267c3fb073ae393f8de8ba7bbbd1

[automerger skipped] Merge "Don't abandon child sessions (1/n)" into rvc-dev am: 966fd5d6 am: 09acdc8e -s ours am: e0823161 -s ours

am skip reason: Merged-In Ib0ba9f3786dda2d3174f3ea8c65d1061a3fcb586 with SHA-1 c981d90b is already in history

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

Change-Id: I91f442ee435ccc69a6f56a93c29ec30c0da9a591

Currently, when we abandon a staged session we mark it as destroyed and
then immediately clean it up. Cleaning up a staged session immediately
causes racing condition with pre-reboot verification.

In order to avoid the racing condition, we want to delay cleanup of
staged session until it is safe to do so. This means, the system will
be carrying around destroyed staged sessions internally.

Since there is now a gap between when a session is destroyed and when it
is cleaned up, the user can reboot in this window. As such, we need to
persist the mDestroyed field of session so that we know session is
destroyed after reboot and act accordingly.

Also, once a session is destroyed, theoretically it doesn't exist.
Carrying it around internally is an implementation details which
shouldn't be exposed externally. As such, we filter out destroyed
sessions before surfacing them to users.

Bug: 145925842
Bug: 67862680
Test: atest PackageInstallerSessionTest
Test: atest StagedInstallTest
Change-Id: I4ede6b7a4b5d861e5c73f13884c7aa86cf7633a2
Merged-In: I4ede6b7a4b5d861e5c73f13884c7aa86cf7633a2
(cherry picked from commit 731bd965)

[automerger skipped] Check group channels for FGSes am: 331b6179 -s ours am: d21f306f -s ours am: a250ac1c -s ours am: 120d0e7a -s ours

am skip reason: skipped by user juliacr

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

Change-Id: I06223fb777295c90c583b96ac547f9d5983bf293

[automerger skipped] Merge "Don't abandon child sessions (1/n)" into rvc-dev am: 966fd5d6 am: 09acdc8e -s ours

am skip reason: Merged-In Ib0ba9f3786dda2d3174f3ea8c65d1061a3fcb586 with SHA-1 c981d90b is already in history

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

Change-Id: Ice9798077dde794de04fc5ae888c1a4930f35a68

[automerger skipped] Check group channels for FGSes am: 331b6179 -s ours am: d21f306f -s ours am: a250ac1c -s ours am: 97ef77e4 -s ours am: 62c00185 -s ours am: 60c77662 -s ours

am skip reason: skipped by user juliacr

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

Change-Id: I90a67b3611eafae52f2cea75f920380a6f81a91f