angular
diff --git a/‎packages/router/src/router.ts
Copy file name to clipboardExpand all lines: packages/router/src/router.ts
+12-1Lines changed: 12 additions & 1 deletion b/‎packages/router/src/router.ts
Copy file name to clipboardExpand all lines: packages/router/src/router.ts
+12-1Lines changed: 12 additions & 1 deletion
diff --git a/‎packages/router/test/integration.spec.ts
Copy file name to clipboardExpand all lines: packages/router/test/integration.spec.ts
+5-1Lines changed: 5 additions & 1 deletion b/‎packages/router/test/integration.spec.ts
Copy file name to clipboardExpand all lines: packages/router/test/integration.spec.ts
+5-1Lines changed: 5 additions & 1 deletion
diff --git a/‎packages/router/test/regression_integration.spec.ts
Copy file name to clipboardExpand all lines: packages/router/test/regression_integration.spec.ts
+3-8Lines changed: 3 additions & 8 deletions b/‎packages/router/test/regression_integration.spec.ts
Copy file name to clipboardExpand all lines: packages/router/test/regression_integration.spec.ts
+3-8Lines changed: 3 additions & 8 deletions
@@ -636,6 +636,12 @@ export class Router {
                            (this.onSameUrlNavigation === 'reload' ? true : urlTransition) &&
                            this.urlHandlingStrategy.shouldProcessUrl(t.rawUrl);
 
+                       // If the source of the navigation is from a browser event, the URL is
+                       // already updated. We already need to sync the internal state.
+                       if (t.source !== 'imperative') {
+                         this.browserUrlTree = t.rawUrl;
+                       }
+
                        if (processCurrentUrl) {
                          return of(t).pipe(
                              // Fire NavigationStart event
@@ -954,7 +960,12 @@ export class Router {
                                  this.urlHandlingStrategy.merge(e.url, this.rawUrlTree);
                              const extras = {
                                skipLocationChange: t.extras.skipLocationChange,
-                               replaceUrl: this.urlUpdateStrategy === 'eager'
+                               // The URL is already updated at this point if we have 'eager' URL
+                               // updates or if the navigation was triggered by the browser (back
+                               // button, URL bar, etc). We want to replace that item in history if
+                               // the navigation is rejected.
+                               replaceUrl:
+                                   this.urlUpdateStrategy === 'eager' || t.source !== 'imperative'
                              };
 
                              this.scheduleNavigation(
 
@@ -1230,7 +1230,11 @@ describe('Integration', () => {
          location.simulateHashChange('/blocked');
 
          advance(fixture);
-         expectEvents(recordedEvents, [[NavigationStart, '/blocked']]);
+         expectEvents(recordedEvents, [
+           [NavigationStart, '/blocked'],
+           [NavigationStart, '/simple'],
+           [NavigationEnd, '/simple'],
+         ]);
        }));
   });
 
 
@@ -266,14 +266,9 @@ describe('Integration', () => {
          location.simulateHashChange('/one');
          advance(fixture);
 
-         const BASE_ERROR_MESSAGE =
-             'This asserts current behavior, which is incorrect. When #28561 is fixed, it should be: ';
-
-         expect(location.path()).toEqual('/one', BASE_ERROR_MESSAGE + '/');
-         const urlChanges = ['replace: /', 'hash: /one'];
-         expect(location.urlChanges)
-             .toEqual(
-                 urlChanges, BASE_ERROR_MESSAGE + JSON.stringify(urlChanges.concat('replace: /')));
+         expect(location.path()).toEqual('/');
+         const urlChanges = ['replace: /', 'hash: /one', 'replace: /'];
+         expect(location.urlChanges).toEqual(urlChanges);
        }));
   });
 });