Settings Router Refactor: Improve chrome://md-settings/foobar behavior.

chrome/browser/resources/settings/route.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.
 
 cr.define('settings', function() {
   /**
    * Class for navigable routes. May only be instantiated within this file.
    * @constructor
    * @param {string} path
    * @private
@@ skipping 279 matching lines @@
 
     return Route[matchingKey] || null;
   };
 
   /**
    * The current active route. This may only be updated via the global
    * function settings.navigateTo.
    * @private {!settings.Route}
    */
   var currentRoute_ = getRouteForPath(window.location.pathname) || Route.BASIC;
+  window.history.replaceState(undefined, '', currentRoute_.path);

[Dan Beam, 2016/08/01 23:39:19]

can we do this only if currentRoute_ is null?

[tommycli, 2016/08/02 00:00:39]

Done.

 
   /**
    * Helper function to set the current route and notify all observers.
    * @param {!settings.Route} route
    */
   var setCurrentRoute = function(route) {
     currentRoute_ = route;
     for (var observer of routeObservers_)
       observer.currentRouteChanged();
   };
 
   /** @return {!settings.Route} */
   var getCurrentRoute = function() { return currentRoute_; };
 
   /**
    * Navigates to a canonical route and pushes a new history entry.
    * @param {!settings.Route} route
    * @private
    */
   var navigateTo = function(route) {
     if (assert(route) == currentRoute_)
       return;
 
-    window.history.pushState(undefined, document.title, route.path);
+    window.history.pushState(undefined, '', route.path);

[tommycli, 2016/08/01 23:38:18]

It's unused (by both Firefox and Chrome), and MDN recommends setting it an empty
string to be robust to future changes.

[Dan Beam, 2016/08/01 23:39:19]

Acknowledged.

     setCurrentRoute(route);
   };
 
   window.addEventListener('popstate', function(event) {
     // On pop state, do not push the state onto the window.history again.
     setCurrentRoute(getRouteForPath(window.location.pathname) || Route.BASIC);
   });
 
   return {
     Route: Route,
     RouteObserverBehavior: RouteObserverBehavior,
     getRouteForPath: getRouteForPath,
     getCurrentRoute: getCurrentRoute,
     navigateTo: navigateTo,
   };
 });