Merge pull request #15029 from emberjs/cache-factory-for-injections

[PERF] [BUGFIX Beta] cache factoryFor injections when possible
emberjs · Mar 16, 2017 · 5fba442 · 5fba442
2 parents 86dd9d2 + c807ac5
commit 5fba442
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/packages/container/lib/container.js b/packages/container/lib/container.js
@@ -642,10 +642,18 @@ class FactoryManager {
     this.fullName = fullName;
     this.normalizedName = normalizedName;
     this.madeToString = undefined;
+    this.injections = undefined;
   }
 
   create(options = {}) {
-    let injections = injectionsFor(this.container, this.normalizedName);
+
+    let injections = this.injections;
+    if (injections === undefined) {
+      injections = injectionsFor(this.container, this.normalizedName);
+      if (areInjectionsDynamic(injections) === false) {
+        this.injections = injections;
+      }
+    }
     let props = assign({}, injections, options);
 
     props[NAME_KEY] = this.madeToString || (this.madeToString = this.container.registry.makeToString(this.class, this.fullName));