Skip to content
Browse files

fix(router): fix a problem with router not responding to back button (#…


There was a problem with a combination of the `eager` URL update, browser `back` button, and hybrid applications. Details provided in internal ticket http://b/123667227.

This fix handles the problem by setting `router.browserUrlTree` when all conditions have failed, meaning the browser doesn't do anything with the navigation other than update internal data structures. Without this change, the problem was an old value was stored in `router.broserUrlTree` causing some new navigations to be compared to an old value and breaking future navigations.

PR Close #30160
  • Loading branch information...
jasonaden authored and kara committed Apr 26, 2019
1 parent ea9a381 commit 3327bd8eab0cf5bfb9a6a21d6ee4a8e081aab630
Showing with 28 additions and 0 deletions.
  1. +1 −0 packages/router/src/router.ts
  2. +27 −0 packages/router/test/integration.spec.ts
@@ -555,6 +555,7 @@ export class Router {
* way the next navigation will be coming from the current URL in the browser.
this.rawUrlTree = t.rawUrl;
this.browserUrlTree = t.urlAfterRedirects;
return EMPTY;
@@ -661,6 +661,33 @@ describe('Integration', () => {

it('should set browserUrlTree with urlUpdateStrategy="eagar" and false `shouldProcessUrl`',
fakeAsync(inject([Router, Location], (router: Router, location: Location) => {
const fixture = TestBed.createComponent(RootCmp);

router.urlUpdateStrategy = 'eager';

{path: 'team/:id', component: SimpleCmp},
{path: 'login', component: AbsoluteSimpleLinkCmp}

advance(fixture, 1);

expect((router as any).browserUrlTree.toString()).toBe('/team/22');

// Force to not process URL changes
router.urlHandlingStrategy.shouldProcessUrl = (url: UrlTree) => false;

advance(fixture, 1);

// Because we now can't process any URL, we will end up back at the root.
expect((router as any).browserUrlTree.toString()).toBe('/');

it('should eagerly update URL after redirects are applied with urlUpdateStrategy="eagar"',
fakeAsync(inject([Router, Location], (router: Router, location: Location) => {
const fixture = TestBed.createComponent(RootCmp);

0 comments on commit 3327bd8

Please sign in to comment.
You can’t perform that action at this time.