Update OffTheRecord profile APIs in notifications.

Profile API is being updated (issue 1033903) and the usage of
GetOffTheRecordProfile and HasOffTheRecordProfile without argument is
getting deprecated. Use cases of this call that ONLY refer to incognito
or guest mode can use [Get/Has]PrimaryOTRProfile() instead.

c/b/notifications/* are updated to use the new API.

Bug: 1060940
Change-Id: Ic815443fe5d7f3781578813abf4be7470128690b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2210982
Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org>
Auto-Submit: Ramin Halavati <rhalavati@chromium.org>
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#771675}

chrome/browser/notifications/notification_permission_context_unittest.cc

@@ -326,7 +326,7 @@ TEST_F(NotificationPermissionContextTest, SecureOriginRequirement) {
 // Tests auto-denial after a time delay in incognito.
 TEST_F(NotificationPermissionContextTest, TestDenyInIncognitoAfterDelay) {
   TestNotificationPermissionContext permission_context(
-      profile()->GetOffTheRecordProfile());
+      profile()->GetPrimaryOTRProfile());
   GURL url("https://www.example.com");
   NavigateAndCommit(url);
 
@@ -392,7 +392,7 @@ TEST_F(NotificationPermissionContextTest, TestDenyInIncognitoAfterDelay) {
 // Tests how multiple parallel permission requests get auto-denied in incognito.
 TEST_F(NotificationPermissionContextTest, TestParallelDenyInIncognito) {
   TestNotificationPermissionContext permission_context(
-      profile()->GetOffTheRecordProfile());
+      profile()->GetPrimaryOTRProfile());
   GURL url("https://www.example.com");
   NavigateAndCommit(url);
   web_contents()->WasShown();

chrome/browser/notifications/notification_platform_bridge_win_interactive_uitest.cc

@@ -396,7 +396,7 @@ IN_PROC_BROWSER_TEST_F(NotificationPlatformBridgeWinUITest, GetDisplayed) {
   {
     base::RunLoop run_loop;
     bridge->GetDisplayed(
-        profile1->GetOffTheRecordProfile(),
+        profile1->GetPrimaryOTRProfile(),
         base::BindOnce(
             &NotificationPlatformBridgeWinUITest::DisplayedNotifications,
             base::Unretained(this), run_loop.QuitClosure()));
@@ -422,7 +422,7 @@ IN_PROC_BROWSER_TEST_F(NotificationPlatformBridgeWinUITest, GetDisplayed) {
   {
     base::RunLoop run_loop;
     bridge->GetDisplayed(
-        profile2->GetOffTheRecordProfile(),
+        profile2->GetPrimaryOTRProfile(),
         base::BindOnce(
             &NotificationPlatformBridgeWinUITest::DisplayedNotifications,
             base::Unretained(this), run_loop.QuitClosure()));

chrome/browser/notifications/notification_trigger_scheduler.cc

@@ -75,10 +75,14 @@ void NotificationTriggerScheduler::TriggerNotifications() {
   auto profiles = g_browser_process->profile_manager()->GetLoadedProfiles();
   for (Profile* profile : profiles) {
     TriggerNotificationsForProfile(profile);
-    // Notifications are technically not supported in Incognito, but in case we
-    // ever change that lets handle these profiles too.
-    if (profile->HasOffTheRecordProfile())
-      TriggerNotificationsForProfile(profile->GetOffTheRecordProfile());
+    // Notifications are technically not supported in OffTheRecord, but in case
+    //  weever change that lets handle these profiles too.
+    if (profile->HasAnyOffTheRecordProfile()) {
+      std::vector<Profile*> otr_profiles =
+          profile->GetAllOffTheRecordProfiles();
+      for (Profile* otr : otr_profiles)
+        TriggerNotificationsForProfile(otr);
+    }
   }
 }