Settings Router: Restrict navigateToPreviousRoute from going deeper.

chrome/test/data/webui/settings/route_tests.js

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 suite('route', function() {
   test('tree structure', function() {
     // Set up root page routes.
     var BASIC = new settings.Route('/');
     assertEquals(0, BASIC.depth);
 
@@ skipping 36 matching lines @@
     assertEquals('', CLEAR_BROWSER_DATA.section);
   });
 
   test('no duplicate routes', function() {
     var paths = new Set();
     Object.values(settings.Route).forEach(function(route) {
       assertFalse(paths.has(route.path), route.path);
       paths.add(route.path);
     });
   });
+
+  /**
+   * Tests a specific navigation situation.
+   * @param {!settings.Route} previousRoute
+   * @param {!settings.Route} currentRoute
+   * @param {!settings.Route} expectedNavigatePreviousResult
+   * @return {!Promise}
+   */
+  function testNavigateBack(previousRoute, currentRoute,

[michaelpg, 2016/08/24 20:25:08]

testNavigateBackUsesHistory

[tommycli, 2016/08/24 20:41:36]

Done.

+                            expectedNavigatePreviousResult) {
+    /**
+     * Returns a new promise that resolves after a window 'popstate' event.
+     * @return {!Promise}
+     */
+    function whenPopState() {
+      return new Promise(function(resolve) {
+        var callback = function() {
+          window.removeEventListener('popstate', callback);
+          resolve();
+        };
+        window.addEventListener('popstate', callback);

[michaelpg, 2016/08/24 20:25:08]

opt nit: I prefer this pattern without a variable:

window.addEventListener('popstate', function callback() {
  window.removeEventListener('popstate', callback);
  ...
});

[tommycli, 2016/08/24 20:41:36]

Done. Whoa... i didn't even know that was possible.

+      });
+    }
+
+    settings.navigateTo(previousRoute);
+    settings.navigateTo(currentRoute);
+    settings.navigateToPreviousRoute();
+
+    return whenPopState().then(function() {
+      assertEquals(expectedNavigatePreviousResult,
+                   settings.getCurrentRoute());
+    });
+  };
+
+  test('navigate back to parent previous route', function() {
+    return testNavigateBack(settings.Route.BASIC, settings.Route.PEOPLE,
+                            settings.Route.BASIC);
+  });
+
+  test('navigate back to non-ancestor shallower route', function() {
+    return testNavigateBack(settings.Route.ADVANCED, settings.Route.PEOPLE,
+                            settings.Route.ADVANCED);
+  });
+
+  test('navigate back to sibling route', function() {
+    return testNavigateBack(settings.Route.APPEARANCE, settings.Route.PEOPLE,
+                            settings.Route.APPEARANCE);
+  });
+
+  test('navigate back to parent when previous route is deeper', function() {
+    settings.navigateTo(settings.Route.SYNC);
+    settings.navigateTo(settings.Route.PEOPLE);
+    settings.navigateToPreviousRoute();
+    assertEquals(settings.Route.BASIC, settings.getCurrentRoute());
+  });
+
+  test('navigate back to BASIC when going back from root pages', function() {
+    settings.navigateTo(settings.Route.PEOPLE);
+    settings.navigateTo(settings.Route.ADVANCED);
+    settings.navigateToPreviousRoute();
+    assertEquals(settings.Route.BASIC, settings.getCurrentRoute());
+  });
 });