Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(91)

Side by Side Diff: chrome/browser/resources/extensions/extension_list.js

Issue 462273003: Show extension name for dependent extensions (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: feedback Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | chrome/browser/resources/extensions/extensions.html » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 <include src="extension_error.js"> 5 <include src="extension_error.js">
6 6
7 cr.define('options', function() { 7 cr.define('options', function() {
8 'use strict'; 8 'use strict';
9 9
10 /** 10 /**
(...skipping 303 matching lines...) Expand 10 before | Expand all | Expand 10 after
314 // Then the 'This is a corrupt extension' message. 314 // Then the 'This is a corrupt extension' message.
315 node.querySelector('.corrupt-install-message').hidden = false; 315 node.querySelector('.corrupt-install-message').hidden = false;
316 } 316 }
317 } 317 }
318 318
319 if (extension.dependentExtensions.length > 0) { 319 if (extension.dependentExtensions.length > 0) {
320 var dependentMessage = 320 var dependentMessage =
321 node.querySelector('.dependent-extensions-message'); 321 node.querySelector('.dependent-extensions-message');
322 dependentMessage.hidden = false; 322 dependentMessage.hidden = false;
323 var dependentList = dependentMessage.querySelector('ul'); 323 var dependentList = dependentMessage.querySelector('ul');
324 extension.dependentExtensions.forEach(function(id) { 324 var dependentTemplate = $('template-collection').querySelector(
325 var li = document.createElement('li'); 325 '.dependent-list-item');
326 li.innerText = id; 326 extension.dependentExtensions.forEach(function(elem) {
327 dependentList.appendChild(li); 327 var depNode = dependentTemplate.cloneNode(true);
328 depNode.querySelector('.dep-extension-title').textContent = elem.name;
329 depNode.querySelector('.dep-extension-id').textContent = elem.id;
330 dependentList.appendChild(depNode);
328 }); 331 });
329 } 332 }
330 333
331 // Then active views. 334 // Then active views.
332 if (extension.views.length > 0) { 335 if (extension.views.length > 0) {
333 var activeViews = node.querySelector('.active-views'); 336 var activeViews = node.querySelector('.active-views');
334 activeViews.hidden = false; 337 activeViews.hidden = false;
335 var link = activeViews.querySelector('a'); 338 var link = activeViews.querySelector('a');
336 339
337 extension.views.forEach(function(view, i) { 340 extension.views.forEach(function(view, i) {
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
406 topScroll -= pad / 2; 409 topScroll -= pad / 2;
407 setScrollTopForDocument(document, topScroll); 410 setScrollTopForDocument(document, topScroll);
408 } 411 }
409 }, 412 },
410 }; 413 };
411 414
412 return { 415 return {
413 ExtensionsList: ExtensionsList 416 ExtensionsList: ExtensionsList
414 }; 417 };
415 }); 418 });
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/resources/extensions/extensions.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698