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

Side by Side Diff: ui/webui/resources/js/event_tracker.js

Issue 575333002: Compile print_preview, part 2: reduce down to 260 errors (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@I_print_preview
Patch Set: rebase Created 6 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * @fileoverview EventTracker is a simple class that manages the addition and 6 * @fileoverview EventTracker is a simple class that manages the addition and
7 * removal of DOM event listeners. In particular, it keeps track of all 7 * removal of DOM event listeners. In particular, it keeps track of all
8 * listeners that have been added and makes it easy to remove some or all of 8 * listeners that have been added and makes it easy to remove some or all of
9 * them without requiring all the information again. This is particularly handy 9 * them without requiring all the information again. This is particularly handy
10 * when the listener is a generated function such as a lambda or the result of 10 * when the listener is a generated function such as a lambda or the result of
(...skipping 22 matching lines...) Expand all
33 /** 33 /**
34 * @type {Array.<EventTrackerEntry>} 34 * @type {Array.<EventTrackerEntry>}
35 * @private 35 * @private
36 */ 36 */
37 this.listeners_ = []; 37 this.listeners_ = [];
38 } 38 }
39 39
40 EventTracker.prototype = { 40 EventTracker.prototype = {
41 /** 41 /**
42 * Add an event listener - replacement for Node.addEventListener. 42 * Add an event listener - replacement for Node.addEventListener.
43 * @param {!Node} node The DOM node to add a listener to. 43 * @param {!EventTarget} node The DOM node to add a listener to.
Dan Beam 2014/09/23 00:46:34 shouldn't we rename to target everywhere?
Vitaly Pavlenko 2014/09/23 01:05:17 Done inside this function. Or you mean everywhere
44 * @param {string} eventType The type of event to subscribe to. 44 * @param {string} eventType The type of event to subscribe to.
45 * @param {EventListener|Function} listener The listener to add. 45 * @param {EventListener|Function} listener The listener to add.
46 * @param {boolean=} opt_capture Whether to invoke during the capture phase. 46 * @param {boolean=} opt_capture Whether to invoke during the capture phase.
47 */ 47 */
48 add: function(node, eventType, listener, opt_capture) { 48 add: function(node, eventType, listener, opt_capture) {
49 var capture = !!opt_capture; 49 var capture = !!opt_capture;
50 var h = { 50 var h = {
51 node: node, 51 node: node,
52 eventType: eventType, 52 eventType: eventType,
53 listener: listener, 53 listener: listener,
(...skipping 29 matching lines...) Expand all
83 83
84 /** 84 /**
85 * Remove a single event listener given it's tracking entry. It's up to the 85 * Remove a single event listener given it's tracking entry. It's up to the
86 * caller to ensure the entry is removed from listeners_. 86 * caller to ensure the entry is removed from listeners_.
87 * @param {EventTrackerEntry} h The entry describing the listener to remove. 87 * @param {EventTrackerEntry} h The entry describing the listener to remove.
88 * @private 88 * @private
89 */ 89 */
90 EventTracker.removeEventListener_ = function(h) { 90 EventTracker.removeEventListener_ = function(h) {
91 h.node.removeEventListener(h.eventType, h.listener, h.capture); 91 h.node.removeEventListener(h.eventType, h.listener, h.capture);
92 }; 92 };
OLDNEW
« ui/webui/resources/js/cr/event_target.js ('K') | « ui/webui/resources/js/cr/event_target.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698