feat($compile): put custom annotations on DDO

Closes angular#14369
Closes angular#14279
Closes angular#14284

Author: petebacondarwin

src/ng/compile.js

@@ -1106,7 +1106,7 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
       }
 
       var template = (!options.template && !options.templateUrl ? '' : options.template);
-      return {
+      var ddo = {
         controller: controller,
         controllerAs: identifierForController(options.controller) || options.controllerAs || '$ctrl',
         template: makeInjectable(template),
@@ -1117,9 +1117,21 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
         restrict: 'E',
         require: options.require
       };
+
+      // Copy annotations (starting with $) over to the DDO
+      forEach(options, function(val, key) {
+        if (key.charAt(0) === '$') ddo[key] = val;
+      });
+
+      return ddo;
     }
 
-    // Copy any annotation properties (starting with $) over to the factory function
+    // TODO(pete) remove the following `forEach` before we release 1.6.0
+    // The [email protected] looks for the annotations on the controller constructor
+    // Nothing in Angular looks for annotations on the factory function but we can't remove
+    // it from 1.5.x yet.
+
+    // Copy any annotation properties (starting with $) over to the factory and controller constructor functions
     // These could be used by libraries such as the new component router
     forEach(options, function(val, key) {
       if (key.charAt(0) === '$') {

test/ng/compileSpec.js

@@ -10493,15 +10493,15 @@ describe('$compile', function() {
       }));
     });
 
-    it('should add additional annotations to the controller constructor', function() {
+    it('should expose additional annotations on the directive definition object', function() {
       var myModule = angular.module('my', []).component('myComponent', {
         $canActivate: 'canActivate',
         $routeConfig: 'routeConfig',
         $customAnnotation: 'XXX'
       });
       module('my');
       inject(function(myComponentDirective) {
-        expect(myComponentDirective[0].controller).toEqual(jasmine.objectContaining({
+        expect(myComponentDirective[0]).toEqual(jasmine.objectContaining({
           $canActivate: 'canActivate',
           $routeConfig: 'routeConfig',
           $customAnnotation: 'XXX'
@@ -10515,9 +10515,11 @@ describe('$compile', function() {
         controller: 'SomeNamedController'
       });
       module('my');
-      expect(function() {
-        inject(function(myComponentDirective) {});
-      }).not.toThrow();
+      inject(function(myComponentDirective) {
+        expect(myComponentDirective[0]).toEqual(jasmine.objectContaining({
+          $customAnnotation: 'XXX'
+        }));
+      });
     });
 
     it('should return ddo with reasonable defaults', function() {