Skip to content
Commit 037d94fd authored by Daniel Murphy's avatar Daniel Murphy Committed by Chromium LUCI CQ
Browse files

[dPWA] Change FullSystemLock to AllAppsLock, allowing NoopLocks

This change removes the 'FullSystemLock' which blocks ALL locks,
including 'NoopLock's, in leiu of 'AllAppsLock', which locks all app
ids (e.g. it blocks all 'AppLock's).

This fixes a number of issues where a NoopLock being held by a command
is blocking something that needs to lock all apps, which it shouldn't
need to block.

If needed, we can easily add back a 'FullSystemLock', but it is not
necessary right now.

Bug: 1378271
Change-Id: I692a62a92166bcab26baad8a96704a6a251bd0bb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4425472


Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Auto-Submit: Daniel Murphy <dmurph@chromium.org>
Reviewed-by: default avatarGlen Robertson <glenrob@chromium.org>
Reviewed-by: default avatarAlan Cutter <alancutter@chromium.org>
Reviewed-by: default avatarDibyajyoti Pal <dibyapal@chromium.org>
Code-Coverage: Findit <findit-for-me@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#1130494}
parent e0c653f6
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment