DownloadManager::Observer::OnDownloadCreated

content/browser/download/download_item_impl_unittest.cc

Index: content/browser/download/download_item_impl_unittest.cc
diff --git a/content/browser/download/download_item_impl_unittest.cc b/content/browser/download/download_item_impl_unittest.cc
index cbc583852176f42d7d48e7e7ea50d7732b604998..8bdd159d1dcde3349f363a687afee24312ff8763 100644
--- a/content/browser/download/download_item_impl_unittest.cc
+++ b/content/browser/download/download_item_impl_unittest.cc
@@ -299,8 +299,8 @@ TEST_F(DownloadItemTest, NotificationAfterOnTargetPathDetermined) {
   DownloadItemImpl* safe_item = CreateDownloadItem(DownloadItem::IN_PROGRESS);
   MockObserver safe_observer(safe_item);
 
-  // Calling OnTargetPathDetermined does not trigger notification if danger type
-  // is NOT_DANGEROUS.
+  // Calling OnTargetPathDetermined triggers notification regardless of danger
+  // type.
   safe_item->OnTargetPathDetermined(
       FilePath(kDummyPath), DownloadItem::TARGET_DISPOSITION_OVERWRITE,
       content::DOWNLOAD_DANGER_TYPE_NOT_DANGEROUS);
@@ -315,7 +315,7 @@ TEST_F(DownloadItemTest, NotificationAfterOnTargetPathDetermined) {
   dangerous_item->OnTargetPathDetermined(
       FilePath(kDummyPath), DownloadItem::TARGET_DISPOSITION_OVERWRITE,
       content::DOWNLOAD_DANGER_TYPE_DANGEROUS_FILE);
-  EXPECT_TRUE(dangerous_observer.CheckUpdated());
+  EXPECT_FALSE(dangerous_observer.CheckUpdated());
 }
 
 TEST_F(DownloadItemTest, NotificationAfterOnTargetPathSelected) {
@@ -342,7 +342,7 @@ TEST_F(DownloadItemTest, NotificationAfterOnContentCheckCompleted) {
   EXPECT_TRUE(safe_observer.CheckUpdated());
   safe_item->OnContentCheckCompleted(
       content::DOWNLOAD_DANGER_TYPE_NOT_DANGEROUS);
-  EXPECT_FALSE(safe_observer.CheckUpdated());
+  EXPECT_TRUE(safe_observer.CheckUpdated());
 
   // Setting to unsafe url or unsafe file should trigger a notification.
   DownloadItemImpl* unsafeurl_item =