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

Unified Diff: chrome/browser/ui/webui/options/certificate_manager_browsertest.js

Issue 2434473004: Disable many tests with flaky accessibility audits. (Closed)
Patch Set: Disable a bunch of flaky tests Created 4 years, 2 months 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: chrome/browser/ui/webui/options/certificate_manager_browsertest.js
diff --git a/chrome/browser/ui/webui/options/certificate_manager_browsertest.js b/chrome/browser/ui/webui/options/certificate_manager_browsertest.js
index cb9e3d8184046fe29695e40f755d69b4dc7c3639..830de666b2ebcd96dcb075ee333d88ede77d012c 100644
--- a/chrome/browser/ui/webui/options/certificate_manager_browsertest.js
+++ b/chrome/browser/ui/webui/options/certificate_manager_browsertest.js
@@ -268,8 +268,16 @@ CertificateManagerSettingsWebUITest.prototype = {
browsePreload: CERTIFICATE_MANAGER_SETTINGS_PAGE_URL,
};
+
+// TODO(crbug.com/657514) Flakes on linux and chromeos.
+GEN('#if defined(OS_LINUX) || defined(OS_CHROMEOS)');
+GEN('#define MAYBE_testViewAndDeleteCert ' +
+ 'DISABLED_testViewAndDeleteCert');
+GEN('#else');
+GEN('#define MAYBE_testViewAndDeleteCert testViewAndDeleteCert');
+GEN('#endif');
TEST_F('CertificateManagerSettingsWebUITest',
- 'testViewAndDeleteCert', function() {
+ 'MAYBE_testViewAndDeleteCert', function() {
assertEquals(this.browsePreload, document.location.href);
this.mockHandler.expects(once()).viewCertificate(['c1']);

Powered by Google App Engine
This is Rietveld 408576698