Fix the settings button and UMA metrics for the message center

chrome/browser/notifications/web_notification_delegate.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/notifications/web_notification_delegate.h"
 
 #include "base/feature_list.h"
 #include "base/metrics/histogram_macros.h"
 #include "chrome/browser/notifications/notification_common.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 24 matching lines @@
     : browser_context_(browser_context),
       notification_id_(notification_id),
       origin_(origin) {}
 
 WebNotificationDelegate::~WebNotificationDelegate() {}
 
 std::string WebNotificationDelegate::id() const {
   return notification_id_;
 }
 
-void WebNotificationDelegate::SettingsClick() {
+bool WebNotificationDelegate::SettingsClick() {
+#if !defined(OS_CHROMEOS)
   NotificationCommon::OpenNotificationSettings(browser_context_);
+  return true;
+#else
+  return false;
+#endif
 }
 
 bool WebNotificationDelegate::ShouldDisplaySettingsButton() {
   return true;
 }
 
 bool WebNotificationDelegate::ShouldDisplayOverFullscreen() const {
 #if !defined(OS_ANDROID)
   Profile* profile = Profile::FromBrowserContext(browser_context_);
   DCHECK(profile);
@@ skipping 29 matching lines @@
             NotifierId::WEB_PAGE,
             NotifierId::SIZE);
       }
       return enabled;
     }
   }
 #endif
 
   return false;
 }