From 797a58bee05c610ccdf8921ff0fd1d6e434f5a3f Mon Sep 17 00:00:00 2001 From: Samuel Attard Date: Fri, 22 Sep 2017 00:32:39 +1000 Subject: [PATCH] Result of RunMenuAt is now void, no need to ignore it --- atom/browser/ui/win/notify_icon.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/atom/browser/ui/win/notify_icon.cc b/atom/browser/ui/win/notify_icon.cc index adf5d1cfa68..84faba0aa2e 100644 --- a/atom/browser/ui/win/notify_icon.cc +++ b/atom/browser/ui/win/notify_icon.cc @@ -152,8 +152,8 @@ void NotifyIcon::PopUpContextMenu(const gfx::Point& pos, menu_runner_.reset(new views::MenuRunner( menu_model != nullptr ? menu_model : menu_model_, views::MenuRunner::CONTEXT_MENU | views::MenuRunner::HAS_MNEMONICS)); - ignore_result(menu_runner_->RunMenuAt( - NULL, NULL, rect, views::MENU_ANCHOR_TOPLEFT, ui::MENU_SOURCE_MOUSE)); + menu_runner_->RunMenuAt( + NULL, NULL, rect, views::MENU_ANCHOR_TOPLEFT, ui::MENU_SOURCE_MOUSE); } void NotifyIcon::SetContextMenu(AtomMenuModel* menu_model) {