//ui/message_center: make NotifierType a scoped enum
Also get rid of the SIZE placeholder constant, since it requires
additional case statements in switches. Define kMaxValue instead: clang
enforces the correct definition of this value, and the UMA enumeration
histograms can take advantage of it without any weird casting.
Bug: 742517
Tbr: [email protected]
Tbr: [email protected]
Change-Id: I34373733f7be363aa1d70827ae88dc03c6934819
Reviewed-on: https://chromium-review.googlesource.com/c/1312685
Commit-Queue: Daniel Cheng <[email protected]>
Reviewed-by: Peter Beverloo <[email protected]>
Cr-Commit-Position: refs/heads/master@{#605831}diff --git a/chrome/browser/notifications/extension_notifier_controller.cc b/chrome/browser/notifications/extension_notifier_controller.cc
index 8e25a4a0..9c0e575 100644
--- a/chrome/browser/notifications/extension_notifier_controller.cc
+++ b/chrome/browser/notifications/extension_notifier_controller.cc
@@ -50,7 +50,7 @@
continue;
message_center::NotifierId notifier_id(
- message_center::NotifierId::APPLICATION, extension->id());
+ message_center::NotifierType::APPLICATION, extension->id());
NotifierStateTracker* const notifier_state_tracker =
NotifierStateTrackerFactory::GetForProfile(profile);
ui_data.push_back(ash::mojom::NotifierUiData::New(
@@ -76,6 +76,6 @@
const std::string& id,
const gfx::ImageSkia& image) {
observer_->OnIconImageUpdated(
- message_center::NotifierId(message_center::NotifierId::APPLICATION, id),
+ message_center::NotifierId(message_center::NotifierType::APPLICATION, id),
image);
}
