Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(256)

Unified Diff: ash/system/web_notification/web_notification_tray.cc

Issue 1961793002: Show the unread button even without any unread notifications (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comment. Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/system/web_notification/web_notification_tray.cc
diff --git a/ash/system/web_notification/web_notification_tray.cc b/ash/system/web_notification/web_notification_tray.cc
index 52504eda4038fdd0656dde03ea3c0df3faf7e57f..ed35187545c7f10e27a1a7f6541b0dac44bb416b 100644
--- a/ash/system/web_notification/web_notification_tray.cc
+++ b/ash/system/web_notification/web_notification_tray.cc
@@ -155,9 +155,9 @@ class WebNotificationButton : public views::CustomButton {
private:
void UpdateIconVisibility() {
- unread_label_->SetEnabledColor(
- (!is_bubble_visible_ && unread_count_ > 0) ?
- kWebNotificationColorWithUnread : kWebNotificationColorNoUnread);
+ unread_label_->SetEnabledColor((unread_count_ > 0)
+ ? kWebNotificationColorWithUnread
+ : kWebNotificationColorNoUnread);
SchedulePaint();
}
@@ -454,7 +454,7 @@ void WebNotificationTray::UpdateTrayContent() {
SetVisible((status_area_widget()->login_status() != user::LOGGED_IN_NONE) &&
(status_area_widget()->login_status() != user::LOGGED_IN_LOCKED) &&
- !userAddingRunning && (message_center->NotificationCount() > 0));
+ !userAddingRunning);
Layout();
SchedulePaint();
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698