[//media] Clarify param name in CopyVideoFrameToSharedImage() [chromium/src : main]

0 views
Skip to first unread message

Colin Blundell (Gerrit)

unread,
11:32 AM (3 hours ago) 11:32 AM
to Colin Blundell, Vasiliy Telezhnikov, Chromium LUCI CQ, AyeAye, chromium...@chromium.org, feature-me...@chromium.org
Attention needed from Vasiliy Telezhnikov

Colin Blundell added 1 comment

Commit Message
Line 11, Patchset 3 (Latest):* The cache of RGB shared images that serve as the destinations of
Colin Blundell . unresolved

Should we (in followup) move the generation of dest images from this cache and check on whether the copy actually needs to be done into CopyVideoFrameToSharedImage() as well, passing the generated SharedImage back as some kind of out-param?

Open in Gerrit

Related details

Attention is currently required from:
  • Vasiliy Telezhnikov
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedNo-Unresolved-Comments
  • 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: I4146442cb14286935217c85e0a07fe1076a4d248
Gerrit-Change-Number: 7638769
Gerrit-PatchSet: 3
Gerrit-Owner: Colin Blundell <blun...@chromium.org>
Gerrit-Reviewer: Colin Blundell <blun...@chromium.org>
Gerrit-Reviewer: Vasiliy Telezhnikov <vas...@chromium.org>
Gerrit-Attention: Vasiliy Telezhnikov <vas...@chromium.org>
Gerrit-Comment-Date: Thu, 05 Mar 2026 16:32:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages