Show extension name for dependent extensions

chrome/browser/resources/extensions/extension_list.js

 // Copyright (c) 2012 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.
 
 <include src="extension_error.js">
 
 cr.define('options', function() {
   'use strict';
 
   /**
@@ skipping 303 matching lines @@
           // Then the 'This is a corrupt extension' message.
           node.querySelector('.corrupt-install-message').hidden = false;
         }
       }
 
       if (extension.dependentExtensions.length > 0) {
         var dependentMessage =
             node.querySelector('.dependent-extensions-message');
         dependentMessage.hidden = false;
         var dependentList = dependentMessage.querySelector('ul');
-        extension.dependentExtensions.forEach(function(id) {
-          var li = document.createElement('li');
-          li.innerText = id;
-          dependentList.appendChild(li);
+        extension.dependentExtensions.forEach(function(elem) {
+          var dependentTemplate = $('template-dependent').querySelector(
+              '.dependent-list-item');

[not at google - send to devlin, 2014/08/13 19:43:47]

Save this outside the forEach loop?

[elijahtaylor1, 2014/08/13 21:12:45]

Done.

+          var dependentNode = dependentTemplate.cloneNode(true);
+          var dependentTitle = dependentNode.querySelector('.extension-title');
+          dependentTitle.textContent = elem.name;

[not at google - send to devlin, 2014/08/13 19:43:47]

Nit: to save on the saturation of "dependent*" variables here could you do that
inline? node.querySelector('.extension-title').textContent = elem.name; etc.

[elijahtaylor1, 2014/08/13 21:12:45]

Done.

+          var dependentId = dependentNode.querySelector('.extension-id');
+          dependentId.textContent = elem.id;
+          dependentList.appendChild(dependentNode);
         });
       }
 
       // Then active views.
       if (extension.views.length > 0) {
         var activeViews = node.querySelector('.active-views');
         activeViews.hidden = false;
         var link = activeViews.querySelector('a');
 
         extension.views.forEach(function(view, i) {
@@ skipping 68 matching lines @@
           topScroll -= pad / 2;
         setScrollTopForDocument(document, topScroll);
       }
     },
   };
 
   return {
     ExtensionsList: ExtensionsList
   };
 });