Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(133)

Unified Diff: src/polymer-expressions.js

Issue 61630044: resolveEventReceiver -> resolveEventHandler Base URL: https://github.com/Polymer/polymer-expressions.git@master
Patch Set: Created 11 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | tests/tests.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/polymer-expressions.js
diff --git a/src/polymer-expressions.js b/src/polymer-expressions.js
index 4ebeb988c5df84b9cdbbac5d314ce49015e5f1d5..b9954016bf782fa95d94330244e3eb247a1c321c 100644
--- a/src/polymer-expressions.js
+++ b/src/polymer-expressions.js
@@ -543,25 +543,26 @@
var eventType = name.substring(3);
eventType = mixedCaseEventTypes[eventType] || eventType;
- var resolveReceiver = resolveEventReceiver;
-
- if (typeof polymerExpressions.resolveEventReceiver == 'function') {
- resolveReceiver = function(model, path, node) {
- return polymerExpressions.resolveEventReceiver(model, path, node);
- };
- }
-
return function(model, node, oneTime) {
- var fn, receiver;
-
- function handler(e) {
- fn = fn || path.getValueFrom(model);
- receiver = receiver || resolveReceiver(model, path, node);
+ var fn, receiver, handler;
+ if (typeof polymerExpressions.resolveEventHandler == 'function') {
+ handler = function(e) {
+ fn = fn || polymerExpressions.resolveEventHandler(model, path, node);
+ fn(e, e.detail, e.currentTarget);
+
+ if (Platform && typeof Platform.flush == 'function')
+ Platform.flush();
+ };
+ } else {
+ handler = function(e) {
+ fn = fn || path.getValueFrom(model);
+ receiver = receiver || resolveEventReceiver(model, path, node);
- fn.apply(receiver, [e, e.detail, e.currentTarget]);
+ fn.apply(receiver, [e, e.detail, e.currentTarget]);
- if (Platform && typeof Platform.flush == 'function')
- Platform.flush();
+ if (Platform && typeof Platform.flush == 'function')
+ Platform.flush();
+ };
}
node.addEventListener(eventType, handler);
« no previous file with comments | « no previous file | tests/tests.js » ('j') | no next file with comments »

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b