[MD Extensions] Add a details subpage, move to one list

chrome/test/data/webui/extensions/extension_sidebar_test.js

Index: chrome/test/data/webui/extensions/extension_sidebar_test.js
diff --git a/chrome/test/data/webui/extensions/extension_sidebar_test.js b/chrome/test/data/webui/extensions/extension_sidebar_test.js
index 71b6726e0b0af84a3773df68735e2d8052482d49..dd596d3e084c3d35614bd90a069cc2dceeb4e47c 100644
--- a/chrome/test/data/webui/extensions/extension_sidebar_test.js
+++ b/chrome/test/data/webui/extensions/extension_sidebar_test.js
@@ -29,13 +29,10 @@ cr.define('extension_sidebar_tests', function() {
     updateAllExtensions: function() {},
 
     /** @override */
-    scrollToExtensions: function() {},
+    showExtensions: function() {},

[Dan Beam, 2016/04/26 01:32:13]

it's not currently clear from this API that apps and extensions are mutually
exclusive

maybe make an enum mode for the page to be so it's clearer that things are
mutually exclusive, i.e.

/** @enum {number} */
var ShowingType = {
  EXTENSIONS: 1,
  APPS: 2,
};

[Devlin, 2016/04/26 17:24:18]

Done.  For now, I've put ShowingType in sidebar.js, but lemme know if you want
it somewhere else.

 
     /** @override */
-    scrollToApps: function() {},
-
-    /** @override */
-    scrollToWebsites: function() {},
+    showApps: function() {},
   };
 
   /** @enum {string} */
@@ -61,7 +58,7 @@ cr.define('extension_sidebar_tests', function() {
         sidebar = document.querySelector('extensions-manager').sidebar;
         mockDelegate = new MockDelegate();
         sidebar.setDelegate(mockDelegate);
-        sidebar.setScrollDelegate(mockDelegate);
+        sidebar.setListDelegate(mockDelegate);
       });
 
       test(assert(TestNames.Layout), function() {
@@ -95,11 +92,9 @@ cr.define('extension_sidebar_tests', function() {
         mockDelegate.testClickingCalls(
             sidebar.$$('#update-now'), 'updateAllExtensions', []);
         mockDelegate.testClickingCalls(
-            sidebar.$$('#sections-extensions'), 'scrollToExtensions', []);
-        mockDelegate.testClickingCalls(
-            sidebar.$$('#sections-apps'), 'scrollToApps', []);
+            sidebar.$$('#sections-extensions'), 'showExtensions', []);
         mockDelegate.testClickingCalls(
-            sidebar.$$('#sections-websites'), 'scrollToWebsites', []);
+            sidebar.$$('#sections-apps'), 'showApps', []);
       });
     });
   }