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

Side by Side Diff: chrome/browser/ui/webui/options/font_settings_browsertest.js

Issue 33753004: Add link to the Advanced Font Settings extension in Font Settings (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review comments Created 7 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * TestFixture for font settings WebUI testing. 6 * TestFixture for font settings WebUI testing.
7 * @extends {testing.Test} 7 * @extends {testing.Test}
8 * @constructor 8 * @constructor
9 **/ 9 **/
10 function FontSettingsWebUITest() {} 10 function FontSettingsWebUITest() {}
11 11
12 FontSettingsWebUITest.prototype = { 12 FontSettingsWebUITest.prototype = {
13 __proto__: testing.Test.prototype, 13 __proto__: testing.Test.prototype,
14 14
15 /** 15 /**
16 * Browse to the font settings page. 16 * Browse to the font settings page.
17 **/ 17 **/
18 browsePreload: 'chrome://settings-frame/fonts', 18 browsePreload: 'chrome://settings-frame/fonts',
19
20 /** @inheritDoc */
21 preLoad: function() {
22 this.makeAndRegisterMockHandler(['openAdvancedFontSettingsOptions']);
23 }
19 }; 24 };
20 25
21 // Test opening font settings has correct location. 26 // Test opening font settings has correct location.
22 TEST_F('FontSettingsWebUITest', 'testOpenFontSettings', function() { 27 TEST_F('FontSettingsWebUITest', 'testOpenFontSettings', function() {
23 assertEquals(this.browsePreload, document.location.href); 28 assertEquals(this.browsePreload, document.location.href);
24 }); 29 });
30
31 // Test setup of the Advanced Font Settings links.
32 TEST_F('FontSettingsWebUITest', 'testAdvancedFontSettingsLink', function() {
33 var installElement = $('advanced-font-settings-install');
34 var optionsElement = $('advanced-font-settings-options');
35 var expectedUrl = 'https://chrome.google.com/webstore/detail/' +
36 'caclkomlalccbpcdllchkeecicepbmbm';
37
38 FontSettings.notifyAdvancedFontSettingsAvailability(false);
39 assertFalse(installElement.hidden);
40 assertEquals(expectedUrl, installElement.querySelector('a').href);
41 assertTrue(optionsElement.hidden);
42
43 FontSettings.notifyAdvancedFontSettingsAvailability(true);
44 assertTrue(installElement.hidden);
45 assertFalse(optionsElement.hidden);
46 this.mockHandler.expects(once()).openAdvancedFontSettingsOptions();
47 optionsElement.click();
48 });
OLDNEW
« no previous file with comments | « chrome/browser/resources/options/font_settings.js ('k') | chrome/browser/ui/webui/options/font_settings_handler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698