Settings Router Refactor: Implement new Route class w/ test

chrome/test/data/webui/settings/router_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('routing', function() {
   test('no duplicate routes', function() {
     var urls = new Set();
     document.createElement('settings-router').canonicalRoutes_.forEach(
         function(route) {
           assertFalse(urls.has(route.url), route.url);
           urls.add(route.url);
         });
   });
+
+  test('new route tree structure', function() {

[michaelpg, 2016/07/13 05:53:40]

these should be run separately, from a test that loads route.html, IMO

[tommycli, 2016/07/13 17:45:03]

Done.

+    var BASIC = new settings.Route('/');
+    BASIC.page = 'basic';
+
+    var ADVANCED = new settings.Route('/advanced');
+    ADVANCED.page = 'advanced';
+
+    var PRIVACY = ADVANCED.createChild('/privacy');
+    PRIVACY.section = 'privacy';
+    assertEquals('advanced', PRIVACY.page);
+    assertFalse(PRIVACY.isDescendantOf(BASIC));
+    assertTrue(PRIVACY.isDescendantOf(ADVANCED));
+    assertFalse(PRIVACY.isDescendantOf(PRIVACY));
+

[michaelpg, 2016/07/13 05:53:39]

assertFalse(ADVANCED.isDescendantOf(PRIVACY))?

[tommycli, 2016/07/13 17:45:03]

Done.

+    var SITE_SETTINGS = PRIVACY.createChild('/siteSettings');

[michaelpg, 2016/07/13 05:53:40]

we should test opt_subpageName. incidentally, siteSettings is a subpage; did you
intentionally leave that out here?

[tommycli, 2016/07/13 17:45:03]

Done. No it was just an oversight. In fact, testing this caught a bug in the
subpage array implementation.

+    assertFalse(SITE_SETTINGS.dialog);
+    assertEquals('advanced', SITE_SETTINGS.page);
+    assertEquals('privacy', SITE_SETTINGS.section);
+    assertFalse(SITE_SETTINGS.isDescendantOf(BASIC));
+    assertTrue(SITE_SETTINGS.isDescendantOf(ADVANCED));
+    assertTrue(SITE_SETTINGS.isDescendantOf(PRIVACY));
+
+    var CLEAR_BROWSING_DATA = PRIVACY.createDialog('/clearBrowsingData');
+    assertTrue(CLEAR_BROWSING_DATA.dialog);
+    assertEquals('advanced', CLEAR_BROWSING_DATA.page);
+    assertEquals('privacy', CLEAR_BROWSING_DATA.section);

[michaelpg, 2016/07/13 05:53:40]

may as well test isDescendantOf for dialogs too

[tommycli, 2016/07/13 17:45:04]

Done.

+  });
 });