Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(944)

Unified Diff: testing/buildbot/filters/browser-side-navigation.linux.browser_tests.filter

Issue 2538483002: Add management related code to SafeBrowsingNavigationObserverManager (Closed)
Patch Set: address comments from creis@ and holte@ + rebase Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: testing/buildbot/filters/browser-side-navigation.linux.browser_tests.filter
diff --git a/testing/buildbot/filters/browser-side-navigation.linux.browser_tests.filter b/testing/buildbot/filters/browser-side-navigation.linux.browser_tests.filter
index 400f17bf99c4a756ebf1f1a0fe7bb808f3cafbc4..9dea373d68b88c65cb1db51ac087626d9bdaff89 100644
--- a/testing/buildbot/filters/browser-side-navigation.linux.browser_tests.filter
+++ b/testing/buildbot/filters/browser-side-navigation.linux.browser_tests.filter
@@ -1,6 +1,12 @@
-ChromeServiceWorkerTest.FallbackMainResourceRequestWhenJSDisabled
-PDFExtensionTest.PdfAccessibilityInOOPIF
+# NavigationHandle::IsRendererInitiated some times return differently in
+# browser side navigation tests.
+-SBNavigationObserverBrowserTest.NewTabDownload
+-SBNavigationObserverBrowserTest.NewTabDownloadWithDataURL
+-SBNavigationObserverBrowserTest.SubFrameNewTabDownload
+
# https://crbug.com/652767: NavigationHandle::GetResponseHeaders sometimes
# returns null for browser-side navigations
-PageLoadMetricsBrowserTest.Ignore204Pages

Powered by Google App Engine
This is Rietveld 408576698