Message center notifier / settings cleanup.
- enable/disable notifiers by NotifierId rather than Notifier. Add notes
to rename these classes: NotifierId is a source of notifications, and
Notifier is a NotifierId plus UI-specific information.
- use URL, not display name, to track web page notifiers. URL and
display name are the same for these notifiers anyway (and may not be
valid URLs, e.g. may contain wildcards).
- Fix a typo in a class name.
Bug: 755413
Change-Id: I67dc1f4606dffd19f894445ffc124e2162843b71
Reviewed-on: https://chromium-review.googlesource.com/674026
Reviewed-by: Steven Bennetts <[email protected]>
Commit-Queue: Evan Stade <[email protected]>
Cr-Commit-Position: refs/heads/master@{#503195}diff --git a/chrome/browser/notifications/application_notifier_source.cc b/chrome/browser/notifications/application_notifier_source.cc
index 77a9a2a7..5fd578d7 100644
--- a/chrome/browser/notifications/application_notifier_source.cc
+++ b/chrome/browser/notifications/application_notifier_source.cc
@@ -62,11 +62,11 @@
void ApplicationNotifierSource::SetNotifierEnabled(
Profile* profile,
- const message_center::Notifier& notifier,
+ const message_center::NotifierId& notifier_id,
bool enabled) {
NotifierStateTrackerFactory::GetForProfile(profile)->SetNotifierEnabled(
- notifier.notifier_id, enabled);
- observer_->OnNotifierEnabledChanged(notifier.notifier_id, enabled);
+ notifier_id, enabled);
+ observer_->OnNotifierEnabledChanged(notifier_id, enabled);
}
message_center::NotifierId::NotifierType
