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

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

Issue 2440973003: Revert of Disable many tests with flaky accessibility audits. (Closed)
Patch Set: 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 830de666b2ebcd96dcb075ee333d88ede77d012c..cb9e3d8184046fe29695e40f755d69b4dc7c3639 100644
--- a/chrome/browser/ui/webui/options/certificate_manager_browsertest.js
+++ b/chrome/browser/ui/webui/options/certificate_manager_browsertest.js
@@ -268,16 +268,8 @@
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',
- 'MAYBE_testViewAndDeleteCert', function() {
+ 'testViewAndDeleteCert', function() {
assertEquals(this.browsePreload, document.location.href);
this.mockHandler.expects(once()).viewCertificate(['c1']);

Powered by Google App Engine
This is Rietveld 408576698