| commit | c3d4a4cdcdf5f12eb8badcb90b0f6b0e1577bc42 | [log] [tgz] |
|---|---|---|
| author | Toni Barzic <[email protected]> | Thu Nov 02 01:36:39 2023 |
| committer | Chromium LUCI CQ <[email protected]> | Thu Nov 02 01:36:39 2023 |
| tree | be545b8bfe7538ea82b4c5ee10e7943fc8e60ecd | |
| parent | fded49381cbafd4fd1f7ab062c6e2905c3e7e2d7 [diff] [blame] |
Pipe shortcut app badge to shelf model For app shortcuts, app items in the shelf should have a badge (the badge will be the app icon of the app that hosts the app shortcut). This updates AppIconLoaderDelegate::OnAppImageUpdated to pass the badge image in addition to the app image. The badge image is currently set for app shortcuts only. BUG=b:306294395 Change-Id: I3ef60004c924e268c94a7e28e5967aa999282239 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4987448 Reviewed-by: Scott Violet <[email protected]> Reviewed-by: Maggie Cai <[email protected]> Commit-Queue: Toni Barzic <[email protected]> Reviewed-by: Giovanni Ortuno Urquidi <[email protected]> Cr-Commit-Position: refs/heads/main@{#1218589}
diff --git a/chrome/browser/notifications/extension_notifier_controller.cc b/chrome/browser/notifications/extension_notifier_controller.cc index 7eb6050..3292b49 100644 --- a/chrome/browser/notifications/extension_notifier_controller.cc +++ b/chrome/browser/notifications/extension_notifier_controller.cc
@@ -77,7 +77,8 @@ void ExtensionNotifierController::OnAppImageUpdated( const std::string& id, - const gfx::ImageSkia& image) { + const gfx::ImageSkia& image, + const absl::optional<gfx::ImageSkia>& badge_image) { observer_->OnIconImageUpdated( message_center::NotifierId(message_center::NotifierType::APPLICATION, id), image);