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

Unified Diff: chrome/browser/resources/options/font_settings.js

Issue 543493002: Compile chrome://settings, part 2: reduce from 950 to 400 errors (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@F_settings
Patch Set: rebase? rebase! Created 6 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/browser/resources/options/font_settings.js
diff --git a/chrome/browser/resources/options/font_settings.js b/chrome/browser/resources/options/font_settings.js
index 84982d31eb45402bdeb3567501ceb62ac38bf14b..354a14b5a67f12aefb939de31c01024484194d87 100644
--- a/chrome/browser/resources/options/font_settings.js
+++ b/chrome/browser/resources/options/font_settings.js
@@ -155,7 +155,8 @@ cr.define('options', function() {
* @param {Element} el The div containing the sample text.
* @param {number} size The font size of the sample text.
* @param {string} font The font family of the sample text.
- * @param {bool} showSize True if the font size should appear in the sample.
+ * @param {boolean} showSize True if the font size should appear in the
+ * sample.
* @private
*/
setUpFontSample_: function(el, size, font, showSize) {
@@ -179,13 +180,12 @@ cr.define('options', function() {
element.textContent = '';
// Insert new child nodes into select element.
- var value, text, selected, option;
for (var i = 0; i < items.length; i++) {
- value = items[i][0];
- text = items[i][1];
- dir = items[i][2];
+ var value = items[i][0];
+ var text = items[i][1];
+ var dir = items[i][2];
if (text) {
- selected = value == selectedValue;
+ var selected = value == selectedValue;
var option = new Option(text, value, false, selected);
option.dir = dir;
element.appendChild(option);
« no previous file with comments | « chrome/browser/resources/options/editable_text_field.js ('k') | chrome/browser/resources/options/geolocation_options.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698