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

Unified Diff: chrome/browser/ui/webui/options/font_settings_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/font_settings_browsertest.js
diff --git a/chrome/browser/ui/webui/options/font_settings_browsertest.js b/chrome/browser/ui/webui/options/font_settings_browsertest.js
deleted file mode 100644
index 6e419a207cfe0dfd5cfe4448369c7b120d92c14c..0000000000000000000000000000000000000000
--- a/chrome/browser/ui/webui/options/font_settings_browsertest.js
+++ /dev/null
@@ -1,66 +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 font settings WebUI testing.
- * @extends {testing.Test}
- * @constructor
- */
-function FontSettingsWebUITest() {}
-
-FontSettingsWebUITest.prototype = {
- __proto__: OptionsBrowsertestBase.prototype,
-
- /**
- * Browse to the font settings page.
- */
- browsePreload: 'chrome://settings-frame/fonts',
-
- /** @override */
- preLoad: function() {
- this.makeAndRegisterMockHandler(['openAdvancedFontSettingsOptions']);
- },
-
- /** @override */
- setUp: function() {
- OptionsBrowsertestBase.prototype.setUp.call(this);
-
- var controlsWithoutLabelSelectors = [
- '#standard-font-size',
- '#minimum-font-size',
- ];
-
- // Enable when failure is resolved.
- // AX_TEXT_01: http://crbug.com/570555
- this.accessibilityAuditConfig.ignoreSelectors(
- 'controlsWithoutLabel',
- controlsWithoutLabelSelectors);
- },
-};
-
-// Test opening font settings has correct location.
-TEST_F('FontSettingsWebUITest', 'testOpenFontSettings', function() {
- assertEquals(this.browsePreload, document.location.href);
-});
-
-// Test setup of the Advanced Font Settings links.
-TEST_F('FontSettingsWebUITest', 'testAdvancedFontSettingsLink', function() {
- var installElement = $('advanced-font-settings-install');
- var optionsElement = $('advanced-font-settings-options');
- var expectedUrl = 'https://chrome.google.com/webstore/detail/' +
- 'caclkomlalccbpcdllchkeecicepbmbm';
-
- FontSettings.notifyAdvancedFontSettingsAvailability(false);
- assertFalse(installElement.hidden);
- assertEquals(expectedUrl, installElement.querySelector('a').href);
- assertTrue(optionsElement.hidden);
-
- FontSettings.notifyAdvancedFontSettingsAvailability(true);
- assertTrue(installElement.hidden);
- assertFalse(optionsElement.hidden);
- this.mockHandler.expects(once()).openAdvancedFontSettingsOptions();
- optionsElement.click();
-});

Powered by Google App Engine
This is Rietveld 408576698