Use PageManager in About page

chrome/browser/resources/options/options.js

Index: chrome/browser/resources/options/options.js
diff --git a/chrome/browser/resources/options/options.js b/chrome/browser/resources/options/options.js
index 011223f72c31267a6fac949e2ab30246b3c224aa..5c59c2d0b0f3fc003251aa1760eff536a42233c1 100644
--- a/chrome/browser/resources/options/options.js
+++ b/chrome/browser/resources/options/options.js
@@ -225,7 +225,9 @@ function load() {
   OptionsFocusManager.getInstance().initialize();
   Preferences.getInstance().initialize();
   AutomaticSettingsResetBanner.getInstance().initialize();
+  PageManager.addObserver(new uber.PageManagerObserver());
   OptionsPage.initialize();
+  uber.onContentFrameLoaded();

[Dan Beam, 2014/08/06 18:30:54]

why are these calls here and not at the bottom of the method (for example)?

[michaelpg, 2014/08/06 21:58:18]

i've moved these around a bit:

PageManager.initialize has to happen before PageManager.showPageByName, and we
want to add the observer before we start navigating to our given path.

uber.onContentFrameLoaded should actually happen later I think, after
showPageByName has completed, so it registers the correct scroll status.

 
   var pageName = PageManager.getPageNameFromPath();
   // Still update history so that chrome://settings/nonexistant redirects