MFClear Cleanup: Remove MediaFoundationRendererClientExtension mojo interface [chromium/src : main]

0 views
Skip to first unread message

gwsq (Gerrit)

unread,
Oct 16, 2025, 11:04:48 PM (4 days ago) Oct 16
to Nic Williamson, Chromium IPC Reviews, Antonio Sartori, Dale Curtis, Piet Schouten, Chromium LUCI CQ, chromium...@chromium.org, Rijubrata Bhaumik, blink-...@chromium.org, chfreme...@chromium.org, chromeos-gfx-...@google.com, chromium-a...@chromium.org, eme-r...@chromium.org, emircan+watch...@chromium.org, extension...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, ipc-securi...@chromium.org, mcasas+med...@chromium.org, media-cro...@chromium.org
Attention needed from Antonio Sartori, Dale Curtis and Piet Schouten

Message from gwsq

From googleclient/chrome/chromium_gwsq/ipc/config.gwsq:
IPC: antonio...@chromium.org

📎 It looks like you’re making a possibly security-sensitive change! 📎 IPC security review isn’t a rubberstamp, so your friendly security reviewer will need a fair amount of context to review your CL effectively. Please review your CL description and code comments to make sure they provide context for someone unfamiliar with your project/area. Pay special attention to where data comes from and which processes it flows between (and their privilege levels). Feel free to point your security reviewer at design docs, bugs, or other links if you can’t reasonably make a self-contained CL description. (Also see https://cbea.ms/git-commit/).

IPC reviewer(s): antonio...@chromium.org


Reviewer source(s):
antonio...@chromium.org is from context(googleclient/chrome/chromium_gwsq/ipc/config.gwsq)

Open in Gerrit

Related details

Attention is currently required from:
  • Antonio Sartori
  • Dale Curtis
  • Piet Schouten
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement is not satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I198d60ab8c0d725dc5da9d0c72e5174164985d65
Gerrit-Change-Number: 7049854
Gerrit-PatchSet: 1
Gerrit-Owner: Nic Williamson <cham...@microsoft.com>
Gerrit-Reviewer: Antonio Sartori <antonio...@chromium.org>
Gerrit-Reviewer: Dale Curtis <dalec...@chromium.org>
Gerrit-Reviewer: Nic Williamson <cham...@microsoft.com>
Gerrit-Reviewer: Piet Schouten <Piet.S...@microsoft.com>
Gerrit-CC: Chromium IPC Reviews <chrome-ip...@google.com>
Gerrit-CC: Rijubrata Bhaumik <rijubrat...@intel.com>
Gerrit-CC: gwsq
Gerrit-Attention: Piet Schouten <Piet.S...@microsoft.com>
Gerrit-Attention: Antonio Sartori <antonio...@chromium.org>
Gerrit-Attention: Dale Curtis <dalec...@chromium.org>
Gerrit-Comment-Date: Fri, 17 Oct 2025 03:04:13 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy

Antonio Sartori (Gerrit)

unread,
Oct 17, 2025, 1:59:37 AM (4 days ago) Oct 17
to Nic Williamson, Chromium IPC Reviews, Dale Curtis, Piet Schouten, Chromium LUCI CQ, chromium...@chromium.org, Rijubrata Bhaumik, blink-...@chromium.org, chfreme...@chromium.org, chromeos-gfx-...@google.com, chromium-a...@chromium.org, eme-r...@chromium.org, emircan+watch...@chromium.org, extension...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, ipc-securi...@chromium.org, mcasas+med...@chromium.org, media-cro...@chromium.org
Attention needed from Dale Curtis, Nic Williamson and Piet Schouten

Antonio Sartori added 1 comment

Patchset-level comments
File-level comment, Patchset 1 (Latest):
Antonio Sartori . resolved

removing mojo interface LGTM

Open in Gerrit

Related details

Attention is currently required from:
  • Dale Curtis
  • Nic Williamson
  • Piet Schouten
Gerrit-Attention: Dale Curtis <dalec...@chromium.org>
Gerrit-Attention: Nic Williamson <cham...@microsoft.com>
Gerrit-Comment-Date: Fri, 17 Oct 2025 05:59:23 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy

Antonio Sartori (Gerrit)

unread,
Oct 17, 2025, 2:02:31 AM (4 days ago) Oct 17
to Nic Williamson, Chromium IPC Reviews, Dale Curtis, Piet Schouten, Chromium LUCI CQ, chromium...@chromium.org, Rijubrata Bhaumik, blink-...@chromium.org, chfreme...@chromium.org, chromeos-gfx-...@google.com, chromium-a...@chromium.org, eme-r...@chromium.org, emircan+watch...@chromium.org, extension...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, ipc-securi...@chromium.org, mcasas+med...@chromium.org, media-cro...@chromium.org
Attention needed from Dale Curtis, Nic Williamson and Piet Schouten

Antonio Sartori voted Code-Review+1

Code-Review+1
Gerrit-Comment-Date: Fri, 17 Oct 2025 06:02:18 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Dale Curtis (Gerrit)

unread,
Oct 17, 2025, 12:59:56 PM (3 days ago) Oct 17
to Nic Williamson, Antonio Sartori, Chromium IPC Reviews, Piet Schouten, Chromium LUCI CQ, chromium...@chromium.org, Rijubrata Bhaumik, blink-...@chromium.org, chfreme...@chromium.org, chromeos-gfx-...@google.com, chromium-a...@chromium.org, eme-r...@chromium.org, emircan+watch...@chromium.org, extension...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, ipc-securi...@chromium.org, mcasas+med...@chromium.org, media-cro...@chromium.org
Attention needed from Nic Williamson and Piet Schouten

Dale Curtis voted Code-Review+1

Code-Review+1
Open in Gerrit

Related details

Attention is currently required from:
  • Nic Williamson
  • Piet Schouten
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I198d60ab8c0d725dc5da9d0c72e5174164985d65
Gerrit-Change-Number: 7049854
Gerrit-PatchSet: 1
Gerrit-Owner: Nic Williamson <cham...@microsoft.com>
Gerrit-Reviewer: Antonio Sartori <antonio...@chromium.org>
Gerrit-Reviewer: Dale Curtis <dalec...@chromium.org>
Gerrit-Reviewer: Nic Williamson <cham...@microsoft.com>
Gerrit-Reviewer: Piet Schouten <Piet.S...@microsoft.com>
Gerrit-CC: Chromium IPC Reviews <chrome-ip...@google.com>
Gerrit-CC: Rijubrata Bhaumik <rijubrat...@intel.com>
Gerrit-CC: gwsq
Gerrit-Attention: Piet Schouten <Piet.S...@microsoft.com>
Gerrit-Attention: Nic Williamson <cham...@microsoft.com>
Gerrit-Comment-Date: Fri, 17 Oct 2025 16:59:43 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Nic Williamson (Gerrit)

unread,
4:25 PM (4 hours ago) 4:25 PM
to Dale Curtis, Antonio Sartori, Chromium IPC Reviews, Piet Schouten, Chromium LUCI CQ, chromium...@chromium.org, Rijubrata Bhaumik, blink-...@chromium.org, chfreme...@chromium.org, chromeos-gfx-...@google.com, chromium-a...@chromium.org, eme-r...@chromium.org, emircan+watch...@chromium.org, extension...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, ipc-securi...@chromium.org, mcasas+med...@chromium.org, media-cro...@chromium.org
Attention needed from Piet Schouten

Nic Williamson voted Commit-Queue+2

Commit-Queue+2
Open in Gerrit

Related details

Attention is currently required from:
  • Piet Schouten
Gerrit-Comment-Date: Mon, 20 Oct 2025 20:25:45 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages