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

Unified Diff: chrome/common/extensions/docs/examples/api/fontSettings/pending_changes.js

Issue 23434003: UI refresh of Advanced Font Settings Extension (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: nits & minus image assets Created 7 years, 3 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/common/extensions/docs/examples/api/fontSettings/pending_changes.js
diff --git a/chrome/common/extensions/docs/examples/api/fontSettings/pending_changes.js b/chrome/common/extensions/docs/examples/api/fontSettings/pending_changes.js
new file mode 100644
index 0000000000000000000000000000000000000000..629a989ce89ff2c43c6f482c088af7f9c4b2e91b
--- /dev/null
+++ b/chrome/common/extensions/docs/examples/api/fontSettings/pending_changes.js
@@ -0,0 +1,145 @@
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+'use strict';
+
+/**
+ * @fileoverview PendingChanges class tracks changes to be applied when an
+ * "Apply Changes" button is clicked.
+ */
+
+/**
+ * Creates a PendingChanges object with no pending changes.
+ *
+ * @constructor
+ */
+var PendingChanges = function() {
+ // Format: pendingFontChanges_.Cyrl.sansserif = "My SansSerif Cyrillic Font"
+ this.pendingFontChanges_ = {};
+
+ // Format: pendingFontSizeChanges_.defaultFontSize = 12
+ this.pendingFontSizeChanges_ = {};
+};
+
+/**
+ * Returns the pending font setting change for the specified script and family,
+ * or null if it doesn't exist.
+ *
+ * @param {string} script The script code, like "Cyrl".
+ * @param {string} genericFamily The generic family, like "sansserif".
+ * @return {?string} The pending font setting, like "My Cyrillic SansSerif Font"
+ * or null if it doesn't exist.
+ */
+PendingChanges.prototype.getFont = function(script, genericFamily) {
+ if (this.pendingFontChanges_[script])
+ return this.pendingFontChanges_[script][genericFamily];
+ return null;
+};
+
+/**
+ * Returns the pending font size setting change, or null if it doesn't exist.
+ *
+ * @param {string} fontSizeKey The font size setting key. One of
+ * 'defaultFontSize', 'defaultFixedFontSize', or 'minFontSize'.
+ * @return {?number} The pending font size setting in pixels, or null if it
+ * doesn't exist.
+ */
+PendingChanges.prototype.getFontSize = function(fontSizeKey) {
+ return this.pendingFontSizeChanges_[fontSizeKey];
+};
+
+/**
+ * Sets the pending font change for the specified script and family.
+ *
+ * @param {string} script The script code, like "Cyrl".
+ * @param {string} genericFamily The generic family, like "sansserif".
+ * @param {?string} font The font to set the setting to, or null to clear it.
+ */
+PendingChanges.prototype.setFont = function(script, genericFamily, font) {
+ if (!this.pendingFontChanges_[script])
+ this.pendingFontChanges_[script] = {};
+ if (this.pendingFontChanges_[script][genericFamily] == font)
+ return;
+ this.pendingFontChanges_[script][genericFamily] = font;
+};
+
+/**
+ * Sets the pending font size change.
+ *
+ * @param {string} fontSizeKey The font size setting key. See
+ * getFontSize().
+ * @param {number} size The font size to set the setting to.
+ */
+PendingChanges.prototype.setFontSize = function(fontSizeKey, size) {
+ if (this.pendingFontSizeChanges_[fontSizeKey] == size)
+ return;
+ this.pendingFontSizeChanges_[fontSizeKey] = size;
+};
+
+/**
+ * Commits the pending changes to Chrome. After this function is called, there
+ * are no pending changes.
+ */
+PendingChanges.prototype.apply = function() {
+ for (var script in this.pendingFontChanges_) {
+ for (var genericFamily in this.pendingFontChanges_[script]) {
+ var fontId = this.pendingFontChanges_[script][genericFamily];
+ if (fontId == null)
+ continue;
+ var details = {};
+ details.script = script;
+ details.genericFamily = genericFamily;
+ details.fontId = fontId;
+ chrome.fontSettings.setFont(details);
+ }
+ }
+
+ var size = this.pendingFontSizeChanges_['defaultFontSize'];
+ if (size != null)
+ chrome.fontSettings.setDefaultFontSize({pixelSize: size});
+
+ size = this.pendingFontSizeChanges_['defaultFixedFontSize'];
+ if (size != null)
+ chrome.fontSettings.setDefaultFixedFontSize({pixelSize: size});
+
+ size = this.pendingFontSizeChanges_['minFontSize'];
+ if (size != null)
+ chrome.fontSettings.setMinimumFontSize({pixelSize: size});
+
+ this.clear();
+};
+
+/**
+ * Clears the pending font changes for a single script.
+ *
+ * @param {string} script The script code, like "Cyrl".
+ */
+PendingChanges.prototype.clearOneScript = function(script) {
+ this.pendingFontChanges_[script] = {};
+};
+
+/**
+ * Clears all pending font changes.
+ */
+PendingChanges.prototype.clear = function() {
+ this.pendingFontChanges_ = {};
+ this.pendingFontSizeChanges_ = {};
+};
+
+/**
+ * @return {boolean} True if there are no pending changes, otherwise false.
+ */
+PendingChanges.prototype.isEmpty = function() {
+ for (var script in this.pendingFontChanges_) {
+ for (var genericFamily in this.pendingFontChanges_[script]) {
+ if (this.pendingFontChanges_[script][genericFamily] != null)
+ return false;
+ }
+ }
+ for (var name in this.pendingFontSizeChanges_) {
+ if (this.pendingFontSizeChanges_[name] != null)
+ return false;
+ }
+ return true;
+};

Powered by Google App Engine
This is Rietveld 408576698