Don't show the group notification when there is only one in-progress download

chrome/browser/download/notification/download_group_notification.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/download/notification/download_group_notification.h"
 
 #include <algorithm>
 
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 79 matching lines @@
     content::DownloadItem* download) {
   if (items_.find(download) != items_.end()) {
     Update();
   }
 }
 
 void DownloadGroupNotification::OnDownloadAdded(
     content::DownloadItem* download) {
   if (items_.find(download) == items_.end()) {
     items_.insert(download);
+    int inprogress_download_count = 0;
     // If new download is started and there are more than 2 downloads in total,
     // show the group notification.
-    if (items_.size() >= 2)
-      Show();
+    for (auto it = items_.begin(); it != items_.end(); it++) {

[asanka, 2015/08/04 21:55:09]

for (const auto& item : items_) {
  if (!item->IsDone() && ++inprogress_download_count == 2) {
    Show();
    break;
  }
}

[yoshiki, 2015/08/05 09:09:31]

Done.

+      if (!(*it)->IsDone()) {
+        if (++inprogress_download_count >= 2) {
+          Show();
+          break;
+        }
+      }
+    }
   }
 }
 
 void DownloadGroupNotification::OnDownloadRemoved(
     content::DownloadItem* download) {
   // The given |download| may be already free'd.
   if (items_.find(download) != items_.end()) {
     items_.erase(download);
     if (items_.size() <= 1)
       Hide();
@@ skipping 88 matching lines @@
   chrome::ScopedTabbedBrowserDisplayer browser_displayer(
       profile_, chrome::GetActiveDesktop());
   Browser* browser = browser_displayer.browser();
   DCHECK(browser);
 
   browser->OpenURL(content::OpenURLParams(
       GURL(chrome::kChromeUIDownloadsURL), content::Referrer(),
       NEW_FOREGROUND_TAB, ui::PAGE_TRANSITION_LINK,
       false /* is_renderer_initiated */));
 }