fix($location): don't rewrite hash-fragment links in same document

$location will no longer rewrite hash fragment links in the same document (specified via href="#hash...").

Closes angular#8478

Author: caitp

src/ng/location.js

@@ -641,6 +641,13 @@ function $LocationProvider(){
         if (elm[0] === $rootElement[0] || !(elm = elm.parent())[0]) return;
       }
 
+      var href = elm.attr('href') || elm.attr('xlink:href');
+
+      if (href && href.indexOf('#' + hashPrefix) !== 0 && href[0] === '#') {
+        // Don't rewrite hash-fragment links in the same document.
+        return;
+      }
+
       var absHref = elm.prop('href');
 
       if (isObject(absHref) && absHref.toString() === '[object SVGAnimatedString]') {

test/ng/locationSpec.js

@@ -1164,6 +1164,32 @@ describe('$location', function() {
     });
 
 
+    it('should not rewrite when clicking on relative hash fragments', function() {
+      configureService('#foo', true, true, true);
+      inject(
+        initBrowser(),
+        initLocation(),
+        function($browser) {
+          browserTrigger(link, 'click');
+          expectNoRewrite($browser);
+        }
+      );
+    });
+
+
+    it('should not rewrite when clicking on relative hash fragments in old browser', function() {
+      configureService('#foo', true, false, true);
+      inject(
+        initBrowser(),
+        initLocation(),
+        function($browser) {
+          browserTrigger(link, 'click');
+          expectNoRewrite($browser);
+        }
+      );
+    });
+
+
     // don't run next tests on IE<9, as browserTrigger does not simulate pressed keys
     if (!msie || msie >= 9) {