Index: chrome/browser/resources/options/settings_banner.js |
diff --git a/chrome/browser/resources/options/settings_banner.js b/chrome/browser/resources/options/settings_banner.js |
index 5d030a4a309195e60571112345daaeac22f7d553..dc96c08b864a10de2ad538e5331c4d8e3f4b4dde 100644 |
--- a/chrome/browser/resources/options/settings_banner.js |
+++ b/chrome/browser/resources/options/settings_banner.js |
@@ -9,7 +9,6 @@ |
/** |
* Base class for banners that appear at the top of the settings page. |
- * @constructor |
*/ |
function SettingsBannerBase() {} |
@@ -50,13 +49,37 @@ |
setVisibilibyDomElement_: null, |
/** |
+ * Called by the native code to show the banner if needed. |
+ * @private |
+ */ |
+ show_: function() { |
+ if (!this.hadBeenDismissed_) { |
+ chrome.send('metricsHandler:recordAction', [this.showMetricName_]); |
+ this.setVisibility_(true); |
+ } |
+ }, |
+ |
+ /** |
+ * Called when the banner should be closed as a result of something taking |
+ * place on the WebUI page, i.e. when its close button is pressed, or when |
+ * the confirmation dialog for the profile settings reset feature is opened. |
+ * @private |
+ */ |
+ dismiss_: function() { |
+ chrome.send(this.dismissNativeCallbackName_); |
+ this.hadBeenDismissed_ = true; |
+ this.setVisibility_(false); |
+ }, |
+ |
+ /** |
* Sets whether or not the reset profile settings banner shall be visible. |
* @param {boolean} show Whether or not to show the banner. |
- * @protected |
+ * @private |
*/ |
- setVisibility: function(show) { |
+ setVisibility_: function(show) { |
this.setVisibilibyDomElement_.hidden = !show; |
}, |
+ |
}; |
// Export |