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 5571b2d249929d5d2f22da0ac953f2ef182dc7fd..b34704d198cb36af75d8c8a8b6d7d0525c4b404e 100644 |
--- a/chrome/browser/resources/settings/appearance_page/appearance_page.js |
+++ b/chrome/browser/resources/settings/appearance_page/appearance_page.js |
@@ -94,6 +94,9 @@ Polymer({ |
themeSublabel_: String, |
/** @private */ |
+ themeUrl_: String, |
+ |
+ /** @private */ |
useSystemTheme_: { |
type: Boolean, |
value: false, // Can only be true on Linux, but value exists everywhere. |
@@ -115,9 +118,6 @@ Polymer({ |
/** @private {?settings.AppearanceBrowserProxy} */ |
browserProxy_: null, |
- /** @private {string} */ |
- themeUrl_: '', |
- |
observers: [ |
'defaultFontSizeChanged_(prefs.webkit.webprefs.default_font_size.value)', |
'themeChanged_(prefs.extensions.theme.id.value, useSystemTheme_)', |
@@ -187,9 +187,12 @@ Polymer({ |
value - SIZE_DIFFERENCE_FIXED_STANDARD_); |
}, |
- /** @private */ |
- onThemesTap_: function() { |
- window.open(this.themeUrl_ || loadTimeData.getString('themesGalleryUrl')); |
+ /** |
+ * URL for either current theme or the theme gallery. |
dschuyler
2017/04/20 18:50:04
@return {string}
scottchen
2017/04/20 21:48:34
Done. Sorry, missed it the first time you said it.
|
+ * @private |
+ */ |
+ getThemeHref_: function() { |
+ return this.themeUrl_ || loadTimeData.getString('themesGalleryUrl'); |
}, |
// <if expr="chromeos"> |