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

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

Issue 10716003: [Font Settings API]: Rename fontName and localizedName to fontId and displayName (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: patch for landing Created 8 years, 5 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/incognito/launch.js
diff --git a/chrome/test/data/extensions/api_test/font_settings/incognito/launch.js b/chrome/test/data/extensions/api_test/font_settings/incognito/launch.js
index 4c3a04e6d9eb950c312fa4e0183cccc30871a8d7..96bc8b708e00afec86967a547c6fb87c857a2842 100644
--- a/chrome/test/data/extensions/api_test/font_settings/incognito/launch.js
+++ b/chrome/test/data/extensions/api_test/font_settings/incognito/launch.js
@@ -21,22 +21,22 @@ chrome.test.runTests([
function setPerScriptFont() {
var script = 'Hang';
var genericFamily = 'standard';
- var fontName = 'Verdana';
+ var fontId = 'Verdana';
fs.setFont({
script: script,
genericFamily: genericFamily,
- fontName: fontName
+ fontId: fontId
}, chrome.test.callbackFail(SET_FROM_INCOGNITO_ERROR));
},
function setGlobalFontName() {
var genericFamily = 'sansserif';
- var fontName = 'Tahoma';
+ var fontId = 'Tahoma';
fs.setFont({
genericFamily: genericFamily,
- fontName: fontName
+ fontId: fontId
}, chrome.test.callbackFail(SET_FROM_INCOGNITO_ERROR));
},
@@ -50,12 +50,12 @@ chrome.test.runTests([
function getFontList() {
var message = 'getFontList should return an array of objects with ' +
- 'fontName and localizedName properties.';
+ 'fontId and displayName 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);
+ chrome.test.assertEq('string', typeof(value[0].fontId), message);
+ chrome.test.assertEq('string', typeof(value[0].displayName), message);
}));
},
@@ -64,7 +64,7 @@ chrome.test.runTests([
script: 'Hang',
genericFamily: 'standard'
}, expect({
- fontName: 'Tahoma',
+ fontId: 'Tahoma',
levelOfControl: CONTROLLABLE_BY_THIS_EXTENSION
}));
},
@@ -73,7 +73,7 @@ chrome.test.runTests([
fs.getFont({
genericFamily: 'sansserif'
}, expect({
- fontName: 'Arial',
+ fontId: 'Arial',
levelOfControl: CONTROLLABLE_BY_THIS_EXTENSION
}));
},

Powered by Google App Engine
This is Rietveld 408576698