perf(view factory): Compute DI Keys ahead of time

4% improvement in TreeComponent benchmark

For dart-archive#1081

Closes dart-archive#1085

Author: jbdeboer

lib/core_dom/common.dart

@@ -16,11 +16,12 @@ class MappingParts {
 class DirectiveRef {
   final dom.Node element;
   final Type type;
+  final Key typeKey;
   final Directive annotation;
   final String value;
   final mappings = new List<MappingParts>();
 
-  DirectiveRef(this.element, this.type, this.annotation, [ this.value ]);
+  DirectiveRef(this.element, this.type, this.annotation, this.typeKey, [ this.value ]);
 
   String toString() {
     var html = element is dom.Element

lib/core_dom/element_binder.dart

@@ -27,10 +27,10 @@ class TemplateElementBinder extends ElementBinder {
   _registerViewFactory(node, parentInjector, nodeModule) {
     assert(templateViewFactory != null);
     nodeModule
-      ..bind(ViewPort, toFactory: (_) =>
-          new ViewPort(node, parentInjector.get(Animate)))
-      ..bind(ViewFactory, toValue: templateViewFactory)
-      ..bind(BoundViewFactory, toFactory: (Injector injector) =>
+      ..bindByKey(_VIEW_PORT_KEY, toFactory: (_) =>
+          new ViewPort(node, parentInjector.getByKey(_ANIMATE_KEY)))
+      ..bindByKey(_VIEW_FACTORY_KEY, toValue: templateViewFactory)
+      ..bindByKey(_BOUND_VIEW_FACTORY_KEY, toFactory: (Injector injector) =>
           templateViewFactory.bind(injector));
   }
 }
@@ -203,7 +203,7 @@ class ElementBinder {
 
   void _link(nodeInjector, probe, scope, nodeAttrs, formatters) {
     _usableDirectiveRefs.forEach((DirectiveRef ref) {
-      var directive = nodeInjector.get(ref.type);
+      var directive = nodeInjector.getByKey(ref.typeKey);
       probe.directives.add(directive);
 
       if (ref.annotation is Controller) {
@@ -240,18 +240,18 @@ class ElementBinder {
   void _createDirectiveFactories(DirectiveRef ref, nodeModule, node, nodesAttrsDirectives, nodeAttrs,
                                  visibility) {
     if (ref.type == TextMustache) {
-      nodeModule.bind(TextMustache, toFactory: (Injector injector) {
-        return new TextMustache(node, ref.value, injector.get(Interpolate),
-            injector.get(Scope), injector.get(FormatterMap));
+      nodeModule.bindByKey(_TEXT_MUSTACHE_KEY, toFactory: (Injector injector) {
+        return new TextMustache(node, ref.value, injector.getByKey(_INTERPOLATE_KEY),
+            injector.getByKey(_SCOPE_KEY), injector.getByKey(_FORMATTER_MAP_KEY));
       });
     } else if (ref.type == AttrMustache) {
       if (nodesAttrsDirectives.isEmpty) {
         nodeModule.bind(AttrMustache, toFactory: (Injector injector) {
-          var scope = injector.get(Scope);
-          var interpolate = injector.get(Interpolate);
+          var scope = injector.getByKey(_SCOPE_KEY);
+          var interpolate = injector.getByKey(_INTERPOLATE_KEY);
           for (var ref in nodesAttrsDirectives) {
             new AttrMustache(nodeAttrs, ref.value, interpolate, scope,
-                injector.get(FormatterMap));
+                injector.getByKey(_FORMATTER_MAP_KEY));
           }
         });
       }
@@ -266,23 +266,24 @@ class ElementBinder {
       } else {
         factory = _componentFactory;
       }
-      nodeModule.bind(ref.type, toFactory: factory.call(node, ref), visibility: visibility);
+      nodeModule.bindByKey(ref.typeKey, toFactory: factory.call(node, ref), visibility: visibility);
     } else {
-      nodeModule.bind(ref.type, visibility: visibility);
+      nodeModule.bindByKey(ref.typeKey, visibility: visibility);
     }
   }
 
   // Overridden in TemplateElementBinder
   void _registerViewFactory(node, parentInjector, nodeModule) {
-    nodeModule..bind(ViewPort, toValue: null)
-              ..bind(ViewFactory, toValue: null)
-              ..bind(BoundViewFactory, toValue: null);
+    nodeModule..bindByKey(_VIEW_PORT_KEY, toValue: null)
+              ..bindByKey(_VIEW_FACTORY_KEY, toValue: null)
+              ..bindByKey(_BOUND_VIEW_FACTORY_KEY, toValue: null);
   }
 
+
   Injector bind(View view, Injector parentInjector, dom.Node node) {
     Injector nodeInjector;
-    Scope scope = parentInjector.get(Scope);
-    FormatterMap formatters = parentInjector.get(FormatterMap);
+    Scope scope = parentInjector.getByKey(_SCOPE_KEY);
+    FormatterMap formatters = parentInjector.getByKey(_FORMATTER_MAP_KEY);
     var nodeAttrs = node is dom.Element ? new NodeAttrs(node) : null;
     ElementProbe probe;
 
@@ -291,12 +292,12 @@ class ElementBinder {
 
     var nodesAttrsDirectives = [];
     var nodeModule = new Module()
-        ..bind(NgElement)
-        ..bind(View, toValue: view)
-        ..bind(dom.Element, toValue: node)
-        ..bind(dom.Node, toValue: node)
-        ..bind(NodeAttrs, toValue: nodeAttrs)
-        ..bind(ElementProbe, toFactory: (_) => probe);
+        ..bindByKey(_NG_ELEMENT_KEY)
+        ..bindByKey(_VIEW_KEY, toValue: view)
+        ..bindByKey(_ELEMENT_KEY, toValue: node)
+        ..bindByKey(_NODE_KEY, toValue: node)
+        ..bindByKey(_NODE_ATTRS_KEY, toValue: nodeAttrs)
+        ..bindByKey(_ELEMENT_PROBE_KEY, toFactory: (_) => probe);
 
     directiveRefs.forEach((DirectiveRef ref) {
       Directive annotation = ref.annotation;
@@ -317,7 +318,7 @@ class ElementBinder {
 
     nodeInjector = parentInjector.createChild([nodeModule]);
     probe = _expando[node] = new ElementProbe(
-        parentInjector.get(ElementProbe), node, nodeInjector, scope);
+        parentInjector.getByKey(_ELEMENT_PROBE_KEY), node, nodeInjector, scope);
 
     _link(nodeInjector, probe, scope, nodeAttrs, formatters);
 

lib/core_dom/module_internal.dart

@@ -46,6 +46,7 @@ part 'transcluding_component_factory.dart';
 part 'tree_sanitizer.dart';
 part 'walking_compiler.dart';
 part 'ng_element.dart';
+part 'static_keys.dart';
 
 class CoreDomModule extends Module {
   CoreDomModule() {

lib/core_dom/selector.dart

@@ -96,7 +96,7 @@ class DirectiveSelector {
           // the value. Yes it is a bit of a hack.
           _directives[selectorRegExp.annotation].forEach((type) {
             builder.addDirective(new DirectiveRef(
-                node, type, selectorRegExp.annotation, '$attrName=$value'));
+                node, type, selectorRegExp.annotation, new Key(type), '$attrName=$value'));
           });
         }
       }
@@ -131,7 +131,7 @@ class DirectiveSelector {
       if (selectorRegExp.regexp.hasMatch(value)) {
         _directives[selectorRegExp.annotation].forEach((type) {
           builder.addDirective(new DirectiveRef(node, type,
-              selectorRegExp.annotation, value));
+              selectorRegExp.annotation, new Key(type), value));
         });
       }
     }
@@ -204,7 +204,7 @@ class _SelectorPart {
 _addRefs(ElementBinderBuilder builder, List<_Directive> directives, dom.Node node,
          [String attrValue]) {
   directives.forEach((directive) {
-    builder.addDirective(new DirectiveRef(node, directive.type, directive.annotation, attrValue));
+    builder.addDirective(new DirectiveRef(node, directive.type, directive.annotation, new Key(directive.type), attrValue));
   });
 }
 

lib/core_dom/static_keys.dart

@@ -0,0 +1,30 @@
+part of angular.core.dom_internal;
+
+// Keys used to call Injector.getByKey and Module.bindByKey
+
+Key _ANIMATE_KEY = new Key(Animate);
+Key _BOUND_VIEW_FACTORY_KEY = new Key(BoundViewFactory);
+Key _COMPONENT_CSS_REWRITER_KEY = new Key(ComponentCssRewriter);
+Key _DIRECTIVE_MAP_KEY = new Key(DirectiveMap);
+Key _ELEMENT_KEY = new Key(dom.Element);
+Key _ELEMENT_PROBE_KEY = new Key(ElementProbe);
+Key _ELEMENT_PROBE = new Key(ElementProbe);
+Key _EVENT_HANDLER_KEY = new Key(EventHandler);
+Key _FORMATTER_MAP_KEY = new Key(FormatterMap);
+Key _HTTP_KEY = new Key(Http);
+Key _INTERPOLATE_KEY = new Key(Interpolate);
+Key _NG_BASE_CSS_KEY = new Key(NgBaseCss);
+Key _NG_ELEMENT_KEY = new Key(NgElement);
+Key _NODE_ATTRS_KEY = new Key(NodeAttrs);
+Key _NODE_KEY = new Key(dom.Node);
+Key _NODE_TREE_SANITIZER_KEY = new Key(dom.NodeTreeSanitizer);
+Key _SCOPE_KEY = new Key(Scope);
+Key _SHADOW_ROOT_KEY = new Key(dom.ShadowRoot);
+Key _TEMPLATE_CACHE_KEY = new Key(TemplateCache);
+Key _TEMPLATE_LOADER_KEY = new Key(TemplateLoader);
+Key _TEXT_MUSTACHE_KEY = new Key(TextMustache);
+Key _VIEW_CACHE_KEY = new Key(ViewCache);
+Key _VIEW_FACTORY_KEY = new Key(ViewFactory);
+Key _VIEW_KEY = new Key(View);
+Key _VIEW_PORT_KEY = new Key(ViewPort);
+Key _WEB_PLATFORM_KEY = new Key(WebPlatform);

lib/core_dom/tagging_view_factory.dart

@@ -9,14 +9,16 @@ class TaggingViewFactory implements ViewFactory {
 
   BoundViewFactory bind(Injector injector) => new BoundViewFactory(this, injector);
 
+  static Key _EVENT_HANDLER_KEY = new Key(EventHandler);
+
   View call(Injector injector, [List<dom.Node> nodes /* TODO: document fragment */]) {
     if (nodes == null) {
       nodes = cloneElements(templateNodes);
     }
     var timerId;
     try {
       assert((timerId = _perf.startTimer('ng.view')) != false);
-      var view = new View(nodes, injector.get(EventHandler));
+      var view = new View(nodes, injector.getByKey(_EVENT_HANDLER_KEY));
       _link(view, nodes, injector);
       return view;
     } finally {

test/core_dom/element_binder_builder_spec.dart

@@ -38,7 +38,7 @@ main() => describe('ElementBinderBuilder', () {
   addDirective(selector) {
     directives.forEach((Directive annotation, Type type) {
       if (annotation.selector == selector)
-        b.addDirective(new DirectiveRef(node, type, annotation, null));
+        b.addDirective(new DirectiveRef(node, type, annotation, new Key(type), null));
     });
     b = b.binder;
   }