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

Unified Diff: chrome/browser/resources/settings/appearance_page/appearance_fonts_page.js

Issue 2617663002: WIP: run clang-format-js on lots of things (Closed)
Patch Set: merge Created 3 years, 11 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/browser/resources/settings/appearance_page/appearance_fonts_page.js
diff --git a/chrome/browser/resources/settings/appearance_page/appearance_fonts_page.js b/chrome/browser/resources/settings/appearance_page/appearance_fonts_page.js
index 2e26c5905a3b0531ec03d78bea6fc9a9e68f2bba..19f9b769f2303856c354b3e40b9c0b6d0faeb585 100644
--- a/chrome/browser/resources/settings/appearance_page/appearance_fonts_page.js
+++ b/chrome/browser/resources/settings/appearance_page/appearance_fonts_page.js
@@ -14,14 +14,13 @@
/** @const @private {!Array<number>} */
var FONT_SIZE_RANGE_ = [
- 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 20, 22, 24, 26, 28, 30, 32, 34, 36,
- 40, 44, 48, 56, 64, 72,
+ 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 20, 22, 24,
+ 26, 28, 30, 32, 34, 36, 40, 44, 48, 56, 64, 72,
];
/** @const @private {!Array<number>} */
- var MINIMUM_FONT_SIZE_RANGE_ = [
- 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 20, 22, 24
- ];
+ var MINIMUM_FONT_SIZE_RANGE_ =
+ [6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 20, 22, 24];
/**
* 'settings-appearance-fonts-page' is the settings page containing appearance
@@ -93,12 +92,12 @@
/** @override */
ready: function() {
- this.addWebUIListener('advanced-font-settings-installed',
+ this.addWebUIListener(
+ 'advanced-font-settings-installed',
this.setAdvancedExtensionInstalled_.bind(this));
this.browserProxy_.observeAdvancedFontExtensionAvailable();
- this.browserProxy_.fetchFontsData().then(
- this.setFontsData_.bind(this));
+ this.browserProxy_.fetchFontsData().then(this.setFontsData_.bind(this));
},
/** @private */
@@ -116,8 +115,9 @@
*/
setAdvancedExtensionInstalled_: function(isInstalled) {
this.advancedExtensionInstalled_ = isInstalled;
- this.advancedExtensionSublabel_ = this.i18n(isInstalled ?
- 'openAdvancedFontSettings' : 'requiresWebStoreExtension');
+ this.advancedExtensionSublabel_ = this.i18n(
+ isInstalled ? 'openAdvancedFontSettings' :
+ 'requiresWebStoreExtension');
},
/**
@@ -128,10 +128,8 @@
setFontsData_: function(response) {
var fontMenuOptions = [];
for (var i = 0; i < response.fontList.length; ++i) {
- fontMenuOptions.push({
- value: response.fontList[i][0],
- name: response.fontList[i][1]
- });
+ fontMenuOptions.push(
+ {value: response.fontList[i][0], name: response.fontList[i][1]});
}
this.fontOptions_ = fontMenuOptions;
this.advancedExtensionUrl_ = response.extensionUrl;
@@ -145,7 +143,8 @@
// TODO(michaelpg): Whitelist this pref in prefs_utils.cc so it is
// included in the <settings-prefs> getAllPrefs call, otherwise this path
// is invalid and nothing happens. See crbug.com/612535.
- this.set('prefs.webkit.webprefs.default_fixed_font_size.value',
+ this.set(
+ 'prefs.webkit.webprefs.default_fixed_font_size.value',
value - SIZE_DIFFERENCE_FIXED_STANDARD_);
},

Powered by Google App Engine
This is Rietveld 408576698