Skip to content

Commit

Permalink
fix: handle Notification requireInteraction option (#36477)
Browse files Browse the repository at this point in the history
Co-authored-by: Jeremy Spiegel <jeremy.spiegel@frontapp.com>
  • Loading branch information
trop[bot] and jeremyspiegel committed Dec 1, 2022
1 parent 146f8a9 commit 202d91e
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions shell/browser/notifications/platform_notification_service.cc
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,9 @@ void OnWebNotificationAllowed(base::WeakPtr<Notification> notification,
options.icon = icon;
options.silent = audio_muted ? true : data.silent;
options.has_reply = false;
if (data.require_interaction)
options.timeout_type = u"never";

notification->Show(options);
} else {
notification->Destroy();
Expand Down

0 comments on commit 202d91e

Please sign in to comment.