commit c1084de0791bfaaaddf542bb3f9d9a149729b255
author Dibyajyoti Pal <dibyapal@google.com> Wed Jan 24 18:22:34 2024
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Jan 24 18:22:34 2024
tree 54fd1d409b4ce9e6397b92b32706a111f65b9b62
parent aa4402cb41abfe7d6507941180f1526a87662ee7

[dPWA] DownloadedIconsHttpResults measures results per icon URL and size

Since the WebAppIconDownloader now downloads icons per icon URL and
size, this makes the result be also measured per icon URL and size. To
do this, the following changes have been made:
1. DownloadedIconsHttpResults is keyed by the tuple of GURL and
gfx::Size with a custom comparator.
2. The callsites have been updated with a default size of (0,0) to
match current behavior.
3. The declaration has been moved to web_app_install_utils.h closer to
the other data structures which helps solve a clang error about
recursive includes of main.
4. Updates IconUrlWithSize to be a struct instead of a std::tuple.

Bug: b/319669415, crbug.com/1444121, crbug.com/1225906
Change-Id: If0a70a4ac559616a67050692e42ba9c25a6e069c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5218278
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Commit-Queue: Dibyajyoti Pal <dibyapal@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1251531}