Glenn Hartmann (Gerrit)

unread,

Jan 12, 2026, 12:19:26 PMJan 12

to Mark Foltz, Chromium LUCI CQ, chromium...@chromium.org, feature-me...@chromium.org, mfoltz+wa...@chromium.org

Attention needed from Mark Foltz

New activity on the change

Open in Gerrit

Related details

Attention is currently required from:

Mark Foltz

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

satisfied_requirement

unsatisfied_requirement

open

diffy

Mark Foltz (Gerrit)

unread,

Jan 13, 2026, 2:46:15 PMJan 13

to Glenn Hartmann, Mark Foltz, Chromium LUCI CQ, chromium...@chromium.org, feature-me...@chromium.org, mfoltz+wa...@chromium.org

Attention needed from Glenn Hartmann

Mark Foltz voted Code-Review+1

Code-Review

+1

Open in Gerrit

Related details

Attention is currently required from:

Glenn Hartmann

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

satisfied_requirement

open

diffy

Glenn Hartmann (Gerrit)

unread,

Jan 13, 2026, 2:51:25 PMJan 13

to Mark Foltz, Chromium LUCI CQ, chromium...@chromium.org, feature-me...@chromium.org, mfoltz+wa...@chromium.org

Glenn Hartmann voted Commit-Queue+2

Commit-Queue

+2

Open in Gerrit

Related details

Attention set is empty

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

satisfied_requirement

open

diffy

Chromium LUCI CQ (Gerrit)

unread,

Jan 13, 2026, 4:09:06 PMJan 13

to Glenn Hartmann, Mark Foltz, chromium...@chromium.org, feature-me...@chromium.org, mfoltz+wa...@chromium.org

Chromium LUCI CQ submitted the change

Change information

Commit message:

[bedrock] Migrate web_contents_display_observer_view.cc away from BrowserListObserver.

Note that this changes the existing observer from listening to all
browsers globally to just listening to browsers within the relevant
Profile.

This migration is part of project bedrock to reduce the dependencies on
Browser and BrowserList. See https://crbug.com/431671320 for more info.

Bug: 459505408

Change-Id: I76fe752471506663e970c69d5ff08bf86a6a6964

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

Reviewed-by: Mark Foltz <mfo...@chromium.org>

Commit-Queue: Glenn Hartmann <hart...@chromium.org>

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

Files:

M chrome/browser/ui/views/media_router/web_contents_display_observer_view.cc
M chrome/browser/ui/views/media_router/web_contents_display_observer_view.h

Change size: S

Delta: 2 files changed, 17 insertions(+), 7 deletions(-)

Branch: refs/heads/main

Submit Requirements:

Code-Review: +1 by Mark Foltz

Open in Gerrit

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

Gerrit

open

diffy

satisfied_requirement

Reply all

Reply to author

Forward

[bedrock] Migrate web_contents_display_observer_view.cc away from BrowserListObserver. [chromium/src : main]

Glenn Hartmann (Gerrit)

New activity on the change

Related details

Mark Foltz (Gerrit)

Mark Foltz voted Code-Review+1

Related details

Glenn Hartmann (Gerrit)

Glenn Hartmann voted Commit-Queue+2

Related details

Chromium LUCI CQ (Gerrit)

Chromium LUCI CQ submitted the change

Change information