Remove base/hash/md5.h [chromium/src : main]

0 views
Skip to first unread message

Giovanni Ortuno Urquidi (Gerrit)

unread,
Nov 3, 2025, 2:03:07 PM (7 days ago) Nov 3
to Giovanni Ortuno Urquidi, Elly FJ, Chromium LUCI CQ, chromium...@chromium.org, asvitki...@chromium.org, chromeos-gfx-...@google.com, chromiumme...@microsoft.com, feature-me...@chromium.org, media-cro...@chromium.org, net-r...@chromium.org, oshima...@chromium.org, rrsilva+wat...@google.com, spang...@chromium.org, tbarzi...@chromium.org, tracing...@chromium.org, wfh+...@chromium.org
Attention needed from Elly FJ

Giovanni Ortuno Urquidi added 1 comment

Patchset-level comments
File-level comment, Patchset 1 (Latest):
Giovanni Ortuno Urquidi . resolved

ellyjones: PTAL! :)

Open in Gerrit

Related details

Attention is currently required from:
  • Elly FJ
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: Ie643451d670c0ab701cddf2b3e820d50debd571e
Gerrit-Change-Number: 7113738
Gerrit-PatchSet: 1
Gerrit-Owner: Giovanni Ortuno Urquidi <ort...@chromium.org>
Gerrit-Reviewer: Elly FJ <elly...@chromium.org>
Gerrit-Reviewer: Giovanni Ortuno Urquidi <ort...@chromium.org>
Gerrit-Attention: Elly FJ <elly...@chromium.org>
Gerrit-Comment-Date: Mon, 03 Nov 2025 19:03:02 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy

Elly FJ (Gerrit)

unread,
Nov 3, 2025, 3:23:50 PM (7 days ago) Nov 3
to Giovanni Ortuno Urquidi, Elly FJ, Chromium LUCI CQ, chromium...@chromium.org, asvitki...@chromium.org, chromeos-gfx-...@google.com, chromiumme...@microsoft.com, feature-me...@chromium.org, media-cro...@chromium.org, net-r...@chromium.org, oshima...@chromium.org, rrsilva+wat...@google.com, spang...@chromium.org, tbarzi...@chromium.org, tracing...@chromium.org, wfh+...@chromium.org

Elly FJ voted and added 1 comment

Votes added by Elly FJ

Code-Review+1

1 comment

Patchset-level comments
Elly FJ . resolved

lgtm, thank you!

Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
    • requirement satisfiedCode-Coverage
    • requirement is not 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: Ie643451d670c0ab701cddf2b3e820d50debd571e
    Gerrit-Change-Number: 7113738
    Gerrit-PatchSet: 1
    Gerrit-Owner: Giovanni Ortuno Urquidi <ort...@chromium.org>
    Gerrit-Reviewer: Elly FJ <elly...@chromium.org>
    Gerrit-Reviewer: Giovanni Ortuno Urquidi <ort...@chromium.org>
    Gerrit-Comment-Date: Mon, 03 Nov 2025 20:23:41 +0000
    Gerrit-HasComments: Yes
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    unsatisfied_requirement
    open
    diffy

    Giovanni Ortuno Urquidi (Gerrit)

    unread,
    Nov 4, 2025, 11:24:11 AM (6 days ago) Nov 4
    to Giovanni Ortuno Urquidi, Nico Weber, Elly FJ, Chromium LUCI CQ, chromium...@chromium.org, asvitki...@chromium.org, chromeos-gfx-...@google.com, chromiumme...@microsoft.com, feature-me...@chromium.org, media-cro...@chromium.org, net-r...@chromium.org, oshima...@chromium.org, rrsilva+wat...@google.com, spang...@chromium.org, tbarzi...@chromium.org, tracing...@chromium.org, wfh+...@chromium.org
    Attention needed from Nico Weber

    Giovanni Ortuno Urquidi added 1 comment

    Giovanni Ortuno Urquidi . resolved

    thakis: Mind taking a look at this CL to remove a file that's no longer used?

    Open in Gerrit

    Related details

    Attention is currently required from:
    • Nico Weber
    Submit Requirements:
    • requirement satisfiedCode-Coverage
    • requirement is not 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: Ie643451d670c0ab701cddf2b3e820d50debd571e
    Gerrit-Change-Number: 7113738
    Gerrit-PatchSet: 2
    Gerrit-Owner: Giovanni Ortuno Urquidi <ort...@chromium.org>
    Gerrit-Reviewer: Elly FJ <elly...@chromium.org>
    Gerrit-Reviewer: Giovanni Ortuno Urquidi <ort...@chromium.org>
    Gerrit-Reviewer: Nico Weber <tha...@chromium.org>
    Gerrit-Attention: Nico Weber <tha...@chromium.org>
    Gerrit-Comment-Date: Tue, 04 Nov 2025 16:24:04 +0000
    Gerrit-HasComments: Yes
    Gerrit-Has-Labels: No
    satisfied_requirement
    unsatisfied_requirement
    open
    diffy

    Nico Weber (Gerrit)

    unread,
    Nov 4, 2025, 12:40:46 PM (6 days ago) Nov 4
    to Giovanni Ortuno Urquidi, Nico Weber, Elly FJ, Chromium LUCI CQ, chromium...@chromium.org, asvitki...@chromium.org, chromeos-gfx-...@google.com, chromiumme...@microsoft.com, feature-me...@chromium.org, media-cro...@chromium.org, net-r...@chromium.org, oshima...@chromium.org, rrsilva+wat...@google.com, spang...@chromium.org, tbarzi...@chromium.org, tracing...@chromium.org, wfh+...@chromium.org
    Attention needed from Giovanni Ortuno Urquidi

    Nico Weber voted

    Code-Review+1
    Owners-Override+1
    Open in Gerrit

    Related details

    Attention is currently required from:
    • Giovanni Ortuno Urquidi
    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: Ie643451d670c0ab701cddf2b3e820d50debd571e
    Gerrit-Change-Number: 7113738
    Gerrit-PatchSet: 2
    Gerrit-Owner: Giovanni Ortuno Urquidi <ort...@chromium.org>
    Gerrit-Reviewer: Elly FJ <elly...@chromium.org>
    Gerrit-Reviewer: Giovanni Ortuno Urquidi <ort...@chromium.org>
    Gerrit-Reviewer: Nico Weber <tha...@chromium.org>
    Gerrit-Attention: Giovanni Ortuno Urquidi <ort...@chromium.org>
    Gerrit-Comment-Date: Tue, 04 Nov 2025 17:40:40 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    open
    diffy

    Giovanni Ortuno Urquidi (Gerrit)

    unread,
    Nov 4, 2025, 12:41:09 PM (6 days ago) Nov 4
    to Giovanni Ortuno Urquidi, Nico Weber, Elly FJ, Chromium LUCI CQ, chromium...@chromium.org, asvitki...@chromium.org, chromeos-gfx-...@google.com, chromiumme...@microsoft.com, feature-me...@chromium.org, media-cro...@chromium.org, net-r...@chromium.org, oshima...@chromium.org, rrsilva+wat...@google.com, spang...@chromium.org, tbarzi...@chromium.org, tracing...@chromium.org, wfh+...@chromium.org

    Giovanni Ortuno Urquidi voted and added 1 comment

    Votes added by Giovanni Ortuno Urquidi

    Commit-Queue+2

    1 comment

    Patchset-level comments
    Giovanni Ortuno Urquidi . resolved

    thanks!

    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: Ie643451d670c0ab701cddf2b3e820d50debd571e
    Gerrit-Change-Number: 7113738
    Gerrit-PatchSet: 2
    Gerrit-Owner: Giovanni Ortuno Urquidi <ort...@chromium.org>
    Gerrit-Reviewer: Elly FJ <elly...@chromium.org>
    Gerrit-Reviewer: Giovanni Ortuno Urquidi <ort...@chromium.org>
    Gerrit-Reviewer: Nico Weber <tha...@chromium.org>
    Gerrit-Comment-Date: Tue, 04 Nov 2025 17:41:03 +0000
    Gerrit-HasComments: Yes
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    open
    diffy

    Chromium LUCI CQ (Gerrit)

    unread,
    Nov 4, 2025, 12:44:33 PM (6 days ago) Nov 4
    to Giovanni Ortuno Urquidi, Nico Weber, Elly FJ, chromium...@chromium.org, asvitki...@chromium.org, chromeos-gfx-...@google.com, chromiumme...@microsoft.com, feature-me...@chromium.org, media-cro...@chromium.org, net-r...@chromium.org, oshima...@chromium.org, rrsilva+wat...@google.com, spang...@chromium.org, tbarzi...@chromium.org, tracing...@chromium.org, wfh+...@chromium.org

    Chromium LUCI CQ submitted the change

    Change information

    Commit message:
    Remove base/hash/md5.h

    All usages have been migrated to crypto/obsolete/md5.h or upgraded to
    SHA-256.
    Fixed: 406729261
    Change-Id: Ie643451d670c0ab701cddf2b3e820d50debd571e
    Reviewed-by: Elly FJ <elly...@chromium.org>
    Commit-Queue: Giovanni Ortuno Urquidi <ort...@chromium.org>
    Reviewed-by: Nico Weber <tha...@chromium.org>
    Owners-Override: Nico Weber <tha...@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#1540135}
    Files:
    • M base/BUILD.gn
    • D base/hash/md5.cc
    • D base/hash/md5.h
    • D base/hash/md5_unittest.cc
    • M chrome/browser/ash/login/reporting/login_logout_reporter.cc
    • M chrome/browser/metrics/chromeos_metrics_provider.cc
    • M content/browser/tracing/tracing_scenario.cc
    • M media/gpu/chromeos/image_processor_test.cc
    • M net/http/http_auth_handler_digest.cc
    • M net/tools/cachetool/cachetool.cc
    Change size: M
    Delta: 10 files changed, 0 insertions(+), 202 deletions(-)
    Branch: refs/heads/main
    Submit Requirements:
    • requirement satisfiedCode-Review: +1 by Nico Weber, +1 by Elly FJ
    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: Ie643451d670c0ab701cddf2b3e820d50debd571e
    Gerrit-Change-Number: 7113738
    Gerrit-PatchSet: 3
    Gerrit-Owner: Giovanni Ortuno Urquidi <ort...@chromium.org>
    Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
    Gerrit-Reviewer: Elly FJ <elly...@chromium.org>
    Gerrit-Reviewer: Giovanni Ortuno Urquidi <ort...@chromium.org>
    Gerrit-Reviewer: Nico Weber <tha...@chromium.org>
    open
    diffy
    satisfied_requirement
    Reply all
    Reply to author
    Forward
    0 new messages