refactor(dropdownToggle): listen to $locationChangeSuccess

 * Listen to  event instead of adding an extra `$watch` to the `$digest` cycle.
 * Remove `$location` DI, that sometimes caused weird issues.

Closes #1585

Author: bekos

src/dropdownToggle/dropdownToggle.js

@@ -12,13 +12,13 @@
    </li>
  */
 
-angular.module('ui.bootstrap.dropdownToggle', []).directive('dropdownToggle', ['$document', '$location', function ($document, $location) {
+angular.module('ui.bootstrap.dropdownToggle', []).directive('dropdownToggle', ['$document', function ($document) {
   var openElement = null,
       closeMenu   = angular.noop;
   return {
     restrict: 'CA',
     link: function(scope, element, attrs) {
-      scope.$watch('$location.path', function() { closeMenu(); });
+      scope.$on('$locationChangeSuccess', function() { closeMenu(); });
       element.parent().bind('click', function() { closeMenu(); });
       element.bind('click', function (event) {
 

src/dropdownToggle/test/dropdownToggle.spec.js

@@ -50,7 +50,8 @@ describe('dropdownToggle', function() {
   it('should close on $location change', function() {
     var elm = dropdown();
     elm.find('a').click();
-    $location.path('/foo');
+    expect(elm.hasClass('open')).toBe(true);
+    $rootScope.$broadcast('$locationChangeSuccess');
     $rootScope.$apply();
     expect(elm.hasClass('open')).toBe(false);
   });