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

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

Issue 2919343005: Remove tests for deprecated Options UI (Closed)
Patch Set: and more Created 3 years, 6 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/browser_options_browsertest.js
diff --git a/chrome/browser/ui/webui/options/browser_options_browsertest.js b/chrome/browser/ui/webui/options/browser_options_browsertest.js
deleted file mode 100644
index b5713f60ca6b113fffd262491b6b8c80d3a7e24b..0000000000000000000000000000000000000000
--- a/chrome/browser/ui/webui/options/browser_options_browsertest.js
+++ /dev/null
@@ -1,131 +0,0 @@
-// Copyright (c) 2012 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.
-
-GEN_INCLUDE(['options_browsertest_base.js']);
-
-/**
- * TestFixture for browser options WebUI testing.
- * @extends {testing.Test}
- * @constructor
- */
-function BrowserOptionsWebUITest() {}
-
-BrowserOptionsWebUITest.prototype = {
- __proto__: testing.Test.prototype,
-
- /** @override */
- browsePreload: 'chrome://chrome/settings/',
-};
-
-// Test opening the browser options has correct location.
-// Times out on Mac debug only. See http://crbug.com/121030
-GEN('#if defined(OS_MACOSX) && !defined(NDEBUG)');
-GEN('#define MAYBE_testOpenBrowserOptions ' +
- 'DISABLED_testOpenBrowserOptions');
-GEN('#else');
-GEN('#define MAYBE_testOpenBrowserOptions testOpenBrowserOptions');
-GEN('#endif // defined(OS_MACOSX)');
-TEST_F('BrowserOptionsWebUITest', 'MAYBE_testOpenBrowserOptions', function() {
- assertEquals(this.browsePreload, document.location.href);
- expectFalse($('navigation').classList.contains('background'));
-});
-
-/**
- * TestFixture for the uber page when the browser options page has an overlay.
- * @extends {testing.Test}
- * @constructor
- */
-function BrowserOptionsOverlayWebUITest() {}
-
-BrowserOptionsOverlayWebUITest.prototype = {
- __proto__: testing.Test.prototype,
-
- /** @override */
- browsePreload: 'chrome://chrome/settings/autofill',
-
- /** @override */
- isAsync: true,
-};
-
-TEST_F('BrowserOptionsOverlayWebUITest', 'testNavigationInBackground',
- function() {
- assertEquals(this.browsePreload, document.location.href);
-
- if ($('navigation').classList.contains('background')) {
- testDone();
- return;
- }
-
- // Wait for the message to be posted to the Uber page.
- window.addEventListener('message', function(e) {
- if (e.data.method == 'beginInterceptingEvents') {
- window.setTimeout(function() {
- assertTrue($('navigation').classList.contains('background'));
- testDone();
- });
- }
- });
-});
-
-/**
- * @extends {testing.Test}
- * @constructor
- */
-function BrowserOptionsFrameWebUITest() {}
-
-BrowserOptionsFrameWebUITest.prototype = {
- __proto__: OptionsBrowsertestBase.prototype,
-
- /** @override */
- browsePreload: 'chrome://settings-frame/',
-
- /** @override */
- setUp: function() {
- OptionsBrowsertestBase.prototype.setUp.call(this);
-
- // Enable when failure is resolved.
- // AX_TEXT_04: http://crbug.com/570721
- this.accessibilityAuditConfig.ignoreSelectors(
- 'linkWithUnclearPurpose',
- '#sync-overview > A');
-
- // Enable when failure is resolved.
- // AX_ARIA_10: http://crbug.com/570723
- this.accessibilityAuditConfig.ignoreSelectors(
- 'unsupportedAriaAttribute',
- '#profiles-list');
- },
-};
-
-TEST_F('BrowserOptionsFrameWebUITest', 'testAdvancedSettingsHiddenByDefault',
- function() {
- assertEquals(this.browsePreload, document.location.href);
- expectTrue($('advanced-settings').hidden);
-});
-
-/**
- * @extends {testing.Test}
- * @constructor
- */
-function AdvancedSettingsWebUITest() {}
-
-AdvancedSettingsWebUITest.prototype = {
- __proto__: OptionsBrowsertestBase.prototype,
-
- /** @override */
- browsePreload: 'chrome://settings-frame/autofill',
-};
-
-// TODO(crbug.com/617066) Flakes on Win.
-GEN('#if defined(OS_WIN)');
-GEN('#define MAYBE_testAdvancedSettingsShown ' +
- 'DISABLED_testAdvancedSettingsShown');
-GEN('#else');
-GEN('#define MAYBE_testAdvancedSettingsShown testAdvancedSettingsShown');
-GEN('#endif');
-TEST_F('AdvancedSettingsWebUITest', 'MAYBE_testAdvancedSettingsShown',
- function() {
- assertEquals(this.browsePreload, document.location.href);
- expectFalse($('advanced-settings').hidden);
-});

Powered by Google App Engine
This is Rietveld 408576698