Commit bc167c7a authored by Tim Song's avatar Tim Song Committed by Commit Bot

Notifications: Close locale changed notification on click.

Currently, clicking on this notification marks the locale change as accepted,
but does not dismiss the notification. When the user interacts with the
notification again, it does not work.

TEST=manual
BUG=937185

Change-Id: Ib510f78d6695bf2916678673a74198df605e5900
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1693682Reviewed-by: default avatarTetsui Ohkubo <tetsui@chromium.org>
Commit-Queue: Tim Song <tengs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#676167}
parent 6a6d6363
...@@ -75,6 +75,9 @@ void LocaleNotificationDelegate::Click( ...@@ -75,6 +75,9 @@ void LocaleNotificationDelegate::Click(
std::move(callback_).Run(button_index ? LocaleNotificationResult::kRevert std::move(callback_).Run(button_index ? LocaleNotificationResult::kRevert
: LocaleNotificationResult::kAccept); : LocaleNotificationResult::kAccept);
message_center::MessageCenter::Get()->RemoveNotification(
kLocaleChangeNotificationId, true /* by_user */);
} }
} // namespace } // namespace
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment