fix: Remove deprecated calls to DI bind(Type, inject[])

fixes #1259
closes #1260

Author: vicb

lib/core/module_internal.dart

@@ -40,15 +40,15 @@ class CoreModule extends Module {
     bind(FormatterMap);
     bind(Interpolate);
     bind(RootScope);
-    bind(Scope, inject: [RootScope]);
+    bind(Scope, toInstanceOf: RootScope);
     bind(ClosureMap, toFactory: () => throw "Must provide dynamic/static ClosureMap.", inject: []);
     bind(ScopeStats);
     bind(ScopeStatsEmitter);
     bind(ScopeStatsConfig);
     bind(Object, toValue: {}); // RootScope context
 
-    bind(Parser, inject: [DynamicParser]);
-    bind(ParserBackend, inject: [DynamicParserBackend]);
+    bind(Parser, toInstanceOf: DynamicParser);
+    bind(ParserBackend, toInstanceOf: DynamicParserBackend);
     bind(DynamicParser);
     bind(DynamicParserBackend);
     bind(Lexer);

lib/core_dom/module_internal.dart

@@ -72,14 +72,14 @@ class CoreDomModule extends Module {
     bind(Compiler, toImplementation: TaggingCompiler);
     bind(CompilerConfig);
 
-    bind(ComponentFactory, inject: [SHADOW_DOM_COMPONENT_FACTORY_KEY]);
+    bind(ComponentFactory, toInstanceOf: SHADOW_DOM_COMPONENT_FACTORY_KEY);
     bind(ShadowDomComponentFactory);
     bind(TranscludingComponentFactory);
     bind(Content);
     bind(ContentPort, toValue: null);
     bind(ComponentCssRewriter);
     bind(WebPlatform);
-    
+
     bind(Http);
     bind(UrlRewriter);
     bind(HttpBackend);

lib/mock/module.dart

@@ -63,7 +63,7 @@ class AngularMockModule extends Module {
     bind(MockHttpBackend);
     bind(Element, toValue: document.body);
     bind(Node, toValue: document.body);
-    bind(HttpBackend, inject:[MOCK_HTTP_BACKEND_KEY]);
+    bind(HttpBackend, toInstanceOf: MOCK_HTTP_BACKEND_KEY);
     bind(VmTurnZone, toFactory: () {
       return new VmTurnZone()
         ..onError = (e, s, LongStackTrace ls) => dump('EXCEPTION: $e\n$s\n$ls');

lib/tools/expression_extractor.dart

@@ -58,8 +58,8 @@ main(args) {
       ..bind(DynamicParser)
       ..bind(DartGetterSetterGen)
       ..bind(CacheRegister)
-      ..bind(Parser, inject: [DynamicParser])
-      ..bind(ParserBackend, inject: [DartGetterSetterGen]);
+      ..bind(Parser, toInstanceOf: DynamicParser)
+      ..bind(ParserBackend, toInstanceOf: DartGetterSetterGen);
   Injector injector = new ModuleInjector([module]);
 
   runZoned(() {

pubspec.lock

@@ -4,15 +4,15 @@ packages:
   analyzer:
     description: analyzer
     source: hosted
-    version: "0.15.7"
+    version: "0.18.0"
   args:
     description: args
     source: hosted
     version: "0.10.0+2"
   barback:
     description: barback
     source: hosted
-    version: "0.13.0"
+    version: "0.14.1+3"
   benchmark_harness:
     description: benchmark_harness
     source: hosted
@@ -24,11 +24,11 @@ packages:
   code_transformers:
     description: code_transformers
     source: hosted
-    version: "0.1.4+2"
+    version: "0.1.5"
   collection:
     description: collection
     source: hosted
-    version: "0.9.2"
+    version: "0.9.4"
   di:
     description: di
     source: hosted
@@ -44,7 +44,7 @@ packages:
   intl:
     description: intl
     source: hosted
-    version: "0.9.10"
+    version: "0.8.10+4"
   js:
     description: js
     source: hosted
@@ -56,15 +56,19 @@ packages:
   matcher:
     description: matcher
     source: hosted
-    version: "0.10.0"
+    version: "0.11.0"
+  meta:
+    description: meta
+    source: hosted
+    version: "0.8.8"
   mock:
     description: mock
     source: hosted
-    version: "0.11.0+1"
+    version: "0.11.0+2"
   path:
     description: path
     source: hosted
-    version: "1.2.0"
+    version: "1.2.1"
   perf_api:
     description: perf_api
     source: hosted
@@ -80,20 +84,24 @@ packages:
   source_maps:
     description: source_maps
     source: hosted
-    version: "0.9.0"
+    version: "0.9.3"
   stack_trace:
     description: stack_trace
     source: hosted
-    version: "0.9.3+2"
+    version: "1.0.2"
+  typed_mock:
+    description: typed_mock
+    source: hosted
+    version: "0.0.4"
   unittest:
     description: unittest
     source: hosted
-    version: "0.11.0"
+    version: "0.11.0+3"
   utf:
     description: utf
     source: hosted
     version: "0.9.0"
   web_components:
     description: web_components
     source: hosted
-    version: "0.3.4"
+    version: "0.3.5+1"

pubspec.yaml

@@ -19,7 +19,7 @@ dependencies:
   browser: '>=0.10.0 <0.11.0'
   code_transformers: '>=0.1.4+2 <0.2.0'
   collection: '>=0.9.1 <1.0.0'
-  di: '>=2.0.0 <3.0.0'
+  di: '>=2.0.1 <3.0.0'
   html5lib: '>=0.10.0 <0.11.0'
   intl: '>=0.8.7 <0.10.0'
   perf_api: '>=0.0.8 <0.1.0'

test/core/parser/generated_getter_setter_spec.dart

@@ -7,7 +7,7 @@ import 'generated_getter_setter.dart' as gen;
 main() {
   describe('hybrid getter-setter', () {
     beforeEachModule((Module module) {
-      module..bind(Parser, inject: [DynamicParser])
+      module..bind(Parser, toInstanceOf: DynamicParser)
             ..bind(ClosureMap, toValue: gen.closureMap);
     });
     parser_spec.main();