fix(dropdown): compatibility with $location url rewriting

Fixes #2343

Author: chrisirhc

src/dropdown/dropdown.js

@@ -29,6 +29,10 @@ angular.module('ui.bootstrap.dropdown', [])
   };
 
   var closeDropdown = function( evt ) {
+    // This method may still be called during the same mouse event that
+    // unbound this event handler. So check openScope before proceeding.
+    if (!openScope) { return; }
+
     var toggleElement = openScope.getToggleElement();
     if ( evt && toggleElement && toggleElement[0].contains(evt.target) ) {
         return;

src/dropdown/test/dropdown.spec.js

@@ -168,6 +168,33 @@ describe('dropdownToggle', function() {
     });
   });
 
+  describe('integration with $location URL rewriting', function() {
+    function dropdown() {
+
+      // Simulate URL rewriting behavior
+      $document.on('click', 'a[href="#something"]', function () {
+        $rootScope.$broadcast('$locationChangeSuccess');
+        $rootScope.$apply();
+      });
+
+      return $compile('<li dropdown><a href dropdown-toggle></a>' +
+        '<ul><li><a href="#something">Hello</a></li></ul></li>')($rootScope);
+    }
+
+    beforeEach(function() {
+      element = dropdown();
+    });
+
+    it('should close without errors on $location change', function() {
+      $document.find('body').append(element);
+      clickDropdownToggle();
+      expect(element.hasClass('open')).toBe(true);
+      var optionEl = element.find('ul > li').eq(0).find('a').eq(0);
+      optionEl.click();
+      expect(element.hasClass('open')).toBe(false);
+    });
+  });
+
   describe('without trigger', function() {
     beforeEach(function() {
       $rootScope.isopen = true;