Commit f7d02d63 authored by yhanada's avatar yhanada Committed by Commit bot

Open the settings tab in the message center when clicking the settings button on a notification.

Before this change, clicking the settings button on a notification
opens the notification section in the settings
window (chrome://settings)

BUG=706658

Review-Url: https://codereview.chromium.org/2799653004
Cr-Commit-Position: refs/heads/master@{#462815}
parent 7b9f72c2
...@@ -474,7 +474,7 @@ void MessageCenterView::ClickOnNotificationButton( ...@@ -474,7 +474,7 @@ void MessageCenterView::ClickOnNotificationButton(
void MessageCenterView::ClickOnSettingsButton( void MessageCenterView::ClickOnSettingsButton(
const std::string& notification_id) { const std::string& notification_id) {
message_center_->ClickOnSettingsButton(notification_id); tray_->ShowNotifierSettingsBubble();
} }
void MessageCenterView::UpdateNotificationSize( void MessageCenterView::UpdateNotificationSize(
......
...@@ -122,7 +122,7 @@ void MessagePopupCollection::ClickOnNotificationButton( ...@@ -122,7 +122,7 @@ void MessagePopupCollection::ClickOnNotificationButton(
void MessagePopupCollection::ClickOnSettingsButton( void MessagePopupCollection::ClickOnSettingsButton(
const std::string& notification_id) { const std::string& notification_id) {
message_center_->ClickOnSettingsButton(notification_id); tray_->ShowNotifierSettingsBubble();
} }
void MessagePopupCollection::UpdateNotificationSize( void MessagePopupCollection::UpdateNotificationSize(
......
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