directive name should be trimmed

[jtorbicki]

Directive name should probably be either trimmed or angular should throw an exception, so a typo is easier to spot. Leading or following space is clearly an error.

In the following example the name is 'directiveName ' instead of 'directiveName':

https://jsfiddle.net/c2ztgdjr/

Maybe same rule should be applied to controllers, factories etc, although it's still possible to inject those with an extra space.

[pkozlowski-opensource]

I would say we should throw. This would be an easy fix in the compiler, see

angular.js/src/ng/compile.js

Line 800 in 74eb17d

function assertValidDirectiveName(name) {

Anyone fancy sending a PR with an associated test?

[lugovsky]

@pkozlowski-opensource Hey! Please check the PR for this issue. Thanks!