Do not call Map/Set directly with iterables

canjs · Sep 5, 2018 · ac50344 · ac50344
1 parent 2a1e16d
commit ac50344
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/key/key.js b/key/key.js
@@ -72,11 +72,13 @@ module.exports = function keyObservable(root, keyPath) {
 
 		// Register what this observable changes
 		symbolsToAssign["can.getWhatIChange"] = function getWhatIChange() {
+			var m = new Map();
+			var s = new Set();
+			s.add(lastKey);
+			m.set(lastParent, s);
 			return {
 				mutate: {
-					keyDependencies: new Map([
-						[lastParent, new Set([lastKey])]
-					])
+					keyDependencies: m
 				}
 			};
 		};

diff --git a/resolver/resolver.js b/resolver/resolver.js
@@ -96,8 +96,10 @@ canReflect.assignMap(ResolverObservable.prototype, {
 
 		var contextHandler = handler.bind(this.context);
 		contextHandler[getChangesSymbol] = function getChangesDependencyRecord() {
+			var s = new Set();
+			s.add(resolverInstance);
 			return {
-				valueDependencies: new Set([ resolverInstance ])
+				valueDependencies: s
 			};
 		};