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

chrome/browser/resources/md_extensions/service.js

 // Copyright 2015 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('extensions', function() {
   'use strict';
 
   /**
    * @constructor
    * @implements {extensions.ItemDelegate}
@@ skipping 93 matching lines @@
         this.promptIsShowing_ = false;
       }.bind(this));
     },
 
     /** @override */
     setItemEnabled: function(id, isEnabled) {
       chrome.management.setEnabled(id, isEnabled);
     },
 
     /** @override */
-    showItemDetails: function(id) {},
+    showItemDetails: function(id) {
+      var data =
+          this.extensions_.filter(function(e) { return e.id === id; })[0];
+      assert(data);

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

nit: i find this more readable (also, assert() returns what's passed to it)

function matchesId(extension) { return extension.id == id; }
var data = assert(this.extensions_.filter(matchesId)[0]);
this.manager_.showDetailsFor(data);

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

Done.

+      this.manager_.showDetailsFor(data);
+    },
 
     /** @override */
     setItemAllowedIncognito: function(id, isAllowedIncognito) {
       chrome.developerPrivate.updateExtensionConfiguration({
         extensionId: id,
         incognitoAccess: isAllowedIncognito,
       });
     },
 
     /** @override */
@@ skipping 24 matching lines @@
     /** @override */
     updateAllExtensions: function() {
       chrome.developerPrivate.autoUpdate();
     },
   };
 
   cr.addSingletonGetter(Service);
 
   return {Service: Service};
 });