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

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

Issue 2617823002: GRIT: put <if> and <include> behind comments in .js files to make syntactically valid JS (Closed)
Patch Set: add dep 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_page.js
diff --git a/chrome/browser/resources/settings/appearance_page/appearance_page.js b/chrome/browser/resources/settings/appearance_page/appearance_page.js
index 67546e1059569fbaf365135a48d8a021f918bf47..ba9b37d2955649613e24e213b4d103301deba656 100644
--- a/chrome/browser/resources/settings/appearance_page/appearance_page.js
+++ b/chrome/browser/resources/settings/appearance_page/appearance_page.js
@@ -100,10 +100,10 @@ Polymer({
observers: [
'themeChanged_(prefs.extensions.theme.id.value, useSystemTheme_)',
-<if expr="is_linux and not chromeos">
+// <if expr="is_linux and not chromeos">
// NOTE: this pref only exists on Linux.
'useSystemThemePrefChanged_(prefs.extensions.theme.use_system.value)',
-</if>
+// </if>
],
created: function() {
@@ -155,7 +155,7 @@ Polymer({
window.open(this.themeUrl_ || loadTimeData.getString('themesGalleryUrl'));
},
-<if expr="chromeos">
+// <if expr="chromeos">
/**
* ChromeOS only.
* @private
@@ -163,14 +163,14 @@ Polymer({
openWallpaperManager_: function() {
this.browserProxy_.openWallpaperManager();
},
-</if>
+// </if>
/** @private */
onUseDefaultTap_: function() {
this.browserProxy_.useDefaultTheme();
},
-<if expr="is_linux and not chromeos">
+// <if expr="is_linux and not chromeos">
/**
* @param {boolean} useSystemTheme
* @private
@@ -215,7 +215,7 @@ Polymer({
onUseSystemTap_: function() {
this.browserProxy_.useSystemTheme();
},
-</if>
+// </if>
/**
* @param {string} themeId
@@ -235,12 +235,12 @@ Polymer({
}
var i18nId;
-<if expr="is_linux and not chromeos">
+// <if expr="is_linux and not chromeos">
i18nId = useSystemTheme ? 'systemTheme' : 'classicTheme';
-</if>
-<if expr="not is_linux or chromeos">
+// </if>
+// <if expr="not is_linux or chromeos">
i18nId = 'chooseFromWebStore';
-</if>
+// </if>
this.themeSublabel_ = this.i18n(i18nId);
this.themeUrl_ = '';
},

Powered by Google App Engine
This is Rietveld 408576698