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

Unified Diff: chrome/test/data/extensions/api_test/font_settings/test.js

Issue 10008076: Add onFontNameChanged event to Font Settings API (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 8 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/test/data/extensions/api_test/font_settings/test.js
diff --git a/chrome/test/data/extensions/api_test/font_settings/test.js b/chrome/test/data/extensions/api_test/font_settings/test.js
index 4a9094d8cd15229f000c49aff3ad8154f925e455..2b7fe3e78fa6337281d088b3099ad3ae959eec03 100644
--- a/chrome/test/data/extensions/api_test/font_settings/test.js
+++ b/chrome/test/data/extensions/api_test/font_settings/test.js
@@ -14,11 +14,26 @@ function expect(expected, message) {
}
chrome.test.runTests([
+ // This test may fail on Windows if the font is not installed on the
+ // system. See crbug.com/122303
function setPerScriptFontName() {
+ var script = 'Hang';
+ var genericFamily = 'standard';
+ var fontName = 'Verdana';
+
+ chrome.test.listenOnce(fs.onFontNameChanged, function(details) {
+ chrome.test.assertEq(details, {
+ script: script,
+ genericFamily: genericFamily,
+ fontName: fontName,
+ levelOfControl: 'controlled_by_this_extension'
+ });
+ });
+
fs.setFontName({
- script: 'Hang',
- genericFamily: 'standard',
- fontName: 'Verdana'
+ script: script,
+ genericFamily: genericFamily,
+ fontName: fontName
}, chrome.test.callbackPass());
},
@@ -26,15 +41,26 @@ chrome.test.runTests([
var expected = 'Verdana';
var message = 'Setting for Hangul standard font should be ' + expected;
- // This test may fail on Windows if the font is not installed on the
- // system. See crbug.com/122303
fs.getFontName({
script: 'Hang',
genericFamily: 'standard'
}, expect({fontName: expected}, message));
},
+ // This test may fail on Windows if the font is not installed on
+ // the system. See crbug.com/122303
function setGlobalFontName() {
+ var fontName = 'Tahoma';
+ var genericFamily = 'sansserif';
+
+ chrome.test.listenOnce(fs.onFontNameChanged, function(details) {
+ chrome.test.assertEq(details, {
+ genericFamily: genericFamily,
+ fontName: fontName,
+ levelOfControl: 'controlled_by_this_extension'
+ });
+ });
+
fs.setFontName({
genericFamily: 'sansserif',
fontName: 'Tahoma'
@@ -46,8 +72,6 @@ chrome.test.runTests([
var message =
'Setting for global sansserif font should be ' + expected;
- // As above, this test may fail on Windows if the font is not installed on
- // the system. See crbug.com/122303
fs.getFontName({
genericFamily: 'sansserif'
}, expect({fontName: expected}, message));

Powered by Google App Engine
This is Rietveld 408576698