Skip to content

Commit

Permalink
[WebAPK] Loosen min icon size restriction for shortcuts.
Browse files Browse the repository at this point in the history
Currently the minimum size matches the ideal size. Make the minimum size
in px equal to half the the ideal size.

Change-Id: Ib5a68f5e0073ced276afa095ded61ce7c264573d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2055103
Auto-Submit: Rayan Kanso <[email protected]>
Commit-Queue: Dominick Ng <[email protected]>
Reviewed-by: Dominick Ng <[email protected]>
Cr-Commit-Position: refs/heads/master@{#741253}
  • Loading branch information
rayankans authored and Commit Bot committed Feb 13, 2020
1 parent 1c76cd0 commit 5850be5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion chrome/browser/android/shortcut_info.cc
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ void ShortcutInfo::UpdateFromManifest(const blink::Manifest& manifest) {
for (const auto& manifest_shortcut : shortcut_items) {
GURL best_url = blink::ManifestIconSelector::FindBestMatchingSquareIcon(
manifest_shortcut.icons, ideal_shortcut_icons_size_px,
ideal_shortcut_icons_size_px,
/* minimum_icon_size_in_px= */ ideal_shortcut_icons_size_px / 2,
blink::Manifest::ImageResource::Purpose::ANY);
best_shortcut_icon_urls.push_back(std::move(best_url));
}
Expand Down

0 comments on commit 5850be5

Please sign in to comment.