Delete todo about auto-dismissing default notification after real one.

chrome/browser/push_messaging/push_messaging_notification_manager.cc

 // Copyright 2015 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/push_messaging/push_messaging_notification_manager.h"
 
 #include <bitset>
 
 #include "base/metrics/histogram_macros.h"
 #include "base/prefs/pref_service.h"
@@ skipping 135 matching lines @@
     if (requesting_origin == active_url.GetOrigin()) {
       notification_needed = false;
       break;
     }
 #if defined(OS_ANDROID)
   }
 #else
   }
 #endif
 
-  // If more than two notifications are showing for this Service Worker, close
+  // If more than one notification is showing for this Service Worker, close
   // the default notification if it happens to be part of this group.
   if (notification_count >= 2) {
     for (const auto& notification_database_data : data) {
       if (notification_database_data.notification_data.tag !=
           kPushMessagingForcedNotificationTag)
         continue;
 
       PlatformNotificationServiceImpl* platform_notification_service =
           PlatformNotificationServiceImpl::GetInstance();
 
@@ skipping 84 matching lines @@
       content::
           PUSH_USER_VISIBLE_STATUS_REQUIRED_BUT_NOT_SHOWN_GRACE_EXCEEDED);
   rappor::SampleDomainAndRegistryFromGURL(
       g_browser_process->rappor_service(),
       "PushMessaging.GenericNotificationShown.Origin",
       requesting_origin);
 
   // The site failed to show a notification when one was needed, and they have
   // already failed once in the previous 10 push messages, so we will show a
   // generic notification. See https://crbug.com/437277.
-  //
-  // TODO(johnme): The generic notification should probably automatically close
-  // itself when the next push message arrives?
   content::PlatformNotificationData notification_data;
   notification_data.title =
       url_formatter::FormatUrlForSecurityDisplayOmitScheme(
           requesting_origin,
           profile_->GetPrefs()->GetString(prefs::kAcceptLanguages));
   notification_data.direction =
       content::PlatformNotificationData::DIRECTION_LEFT_TO_RIGHT;
   notification_data.body =
       l10n_util::GetStringUTF16(IDS_PUSH_MESSAGING_GENERIC_NOTIFICATION_BODY);
   notification_data.tag = kPushMessagingForcedNotificationTag;
@@ skipping 52 matching lines @@
     return;
   }
   PlatformNotificationServiceImpl::GetInstance()->DisplayPersistentNotification(
       profile_,
       persistent_notification_id,
       requesting_origin,
       SkBitmap() /* icon */,
       notification_data);
   message_handled_closure.Run();
 }