Passing ProfileID instead of Profile* to clarify that profile should not be used for making any cal…

chrome/browser/notifications/profile_notification.h

Index: chrome/browser/notifications/profile_notification.h
diff --git a/chrome/browser/notifications/profile_notification.h b/chrome/browser/notifications/profile_notification.h
index 13952333e2ec142ee3496ea563793f6974cc4eba..1a778c805ba4cce72e9eec46d376e6604a64b6e1 100644
--- a/chrome/browser/notifications/profile_notification.h
+++ b/chrome/browser/notifications/profile_notification.h
@@ -21,15 +21,15 @@ class ProfileNotification {
   static std::string GetProfileNotificationId(const std::string& delegate_id,
                                               ProfileID profile_id);
 
-  ProfileNotification(Profile* profile, const Notification& notification);
+  ProfileNotification(ProfileID profile_id, const Notification& notification);
   ~ProfileNotification();
 
-  Profile* profile() const { return profile_; }
+  ProfileID profile() const { return profile_; }

[dewittj, 2015/05/22 16:54:37]

nit: change to profile_id()

[Deepak, 2015/05/23 06:19:11]

Done.

   const Notification& notification() const { return notification_; }
 
  private:
   // Weak, guaranteed not to be used after profile removal by parent class.

[dewittj, 2015/05/22 16:54:37]

please update this comment to reflect that it's just used as an equality
comparison.

[Deepak, 2015/05/23 06:19:11]

Done.

-  Profile* profile_;
+  ProfileID profile_;

[dewittj, 2015/05/22 16:54:37]

nit: change to profile_id_

[Deepak, 2015/05/23 06:19:11]

Done.

 
   Notification notification_;
 };