mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

chore: remove unused Notification fields icon_path_, has_icon_ (#43286)

Browse source 
Last use of `icon_path_` was removed on May 29, 2017 (c741b584)
Last use of `has_icon_` was removed on May 30, 2017 (5048425e)
This commit is contained in:
Charles Kerr 2024-08-12 02:55:54 -05:00 committed by GitHub
parent 6d13c503c5
commit bff298987f
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 1 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
shell/browser/api/electron_api_notification.cc
View file

@ -58,10 +58,7 @@ Notification::Notification(gin::Arguments* args) {
opts.Get("title", &title_);
opts.Get("subtitle", &subtitle_);
opts.Get("body", &body_);
has_icon_ = opts.Get("icon", &icon_);
if (has_icon_) {
opts.Get("icon", &icon_path_);
}
opts.Get("icon", &icon_);
opts.Get("silent", &silent_);
opts.Get("replyPlaceholder", &reply_placeholder_);
opts.Get("urgency", &urgency_);

2
shell/browser/api/electron_api_notification.h
View file

@ -103,8 +103,6 @@ class Notification : public gin::Wrappable<Notification>,
std::u16string subtitle_;
std::u16string body_;
gfx::Image icon_;
std::u16string icon_path_;
bool has_icon_ = false;
bool silent_ = false;
bool has_reply_ = false;
std::u16string timeout_type_;