fix($location): dont throw while parsing invalid URLs

Fixes #6836

Author: pkozlowski-opensource

src/ng/location.js

@@ -37,10 +37,10 @@ function parseAppUrl(relativeUrl, locationObj, appBase) {
     relativeUrl = '/' + relativeUrl;
   }
   var match = urlResolve(relativeUrl, appBase);
-  locationObj.$$path = decodeURIComponent(prefixed && match.pathname.charAt(0) === '/' ?
+  locationObj.$$path = tryDecodeURIComponent(prefixed && match.pathname.charAt(0) === '/' ?
       match.pathname.substring(1) : match.pathname);
   locationObj.$$search = parseKeyValue(match.search);
-  locationObj.$$hash = decodeURIComponent(match.hash);
+  locationObj.$$hash = tryDecodeURIComponent(match.hash);
 
   // make sure path starts with '/';
   if (locationObj.$$path && locationObj.$$path.charAt(0) != '/') {

test/ng/locationSpec.js

@@ -459,6 +459,16 @@ describe('$location', function() {
         expect(url.hash()).toBe('x <>#');
       });
 
+
+      it('should not break on invalid URLs', function() {
+        url = new LocationHtml5Url('http://example.com/');
+        url.$$parse('http://example.com/%?filter=%%U65-*55z65>ß.2á.3á02312-3-214.32@#%7B%7D%5D%5B%%%');
+        expect(url.path()).toBe('/');
+        expect(url.search()).toEqual({'filter': undefined});
+        expect(url.hash()).toBeUndefined();
+      });
+
+
       it('should decode pluses as spaces in urls', function() {
         url = new LocationHtml5Url('http://host.com/');
         url.$$parse('http://host.com/?a+b=c+d');