Disable DownloadNotificationTest suite on CrOS

chrome/browser/download/notification/download_notification_browsertest.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 "base/command_line.h"
 #include "base/message_loop/message_loop.h"
 #include "base/path_service.h"
 #include "base/run_loop.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/chromeos/profiles/profile_helper.h"
@@ skipping 367 matching lines @@
     return DownloadPrefs::FromDownloadManager(GetDownloadManager(browser()))->
       DownloadPath();
   }
 
  private:
   content::DownloadItem* download_item_ = nullptr;
   Browser* incognito_browser_ = nullptr;
   std::string notification_id_;
 };
 
-IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, DownloadFile) {
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_DownloadFile DISABLED_DownloadFile
+#else
+#define MAYBE_DownloadFile DownloadFile
+#endif
+IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, MAYBE_DownloadFile) {
   CreateDownload();
 
   EXPECT_EQ(l10n_util::GetStringFUTF16(
                 IDS_DOWNLOAD_STATUS_IN_PROGRESS_TITLE,
                 download_item()->GetFileNameToReportUser().LossyDisplayName()),
             GetNotification(notification_id())->title());
   EXPECT_EQ(message_center::NOTIFICATION_TYPE_PROGRESS,
             GetNotification(notification_id())->type());
 
   // Confirms that the download update is delivered to the notification.
@@ skipping 31 matching lines @@
   GetMessageCenter()->SetVisibility(message_center::VISIBILITY_MESSAGE_CENTER);
 
   // Try to open the downloaded item by clicking the notification.
   EXPECT_FALSE(GetDownloadManagerDelegate()->opened());
   GetMessageCenter()->ClickOnNotification(notification_id());
   EXPECT_TRUE(GetDownloadManagerDelegate()->opened());
 
   EXPECT_FALSE(GetNotification(notification_id()));
 }
 
-IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, DownloadDangerousFile) {
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_DownloadDangerousFile DISABLED_DownloadDangerousFile
+#else
+#define MAYBE_DownloadDangerousFile DownloadDangerousFile
+#endif
+IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, MAYBE_DownloadDangerousFile) {
   GURL download_url(embedded_test_server()->GetURL(
       "/downloads/dangerous/dangerous.swf"));
 
   content::DownloadTestObserverTerminal download_terminal_observer(
       GetDownloadManager(browser()),
       1u,  /* wait_count */
       content::DownloadTestObserver::ON_DANGEROUS_DOWNLOAD_IGNORE);
 
   CreateDownloadForBrowserAndURL(browser(), download_url);
 
@@ skipping 30 matching lines @@
   download_terminal_observer.WaitForFinished();
 
   // Checks the download status.
   EXPECT_FALSE(download_item()->IsDangerous());
   EXPECT_EQ(content::DownloadItem::COMPLETE, download_item()->GetState());
 
   // Checks the downloaded file.
   EXPECT_TRUE(base::PathExists(GetDownloadPath().Append(filename.BaseName())));
 }
 
-IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, DiscardDangerousFile) {
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_DiscardDangerousFile DISABLED_DiscardDangerousFile
+#else
+#define MAYBE_DiscardDangerousFile DiscardDangerousFile
+#endif
+IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, MAYBE_DiscardDangerousFile) {
   GURL download_url(embedded_test_server()->GetURL(
       "/downloads/dangerous/dangerous.swf"));
 
   content::DownloadTestObserverTerminal download_terminal_observer(
       GetDownloadManager(browser()),
       1u,  /* wait_count */
       content::DownloadTestObserver::ON_DANGEROUS_DOWNLOAD_IGNORE);
 
   CreateDownloadForBrowserAndURL(browser(), download_url);
 
@@ skipping 28 matching lines @@
   // Checks there is neither any download nor any notification.
   EXPECT_EQ(0u, GetMessageCenter()->GetVisibleNotifications().size());
   std::vector<content::DownloadItem*> downloads;
   GetDownloadManager(browser())->GetAllDownloads(&downloads);
   EXPECT_EQ(0u, downloads.size());
 
   // Checks the downloaded file doesn't exist.
   EXPECT_FALSE(base::PathExists(GetDownloadPath().Append(filename.BaseName())));
 }
 
-IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, DownloadImageFile) {
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_DownloadImageFile DISABLED_DownloadImageFile
+#else
+#define MAYBE_DownloadImageFile DownloadImageFile
+#endif
+IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, MAYBE_DownloadImageFile) {
   GURL download_url(embedded_test_server()->GetURL(
       "/downloads/image-octet-stream.png"));
 
   content::DownloadTestObserverTerminal download_terminal_observer(
       GetDownloadManager(browser()), 1u, /* wait_count */
       content::DownloadTestObserver::ON_DANGEROUS_DOWNLOAD_IGNORE);
 
   CreateDownloadForBrowserAndURL(browser(), download_url);
 
   // Wait for the download completion.
   download_terminal_observer.WaitForFinished();
 
   // Waits for download completion.
   NotificationUpdateObserver
       download_change_notification_observer(notification_id());
   while (GetNotification(notification_id())->image().IsEmpty()) {
     download_change_notification_observer.Wait();
     download_change_notification_observer.Reset();
   }
 }
 
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_CloseNotificationAfterDownload DISABLED_CloseNotificationAfterDownload
+#else
+#define MAYBE_CloseNotificationAfterDownload CloseNotificationAfterDownload
+#endif
 IN_PROC_BROWSER_TEST_F(DownloadNotificationTest,
-                       CloseNotificationAfterDownload) {
+                       MAYBE_CloseNotificationAfterDownload) {
   CreateDownload();
 
   // Requests to complete the download.
   ui_test_utils::NavigateToURL(
       browser(), GURL(net::URLRequestSlowDownloadJob::kFinishDownloadUrl));
 
   // Waits for download completion.
   NotificationUpdateObserver
       download_change_notification_observer(notification_id());
   while (download_item()->GetState() != content::DownloadItem::COMPLETE) {
@@ skipping 11 matching lines @@
 
   EXPECT_EQ(0u, GetMessageCenter()->GetVisibleNotifications().size());
 
   // Confirms that a download is also started.
   std::vector<content::DownloadItem*> downloads;
   GetDownloadManager(browser())->GetAllDownloads(&downloads);
   EXPECT_EQ(1u, downloads.size());
   EXPECT_EQ(content::DownloadItem::COMPLETE, downloads[0]->GetState());
 }
 
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_CloseNotificationWhileDownloading DISABLED_CloseNotificationWhileDownloading
+#else
+#define MAYBE_CloseNotificationWhileDownloading CloseNotificationWhileDownloading
+#endif
 IN_PROC_BROWSER_TEST_F(DownloadNotificationTest,
-                       CloseNotificationWhileDownloading) {
+                       MAYBE_CloseNotificationWhileDownloading) {
   CreateDownload();
 
   // Closes the notification.
   NotificationRemoveObserver notification_close_observer;
   GetMessageCenter()->RemoveNotification(notification_id(), true /* by_user */);
   EXPECT_EQ(notification_id(), notification_close_observer.Wait());
 
   EXPECT_EQ(0u, GetMessageCenter()->GetVisibleNotifications().size());
 
   // Confirms that a download is still in progress.
@@ skipping 18 matching lines @@
   // Waits that new notification.
   download_notification_add_observer.Wait();
 
   // Confirms that there is only one notification.
   message_center::NotificationList::Notifications
       visible_notifications = GetMessageCenter()->GetVisibleNotifications();
   EXPECT_EQ(1u, visible_notifications.size());
   EXPECT_TRUE(IsInNotifications(visible_notifications, notification_id()));
 }
 
-IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, InterruptDownload) {
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_InterruptDownload DISABLED_InterruptDownload
+#else
+#define MAYBE_InterruptDownload InterruptDownload
+#endif
+IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, MAYBE_InterruptDownload) {
   CreateDownload();
 
   // Installs observers before requesting.
   NotificationUpdateObserver
       download_notification_update_observer(notification_id());
   content::DownloadTestObserverTerminal download_terminal_observer(
       GetDownloadManager(browser()),
       1u, /* wait_count */
       content::DownloadTestObserver::ON_DANGEROUS_DOWNLOAD_FAIL);
 
@@ skipping 19 matching lines @@
   EXPECT_NE(GetNotification(notification_id())->message().find(
                 l10n_util::GetStringFUTF16(
                     IDS_DOWNLOAD_STATUS_INTERRUPTED,
                     l10n_util::GetStringUTF16(
                         IDS_DOWNLOAD_INTERRUPTED_DESCRIPTION_NETWORK_ERROR))),
             std::string::npos);
   EXPECT_EQ(message_center::NOTIFICATION_TYPE_BASE_FORMAT,
             GetNotification(notification_id())->type());
 }
 
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_InterruptDownloadAfterClosingNotification DISABLED_InterruptDownloadAfterClosingNotification
+#else
+#define MAYBE_InterruptDownloadAfterClosingNotification InterruptDownloadAfterClosingNotification
+#endif
 IN_PROC_BROWSER_TEST_F(DownloadNotificationTest,
-                       InterruptDownloadAfterClosingNotification) {
+                       MAYBE_InterruptDownloadAfterClosingNotification) {
   CreateDownload();
 
   // Closes the notification.
   NotificationRemoveObserver notification_close_observer;
   GetMessageCenter()->RemoveNotification(notification_id(), true /* by_user */);
   EXPECT_EQ(notification_id(), notification_close_observer.Wait());
 
   EXPECT_EQ(0u, GetMessageCenter()->GetVisibleNotifications().size());
 
   // Confirms that a download is still in progress.
@@ skipping 18 matching lines @@
   // Waits that new notification.
   download_notification_add_observer.Wait();
 
   // Confirms that there is only one notification.
   message_center::NotificationList::Notifications
       visible_notifications = GetMessageCenter()->GetVisibleNotifications();
   EXPECT_EQ(1u, visible_notifications.size());
   EXPECT_TRUE(IsInNotifications(visible_notifications, notification_id()));
 }
 
-IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, DownloadRemoved) {
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_DownloadRemoved DISABLED_DownloadRemoved
+#else
+#define MAYBE_DownloadRemoved DownloadRemoved
+#endif
+IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, MAYBE_DownloadRemoved) {
   CreateDownload();
 
   NotificationRemoveObserver notification_close_observer;
   download_item()->Remove();
   EXPECT_EQ(notification_id(), notification_close_observer.Wait());
 
   // Confirms that the notification is removed.
   EXPECT_EQ(0u, GetMessageCenter()->GetVisibleNotifications().size());
 
   // Confirms that the download item is removed.
   std::vector<content::DownloadItem*> downloads;
   GetDownloadManager(browser())->GetAllDownloads(&downloads);
   EXPECT_EQ(0u, downloads.size());
 }
 
-IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, DownloadMultipleFiles) {
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_DownloadMultipleFiles DISABLED_DownloadMultipleFiles
+#else
+#define MAYBE_DownloadMultipleFiles DownloadMultipleFiles
+#endif
+IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, MAYBE_DownloadMultipleFiles) {
   GURL url1(net::URLRequestSlowDownloadJob::kUnknownSizeUrl);
   GURL url2(net::URLRequestSlowDownloadJob::kKnownSizeUrl);
 
   // Starts the 1st download.
   NotificationAddObserver download_start_notification_observer1;
   ui_test_utils::NavigateToURL(browser(), url1);
   EXPECT_TRUE(download_start_notification_observer1.Wait());
   std::string notification_id1 =
       download_start_notification_observer1.notification_id();
   EXPECT_FALSE(notification_id1.empty());
@@ skipping 111 matching lines @@
   EXPECT_EQ(message_center::DEFAULT_PRIORITY,
             GetNotification(notification_id2)->priority());
 
   // Confirms the types of download notifications are correct.
   EXPECT_EQ(message_center::NOTIFICATION_TYPE_BASE_FORMAT,
             GetNotification(notification_id1)->type());
   EXPECT_EQ(message_center::NOTIFICATION_TYPE_BASE_FORMAT,
             GetNotification(notification_id2)->type());
 }
 
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_DownloadMultipleFilesOneByOne DISABLED_DownloadMultipleFilesOneByOne
+#else
+#define MAYBE_DownloadMultipleFilesOneByOne DownloadMultipleFilesOneByOne
+#endif
 IN_PROC_BROWSER_TEST_F(DownloadNotificationTest,
-                       DownloadMultipleFilesOneByOne) {
+                       MAYBE_DownloadMultipleFilesOneByOne) {
   CreateDownload();
   content::DownloadItem* first_download_item = download_item();
   content::DownloadItem* second_download_item = nullptr;
   std::string first_notification_id = notification_id();
   std::string second_notification_id;
 
   // Requests to complete the first download.
   ui_test_utils::NavigateToURL(
       browser(), GURL(net::URLRequestSlowDownloadJob::kFinishDownloadUrl));
 
@@ skipping 55 matching lines @@
     download_change_notification_observer2.Wait();
     download_change_notification_observer2.Reset();
   }
 
   // Opens the message center.
   GetMessageCenter()->SetVisibility(message_center::VISIBILITY_MESSAGE_CENTER);
   // Checks the message center.
   EXPECT_EQ(2u, GetMessageCenter()->GetVisibleNotifications().size());
 }
 
-IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, CancelDownload) {
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_CancelDownload DISABLED_CancelDownload
+#else
+#define MAYBE_CancelDownload CancelDownload
+#endif
+IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, MAYBE_CancelDownload) {
   CreateDownload();
 
   // Opens the message center.
   GetMessageCenter()->SetVisibility(message_center::VISIBILITY_MESSAGE_CENTER);
 
   // Cancels the notification by clicking the "cancel' button.
   NotificationRemoveObserver notification_close_observer;
   notification()->ButtonClick(1);
   EXPECT_EQ(notification_id(), notification_close_observer.Wait());
   EXPECT_EQ(0u, GetMessageCenter()->GetVisibleNotifications().size());
 
   // Confirms that a download is also cancelled.
   std::vector<content::DownloadItem*> downloads;
   GetDownloadManager(browser())->GetAllDownloads(&downloads);
   EXPECT_EQ(1u, downloads.size());
   EXPECT_EQ(content::DownloadItem::CANCELLED, downloads[0]->GetState());
 }
 
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_DownloadCancelledByUserExternally DISABLED_DownloadCancelledByUserExternally
+#else
+#define MAYBE_DownloadCancelledByUserExternally DownloadCancelledByUserExternally
+#endif
 IN_PROC_BROWSER_TEST_F(DownloadNotificationTest,
-                       DownloadCancelledByUserExternally) {
+                       MAYBE_DownloadCancelledByUserExternally) {
   CreateDownload();
 
   // Cancels the notification by clicking the "cancel' button.
   NotificationRemoveObserver notification_close_observer;
   download_item()->Cancel(true /* by_user */);
   EXPECT_EQ(notification_id(), notification_close_observer.Wait());
   EXPECT_EQ(0u, GetMessageCenter()->GetVisibleNotifications().size());
 
   // Confirms that a download is also cancelled.
   std::vector<content::DownloadItem*> downloads;
   GetDownloadManager(browser())->GetAllDownloads(&downloads);
   EXPECT_EQ(1u, downloads.size());
   EXPECT_EQ(content::DownloadItem::CANCELLED, downloads[0]->GetState());
 }
 
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_DownloadCancelledExternally DISABLED_DownloadCancelledExternally
+#else
+#define MAYBE_DownloadCancelledExternally DownloadCancelledExternally
+#endif
 IN_PROC_BROWSER_TEST_F(DownloadNotificationTest,
-                       DownloadCancelledExternally) {
+                       MAYBE_DownloadCancelledExternally) {
   CreateDownload();
 
   // Cancels the notification by clicking the "cancel' button.
   NotificationRemoveObserver notification_close_observer;
   download_item()->Cancel(false /* by_user */);
   EXPECT_EQ(notification_id(), notification_close_observer.Wait());
   EXPECT_EQ(0u, GetMessageCenter()->GetVisibleNotifications().size());
 
   // Confirms that a download is also cancelled.
   std::vector<content::DownloadItem*> downloads;
   GetDownloadManager(browser())->GetAllDownloads(&downloads);
   EXPECT_EQ(1u, downloads.size());
   EXPECT_EQ(content::DownloadItem::CANCELLED, downloads[0]->GetState());
 }
 
-IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, IncognitoDownloadFile) {
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_IncognitoDownloadFile DISABLED_IncognitoDownloadFile
+#else
+#define MAYBE_IncognitoDownloadFile IncognitoDownloadFile
+#endif
+IN_PROC_BROWSER_TEST_F(DownloadNotificationTest, MAYBE_IncognitoDownloadFile) {
   PrepareIncognitoBrowser();
 
   // Starts an incognito download.
   CreateDownloadForBrowserAndURL(
       incognito_browser(),
       GURL(net::URLRequestSlowDownloadJob::kKnownSizeUrl));
 
   EXPECT_EQ(l10n_util::GetStringFUTF16(
                 IDS_DOWNLOAD_STATUS_IN_PROGRESS_TITLE,
                 download_item()->GetFileNameToReportUser().LossyDisplayName()),
@@ skipping 28 matching lines @@
   // Try to open the downloaded item by clicking the notification.
   EXPECT_FALSE(GetIncognitoDownloadManagerDelegate()->opened());
   GetMessageCenter()->ClickOnNotification(notification_id());
   EXPECT_TRUE(GetIncognitoDownloadManagerDelegate()->opened());
   EXPECT_FALSE(GetDownloadManagerDelegate()->opened());
 
   EXPECT_FALSE(GetNotification(notification_id()));
   chrome::CloseWindow(incognito_browser());
 }
 
+// TODO(yoshiki): Disabled due to crbug.com/560329
+#if defined(OS_CHROMEOS)
+#define MAYBE_SimultaneousIncognitoAndNormalDownloads DISABLED_SimultaneousIncognitoAndNormalDownloads
+#else
+#define MAYBE_SimultaneousIncognitoAndNormalDownloads SimultaneousIncognitoAndNormalDownloads
+#endif
 IN_PROC_BROWSER_TEST_F(DownloadNotificationTest,
-                       SimultaneousIncognitoAndNormalDownloads) {
+                       MAYBE_SimultaneousIncognitoAndNormalDownloads) {
   PrepareIncognitoBrowser();
 
   GURL url_incognito(net::URLRequestSlowDownloadJob::kUnknownSizeUrl);
   GURL url_normal(net::URLRequestSlowDownloadJob::kKnownSizeUrl);
 
   // Starts the incognito download.
   NotificationAddObserver download_start_notification_observer1;
   ui_test_utils::NavigateToURL(incognito_browser(), url_incognito);
   EXPECT_TRUE(download_start_notification_observer1.Wait());
   std::string notification_id1 =
@@ skipping 212 matching lines @@
   EXPECT_EQ(message_center::NOTIFICATION_TYPE_BASE_FORMAT,
             GetNotification(notification_id_user1)->type());
   EXPECT_EQ(message_center::NOTIFICATION_TYPE_BASE_FORMAT,
             GetNotification(notification_id_user2_1)->type());
   // Normal notifications for user2.
   EXPECT_EQ(message_center::NOTIFICATION_TYPE_BASE_FORMAT,
             GetNotification(notification_id_user2_1)->type());
   EXPECT_EQ(message_center::NOTIFICATION_TYPE_BASE_FORMAT,
             GetNotification(notification_id_user2_2)->type());
 }