From 848e8c9be6a72d12c4d2891363bfe27d6ce14675 Mon Sep 17 00:00:00 2001 From: Kevin Schaaf Date: Thu, 19 Sep 2019 18:49:09 -0700 Subject: [PATCH] Revert optimization to not wrap change notifications. This was causing a number of rendering tests to fail. Needs investigation, but possibly because wrapping calls ShadyDOM.flush, and this alters distribution timing which some tests may have inadvertently relied on. --- lib/mixins/property-effects.js | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lib/mixins/property-effects.js b/lib/mixins/property-effects.js index 5da858450f..ee210592c6 100644 --- a/lib/mixins/property-effects.js +++ b/lib/mixins/property-effects.js @@ -320,8 +320,7 @@ function dispatchNotifyEvent(inst, eventName, value, path) { if (path) { detail.path = path; } - // Note, since this does not bubble, it should not need to be wrapped. - inst.dispatchEvent(new CustomEvent(eventName, { detail })); + wrap(/** @type {!HTMLElement} */(inst)).dispatchEvent(new CustomEvent(eventName, { detail })); } /**