Introduces the notifier id for multi-profile notifications.

chrome/browser/chromeos/login/multi_profile_first_run_notification.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/login/multi_profile_first_run_notification.h"
 
+#include "ash/system/system_notifier.h"
 #include "base/prefs/pref_service.h"
 #include "base/strings/string16.h"
 #include "chrome/browser/chromeos/login/user_manager.h"
 #include "chrome/browser/prefs/pref_service_syncable.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/pref_names.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/message_center/message_center.h"
 #include "ui/message_center/notification.h"
 #include "ui/message_center/notification_delegate.h"
 
 using message_center::Notification;
 
 namespace chromeos {
 
 namespace {
 
 const char kNotificationId[] = "chrome:://login/multiprofile";
 
+class MultiProfileFirstRunNotificationDelegate
+    : public message_center::NotificationDelegate {
+ public:
+  MultiProfileFirstRunNotificationDelegate(base::Closure callback)

[xiyuan, 2013/12/09 19:54:01]

nit: const base::Closure&

[stevenjb, 2013/12/09 20:18:52]

explicit
Also, maybe name 'user_close_callback'?

[Jun Mukai, 2013/12/09 21:23:43]

Done.

[Jun Mukai, 2013/12/09 21:23:43]

Done.

+      : callback_(callback) {}
+
+  // Overridden from message_center::NotificationDelegate:
+  virtual void Display() OVERRIDE {}
+  virtual void Error() OVERRIDE {}
+  virtual void Close(bool by_user) OVERRIDE {
+    if (by_user)
+      callback_.Run();
+  }
+  virtual void Click() OVERRIDE {}
+
+ protected:
+  virtual ~MultiProfileFirstRunNotificationDelegate() {}
+
+ private:
+  base::Closure callback_;
+
+  DISALLOW_COPY_AND_ASSIGN(MultiProfileFirstRunNotificationDelegate);
+};
+
 }  // namespace
 
 MultiProfileFirstRunNotification::MultiProfileFirstRunNotification()
     : weak_ptr_factory_(this) {}
 
 MultiProfileFirstRunNotification::~MultiProfileFirstRunNotification() {}
 
 // static
 void MultiProfileFirstRunNotification::RegisterProfilePrefs(
     user_prefs::PrefRegistrySyncable* registry) {
@@ skipping 15 matching lines @@
   const base::string16 title;
   const base::string16 display_source;
   scoped_ptr<Notification> notification(new Notification(
       message_center::NOTIFICATION_TYPE_SIMPLE,
       kNotificationId,
       title,
       l10n_util::GetStringUTF16(IDS_MULTI_PROFILES_WARNING),
       ui::ResourceBundle::GetSharedInstance().GetImageNamed(
           IDR_NOTIFICATION_ALERT),
       display_source,
-      message_center::NotifierId(),
+      message_center::NotifierId(
+          message_center::NotifierId::SYSTEM_COMPONENT,
+          ash::system_notifier::kNotifierMultiProfileFirstRun),
       message_center::RichNotificationData(),
-      new message_center::HandleNotificationClickedDelegate(
+      new MultiProfileFirstRunNotificationDelegate(
           base::Bind(&MultiProfileFirstRunNotification::OnDismissed,
                      weak_ptr_factory_.GetWeakPtr(),
                      user_profile))));
   notification->SetSystemPriority();
   message_center::MessageCenter::Get()->AddNotification(notification.Pass());
 }
 
 void MultiProfileFirstRunNotification::OnDismissed(Profile* user_profile) {
   user_profile->GetPrefs()->SetBoolean(
       prefs::kMultiProfileNotificationDismissed, true);
 }
 
 }  // namespace chromeos