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

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

Issue 9853013: Expose more font size prefs to the Font Settings Extension API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: retry Created 8 years, 9 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 2c8ec157743d45d37cd8dd2f716036bfa2163a7f..8fa3f07a0bb7695cc67407a4ec218eddff9ac979 100644
--- a/chrome/test/data/extensions/api_test/font_settings/test.js
+++ b/chrome/test/data/extensions/api_test/font_settings/test.js
@@ -68,5 +68,29 @@ chrome.test.runTests([
var expected = 22;
var message = 'Setting for default font size should be ' + expected;
fs.getDefaultFontSize({}, expect({pixelSize: expected}, message));
+ },
+
+ function setDefaultFixedFontSize() {
+ fs.setDefaultFixedFontSize({
+ pixelSize: 42
+ }, chrome.test.callbackPass());
+ },
+
+ function getDefaultFixedFontSize() {
+ var expected = 42;
+ var message = 'Setting for default fixed font size should be ' + expected;
+ fs.getDefaultFixedFontSize({}, expect({pixelSize: expected}, message));
+ },
+
+ function setMinimumFontSize() {
+ fs.setMinimumFontSize({
+ pixelSize: 7
+ }, chrome.test.callbackPass());
+ },
+
+ function getMinimumFontSize() {
+ var expected = 7;
+ var message = 'Setting for minimum font size should be ' + expected;
+ fs.getMinimumFontSize({}, expect({pixelSize: expected}, message));
Matt Perry 2012/03/26 18:53:05 how about putting all the setter tests before the
falken 2012/03/27 02:26:01 Done.
}
]);

Powered by Google App Engine
This is Rietveld 408576698