DirectiveInjector (todo tracking)

[vicb]

• DirectiveInjector implements DirectiveBinder but bind() args do not match
• DirectiveInjector maintains a depth but there is no assertion / check for a max depth

[rkirov]

#1244 adds max depth for DirectiveInjector

[vicb]

@rkirov is max depth intended to support detecting circular dependency ? if yes, there should be a unit test.

bind() (1st item no the list) has been fixed in f8bbd35

[rkirov]

There are two different notions of depth, so I think I got confused.

• Nested views injection fix #1244 adds injector.parent resolution depth (not likely to be ever hit).
• feat(directive-injector): detect and throw on circular deps #1364 checks for internal resolution depth (since we have 20-30 fields max, it can only be caused by circular dependency).

[vicb]

Closing this as defect are solved (or PR pending)