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

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

Issue 9576001: Add getFontList() to Font Settings Extension API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 10 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 d364473ad80994578ec242195926950678449105..bef1e3798213cf0a0ccf1b53a56282f9308b3365 100644
--- a/chrome/test/data/extensions/api_test/font_settings/test.js
+++ b/chrome/test/data/extensions/api_test/font_settings/test.js
@@ -45,5 +45,16 @@ chrome.test.runTests([
fs.getFontName({
genericFamily: 'sansserif'
}, expect({fontName: expected}, message));
+ },
+
+ function getFontList() {
+ var message = 'getFontList should return an array of objects with ' +
+ 'fontName and localizedName properties.';
+ fs.getFontList(chrome.test.callbackPass(function(value) {
+ chrome.test.assertTrue(value.length > 0,
+ "Font list is not expected to be empty.");
+ chrome.test.assertEq('string', typeof(value[0].fontName), message);
+ chrome.test.assertEq('string', typeof(value[0].localizedName), message);
+ }));
}
]);

Powered by Google App Engine
This is Rietveld 408576698