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

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

Issue 10107014: Migrate WebKit "global script" font prefs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review comments Created 8 years, 7 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 adaa5d7d756c705ff76e46682a714a6b7e051b00..4731a76c45f907167fc52f1a062dd6e865d97d33 100644
--- a/chrome/test/data/extensions/api_test/font_settings/test.js
+++ b/chrome/test/data/extensions/api_test/font_settings/test.js
@@ -58,6 +58,7 @@ chrome.test.runTests([
chrome.test.assertEq(details, {
genericFamily: genericFamily,
fontName: fontName,
+ script: 'Zyyy',
levelOfControl: 'controlled_by_this_extension'
});
});
@@ -83,7 +84,7 @@ chrome.test.runTests([
'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.");
+ '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);
}));
@@ -200,11 +201,13 @@ chrome.test.runTests([
// This test may fail on Windows if the font is not installed on the
// system. See crbug.com/122303
function clearGlobalFont() {
+ var script = 'Zyyy';
var genericFamily = 'sansserif';
var fontName = 'Arial';
chrome.test.listenOnce(fs.onFontChanged, function(details) {
chrome.test.assertEq(details, {
+ script: script,
genericFamily: genericFamily,
fontName: fontName,
levelOfControl: CONTROLLABLE_BY_THIS_EXTENSION

Powered by Google App Engine
This is Rietveld 408576698