Allow reentrancy for observer list in [chromium/src : main]

[Mitsuru Oshima (Gerrit)]

Attention needed from Alexander Hendrich

Mitsuru Oshima voted and added 1 comment

Votes added by Mitsuru Oshima

Auto-Submit	+0
Commit-Queue	+2

1 comment

Commit Message

Line 7, Patchset 1:Split CL for ownership optimization

Alexander Hendrich . resolved

nit: CL description just says that it was split off from a larger change, but not what the actual change is about

Mitsuru Oshima

Sorry I set to WIP bc gemini-cli (or split tool) automaticaly sent for review.
I'll update the description and will land.

Mitsuru Oshima

Done

Open in Gerrit

Related details

Attention is currently required from:

• Alexander Hendrich

Submit Requirements:

• Code-Coverage
• Code-Owners
• Code-Review
• Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

Gerrit-MessageType: comment

Gerrit-Project: chromium/src

Gerrit-Branch: main

Gerrit-Change-Id: Id7f7673f75729bccc65bc9a14bc7170c29e07693

Gerrit-Change-Number: 7691923

Gerrit-PatchSet: 3

Gerrit-Owner: Mitsuru Oshima <osh...@chromium.org>

Gerrit-Reviewer: Alexander Hendrich <hend...@chromium.org>

Gerrit-Reviewer: Kaleigh Rosenblat <kal...@google.com>

Gerrit-Reviewer: Mitsuru Oshima <osh...@chromium.org>

Gerrit-CC: Enterprise Policy Reviews <enterprise-p...@google.com>

Gerrit-CC: chromium...@chromium.org

Gerrit-CC: gwsq

Gerrit-Attention: Alexander Hendrich <hend...@chromium.org>

Gerrit-Comment-Date: Mon, 23 Mar 2026 14:11:54 +0000

Gerrit-HasComments: Yes

Gerrit-Has-Labels: Yes

Comment-In-Reply-To: Alexander Hendrich <hend...@chromium.org>

Comment-In-Reply-To: Mitsuru Oshima <osh...@chromium.org>

[Chromium LUCI CQ (Gerrit)]

Chromium LUCI CQ submitted the change

Unreviewed changes

1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Change information

Commit message:

Allow reentrancy for observer list in

components/policy/core/common/cloud, chrome/browser/ash/policy/core

They're identified as currnetly reentrant observer list by gemini-cli.

This CL was uploaded by git cl split.

R=enterprise-p...@google.com

Bug: 40562847, 484371187

Change-Id: Id7f7673f75729bccc65bc9a14bc7170c29e07693

Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7691923

Reviewed-by: Alexander Hendrich <hend...@chromium.org>

Commit-Queue: Mitsuru Oshima <osh...@chromium.org>

Cr-Commit-Position: refs/heads/main@{#1603486}

Files:

• M chrome/browser/ash/policy/core/device_cloud_policy_manager_ash.cc
• M components/policy/core/common/cloud/cloud_policy_store.h

Change size: S

Delta: 2 files changed, 10 insertions(+), 4 deletions(-)

Branch: refs/heads/main

Submit Requirements:

• Code-Review: +1 by Alexander Hendrich

Open in Gerrit

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

Gerrit-MessageType: merged

Gerrit-Project: chromium/src

Gerrit-Branch: main

Gerrit-Change-Id: Id7f7673f75729bccc65bc9a14bc7170c29e07693

Gerrit-Change-Number: 7691923

Gerrit-PatchSet: 4

Gerrit-Owner: Mitsuru Oshima <osh...@chromium.org>

Gerrit-Reviewer: Alexander Hendrich <hend...@chromium.org>

Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>