[TDR] Remove TODOs linked to issue 41419817 [chromium/src : main]

0 views
Skip to first unread message

Tomasz Lichoń (Gerrit)

unread,
Jan 19, 2026, 7:26:55 AM (3 days ago) Jan 19
to Dale Curtis, AyeAye, feature-me...@chromium.org
Attention needed from Dale Curtis

Message from Tomasz Lichoń

Set Ready For Review

Open in Gerrit

Related details

Attention is currently required from:
  • Dale Curtis
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: I730f3892469b0d06c1303bbfca49eb640b537625
Gerrit-Change-Number: 7485572
Gerrit-PatchSet: 2
Gerrit-Owner: Tomasz Lichoń <lic...@google.com>
Gerrit-Reviewer: Dale Curtis <dalec...@chromium.org>
Gerrit-Attention: Dale Curtis <dalec...@chromium.org>
Gerrit-Comment-Date: Mon, 19 Jan 2026 12:26:42 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy

Dale Curtis (Gerrit)

unread,
Jan 20, 2026, 1:52:25 PM (2 days ago) Jan 20
to Tomasz Lichoń, AyeAye, feature-me...@chromium.org
Attention needed from Tomasz Lichoń

Dale Curtis voted Code-Review+1

Code-Review+1
Open in Gerrit

Related details

Attention is currently required from:
  • Tomasz Lichoń
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement is not 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: I730f3892469b0d06c1303bbfca49eb640b537625
Gerrit-Change-Number: 7485572
Gerrit-PatchSet: 2
Gerrit-Owner: Tomasz Lichoń <lic...@google.com>
Gerrit-Reviewer: Dale Curtis <dalec...@chromium.org>
Gerrit-Attention: Tomasz Lichoń <lic...@google.com>
Gerrit-Comment-Date: Tue, 20 Jan 2026 18:52:14 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Erik Staab (Gerrit)

unread,
Jan 21, 2026, 11:28:44 AM (23 hours ago) Jan 21
to Tomasz Lichoń, Dale Curtis, AyeAye, feature-me...@chromium.org
Attention needed from Tomasz Lichoń

Erik Staab voted Code-Review+1

Code-Review+1
Open in Gerrit

Related details

Attention is currently required from:
  • Tomasz Lichoń
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: I730f3892469b0d06c1303bbfca49eb640b537625
    Gerrit-Change-Number: 7485572
    Gerrit-PatchSet: 2
    Gerrit-Owner: Tomasz Lichoń <lic...@google.com>
    Gerrit-Reviewer: Dale Curtis <dalec...@chromium.org>
    Gerrit-Reviewer: Erik Staab <est...@chromium.org>
    Gerrit-Attention: Tomasz Lichoń <lic...@google.com>
    Gerrit-Comment-Date: Wed, 21 Jan 2026 16:28:34 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    open
    diffy

    Tomasz Lichoń (Gerrit)

    unread,
    5:50 AM (5 hours ago) 5:50 AM
    to Dale Curtis, AyeAye, feature-me...@chromium.org

    Tomasz Lichoń voted Commit-Queue+2

    Commit-Queue+2
    Open in Gerrit

    Related details

    Attention set is empty
    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: I730f3892469b0d06c1303bbfca49eb640b537625
    Gerrit-Change-Number: 7485572
    Gerrit-PatchSet: 2
    Gerrit-Owner: Tomasz Lichoń <lic...@google.com>
    Gerrit-Reviewer: Dale Curtis <dalec...@chromium.org>
    Gerrit-Reviewer: Erik Staab <est...@chromium.org>
    Gerrit-Reviewer: Tomasz Lichoń <lic...@google.com>
    Gerrit-Comment-Date: Thu, 22 Jan 2026 10:49:50 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    open
    diffy

    Chromium LUCI CQ (Gerrit)

    unread,
    6:45 AM (4 hours ago) 6:45 AM
    to Tomasz Lichoń, Dale Curtis, AyeAye, feature-me...@chromium.org

    Chromium LUCI CQ submitted the change

    Change information

    Commit message:
    [TDR] Remove TODOs linked to issue 41419817

    This CL was generated by an AI agent and verified by the CL owner before
    sending for review. The TODOs were identified as stale by TODO Removal
    Agent (http://go/dead-todo-removal-agent).
    * Detailed reasoning can be found in the CL comments.
    * How to review this CL: http://go/tdr-review-instructions
    * To opt-out from future reviews: http://go/tdr-opt-out.

    The associated bug crbug.com/41419817, which tracked a large-scale
    effort to add GUARDED_BY annotations, was closed as OBSOLETE. A comment
    on the bug explicitly states, "Fixed as much as we plan to," indicating
    that the work mentioned in this TODO was intentionally abandoned and is
    no longer planned.
    Change-Id: I730f3892469b0d06c1303bbfca49eb640b537625
    Bug: 41419817
    Reviewed-by: Erik Staab <est...@chromium.org>
    Commit-Queue: Tomasz Lichoń <lic...@google.com>
    Reviewed-by: Dale Curtis <dalec...@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#1572876}
    Files:
    • M media/base/pipeline_impl.cc
    Change size: XS
    Delta: 1 file changed, 0 insertions(+), 1 deletion(-)
    Branch: refs/heads/main
    Submit Requirements:
    • requirement satisfiedCode-Review: +1 by Dale Curtis, +1 by Erik Staab
    Open in Gerrit
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: merged
    Gerrit-Project: chromium/src
    Gerrit-Branch: main
    Gerrit-Change-Id: I730f3892469b0d06c1303bbfca49eb640b537625
    Gerrit-Change-Number: 7485572
    Gerrit-PatchSet: 3
    Gerrit-Owner: Tomasz Lichoń <lic...@google.com>
    Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
    open
    diffy
    satisfied_requirement
    Reply all
    Reply to author
    Forward
    0 new messages