commit d0ea5acf1b288b4217a694511748b508823d07f4
author Elly <ellyjones@chromium.org> Thu Apr 03 15:58:54 2025
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Apr 03 15:58:54 2025
tree 4bd037e5ad9a8075958219cc52459df93c8dc1dc
parent f735d8d371dd0e4e8a90cb5c184ecb558f7a56b1

ash/quick_start: migrate from crypto/sha2 to crypto/hash

Specifically, this change:
* Drops the (test-only) deps on crypto::SHA256HashString() in favor of
  using a hardcoded client data hash;
* Adds an extra assertion to one of the unit tests that the client data
  hash is actually included when generating an assertion request;
* Has some of the message factory functions take their client data
  hashes as base::span rather than std::array (which saves a copy);
* Corrects a misuse of base::span (const span<T> just means span<T>, the
  code wanted span<const T> instead)
* Removes two redundant data copies in BuildNotifySourceOfUpdateMessage;
* Adds some `auto` where it does not reduce clarity, which makes the
  code a bit more compact

Bug: 374310081
Change-Id: Ic6c4a91da628a01805c8cc97897e83b31a539d04
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6423849
Reviewed-by: Hidehiko Abe <hidehiko@chromium.org>
Commit-Queue: Elly FJ <ellyjones@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1442192}