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

Unified Diff: chrome/browser/resources/settings/appearance_page/appearance_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_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..192837c5c8e84839388b7d544a1de01965347500 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
@@ -208,14 +208,14 @@ Polymer({
*/
showThemesSecondary_: function(themeId, useSystemTheme) {
return this.showUseClassic_(themeId, useSystemTheme) ||
- this.showUseSystem_(themeId, useSystemTheme);
+ this.showUseSystem_(themeId, useSystemTheme);
},
/** @private */
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