New issue
Advanced search Search tips

Issue 780342 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

change notification settings button enum into boolean

Project Member Reported by est...@chromium.org, Nov 1 2017

Issue description

We should be able to remove the TRAY value from the SettingsButtonHandler by making all delegates responsible for handling settings button clicks. Then we can make the enum a boolean.

This will be easier after all client notifications on chrome os are sent through mojo. AshMessageCenterController creates its own NotificationDelegate for these, which can just tell status_area_widget->web_notification_tray() to show the settings.
 

Comment 1 by tetsui@chromium.org, Nov 24 2017

Cc: tetsui@chromium.org

Sign in to add a comment