fix($browser): normalize inputted URLs

Fixes #16100
Fixes #16592

Author: jbedard

src/ng/browser.js

@@ -131,6 +131,15 @@ function Browser(window, document, $log, $sniffer) {
     // setter
     if (url) {
       var sameState = lastHistoryState === state;
+      var hasTrailingSlash = url[url.length - 1] === '/';
+
+      // Normalize the inputted URL
+      url = urlResolve(url).href;
+
+      // ... but keep the (lack of) trailing '/' consistent
+      if (!hasTrailingSlash && url[url.length - 1] === '/') {
+        url = url.slice(0, url.length - 1);
+      }
 
       // Don't change anything if previous and current URLs and states match. This also prevents
       // IE<10 from getting into redirect loop when in LocationHashbangInHtml5Url mode.

test/ng/browserSpecs.js

@@ -1013,7 +1013,7 @@ describe('browser', function() {
     it('should not interfere with legacy browser url replace behavior', function() {
       inject(function($rootScope) {
         var current = fakeWindow.location.href;
-        var newUrl = 'notyet';
+        var newUrl = 'http://notyet';
         sniffer.history = false;
         expect(historyEntriesLength).toBe(1);
         browser.url(newUrl, true);