Disable SaveHTMLOnlyTabDestroy in debug builds too.

Fails here: https://ci.chromium.org/p/chromium/builders/ci/Win7%20Tests%20%28dbg%29%281%29/84937 TBR=qinmin@chromium.org Bug: 580766 Change-Id: I762cd32aac3bd438f33877d2ae51ed3ada81d056 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2403244Reviewed-by: Morten Stenshorne <mstensho@chromium.org> Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#805713}

Disable SaveHTMLOnlyTabDestroy in debug builds too.
Fails here: https://ci.chromium.org/p/chromium/builders/ci/Win7%20Tests%20%28dbg%29%281%29/84937 TBR=qinmin@chromium.org Bug: 580766 Change-Id: I762cd32aac3bd438f33877d2ae51ed3ada81d056 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2403244Reviewed-by: Morten Stenshorne <mstensho@chromium.org> Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#805713}
6f032fda · Morten Stenshorne · Commit Bot · ebe94954 · 6f032fda
Commit 6f032fda authored Sep 10, 2020 by Morten Stenshorne Committed by Commit Bot Sep 10, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 2 deletions

chrome/browser/download/save_page_browsertest.cc chrome/browser/download/save_page_browsertest.cc +1 -2

No files found.
--- a/chrome/browser/download/save_page_browsertest.cc
+++ b/chrome/browser/download/save_page_browsertest.cc
@@ -535,8 +535,7 @@ class DelayingDownloadManagerDelegate : public ChromeDownloadManagerDelegate {
 };
 // Disabled on multiple platforms due to flakiness. crbug.com/580766
-#if defined(OS_CHROMEOS) || \
+#if defined(OS_CHROMEOS) || defined(OS_WIN) || defined(OS_LINUX)
-    (defined(OS_WIN) || defined(OS_LINUX)) && defined(NDEBUG)
 #define MAYBE_SaveHTMLOnlyTabDestroy DISABLED_SaveHTMLOnlyTabDestroy
 #else
 #define MAYBE_SaveHTMLOnlyTabDestroy SaveHTMLOnlyTabDestroy