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

Unified Diff: chrome/browser/resources/print_preview/component.js

Issue 2935303003: Print Preview: Remove global onPrivetPrintFailed (Closed)
Patch Set: Fix annotations Created 3 years, 6 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/print_preview/component.js
diff --git a/chrome/browser/resources/print_preview/component.js b/chrome/browser/resources/print_preview/component.js
index 5431b74e5cae69c4053a2d5f8632b73f85ad6c2c..13a7a2f2df39e523a6f866f0344f20fc9105343f 100644
--- a/chrome/browser/resources/print_preview/component.js
+++ b/chrome/browser/resources/print_preview/component.js
@@ -15,8 +15,7 @@ cr.define('print_preview', function() {
/**
* Component's HTML element.
- * @type {Element}
- * @private
+ * @private {Element}
*/
this.element_ = null;
@@ -24,15 +23,19 @@ cr.define('print_preview', function() {
/**
* Component's event tracker.
- * @type {EventTracker}
- * @private
+ * @private {!EventTracker}
*/
this.tracker_ = new EventTracker();
+ /**
+ * Component's WebUI listener tracker.
+ * @private {!WebUIListenerTracker}
+ */
+ this.listenerTracker_ = new WebUIListenerTracker();
+
/**
* Child components of the component.
- * @type {!Array<!print_preview.Component>}
- * @private
+ * @private {!Array<!print_preview.Component>}
*/
this.children_ = [];
}
@@ -45,11 +48,16 @@ cr.define('print_preview', function() {
return this.element_;
},
- /** @return {EventTracker} Component's event tracker. */
+ /** @return {!EventTracker} Component's event tracker. */
get tracker() {
return this.tracker_;
},
+ /** @return {!WebUIListenerTracker} Component's Web UI listener tracker. */
+ get listenerTracker() {
+ return this.listenerTracker_;
+ },
+
/**
* @return {boolean} Whether the element of the component is already in the
* HTML document.
@@ -88,6 +96,7 @@ cr.define('print_preview', function() {
}
});
this.tracker_.removeAll();
+ this.listenerTracker_.removeAll();
this.isInDocument_ = false;
},

Powered by Google App Engine
This is Rietveld 408576698