refactor($compile): combining elementControllers and controllers

Author: jbedard

src/ng/compile.js

@@ -1566,7 +1566,6 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
       var terminalPriority = -Number.MAX_VALUE,
           newScopeDirective,
           controllerDirectives = previousCompileContext.controllerDirectives,
-          controllers,
           newIsolateScopeDirective = previousCompileContext.newIsolateScopeDirective,
           templateDirective = previousCompileContext.templateDirective,
           nonTlbTranscludeDirective = previousCompileContext.nonTlbTranscludeDirective,
@@ -1862,8 +1861,6 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
         }
 
         if (controllerDirectives) {
-          // TODO: merge `controllers` and `elementControllers` into single object.
-          controllers = {};
           elementControllers = {};
           forEach(controllerDirectives, function(directive) {
             var locals = {
@@ -1889,8 +1886,6 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
             if (!hasElementTranscludeDirective) {
               $element.data('$' + directive.name + 'Controller', controllerInstance.instance);
             }
-
-            controllers[directive.name] = controllerInstance;
           });
         }
 
@@ -1899,7 +1894,7 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
               templateDirective === newIsolateScopeDirective.$$originalDirective)));
           compile.$$addScopeClass($element, true);
 
-          var isolateScopeController = controllers && controllers[newIsolateScopeDirective.name];
+          var isolateScopeController = elementControllers && elementControllers[newIsolateScopeDirective.name];
           var isolateBindingContext = isolateScope;
           if (isolateScopeController && isolateScopeController.identifier &&
               newIsolateScopeDirective.bindToController === true) {
@@ -1977,11 +1972,10 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
             }
           });
         }
-        if (controllers) {
-          forEach(controllers, function(controller) {
+        if (elementControllers) {
+          forEach(elementControllers, function(controller) {
             controller();
           });
-          controllers = null;
         }
 
         // PRELINKING