Fix the settings button and UMA metrics for the message center
The settings button was unintentionally broken for Windows and
Linux users in 2799653004, as was gathering of some UMA. This
CL offers a slight refactoring that meets all goals.
BUG=706658
Review-Url: https://codereview.chromium.org/2807623002
Cr-Commit-Position: refs/heads/master@{#463339}
diff --git a/chrome/browser/notifications/web_notification_delegate.h b/chrome/browser/notifications/web_notification_delegate.h
index 78aaa6b1c..d4629fc6 100644
--- a/chrome/browser/notifications/web_notification_delegate.h
+++ b/chrome/browser/notifications/web_notification_delegate.h
@@ -31,7 +31,7 @@
public:
// NotificationDelegate implementation.
std::string id() const override;
- void SettingsClick() override;
+ bool SettingsClick() override;
bool ShouldDisplaySettingsButton() override;
bool ShouldDisplayOverFullscreen() const override;
