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

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

Issue 954943007: [Extensions] Make chrome://extensions use api functions for file access, reload (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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
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 /** 7 /**
8 * The type of the extension data object. The definition is based on 8 * The type of the extension data object. The definition is based on
9 * chrome/browser/ui/webui/extensions/extension_basic_info.cc 9 * chrome/browser/ui/webui/extensions/extension_basic_info.cc
10 * and 10 * and
(...skipping 388 matching lines...) Expand 10 before | Expand all | Expand 10 after
399 row.querySelector('.site-link').setAttribute('column-type', 'website'); 399 row.querySelector('.site-link').setAttribute('column-type', 'website');
400 400
401 // The 'Permissions' link. 401 // The 'Permissions' link.
402 row.setupColumn('.permissions-link', 'details', 'click', function(e) { 402 row.setupColumn('.permissions-link', 'details', 'click', function(e) {
403 chrome.send('extensionSettingsPermissions', [extension.id]); 403 chrome.send('extensionSettingsPermissions', [extension.id]);
404 e.preventDefault(); 404 e.preventDefault();
405 }); 405 });
406 406
407 // The 'Reload' link. 407 // The 'Reload' link.
408 row.setupColumn('.reload-link', 'localReload', 'click', function(e) { 408 row.setupColumn('.reload-link', 'localReload', 'click', function(e) {
409 chrome.send('extensionSettingsReload', [extension.id]); 409 chrome.developerPrivate.reload(extension.id, {failQuietly: true});
410 extensionReloadedTimestamp[extension.id] = Date.now(); 410 extensionReloadedTimestamp[extension.id] = Date.now();
411 }); 411 });
412 412
413 // The 'Launch' link. 413 // The 'Launch' link.
414 row.setupColumn('.launch-link', 'launch', 'click', function(e) { 414 row.setupColumn('.launch-link', 'launch', 'click', function(e) {
415 chrome.send('extensionSettingsLaunch', [extension.id]); 415 chrome.send('extensionSettingsLaunch', [extension.id]);
416 }); 416 });
417 417
418 // The 'Reload' terminated link. 418 // The 'Reload' terminated link.
419 row.setupColumn('.terminated-reload-link', 'terminatedReload', 'click', 419 row.setupColumn('.terminated-reload-link', 'terminatedReload', 'click',
(...skipping 459 matching lines...) Expand 10 before | Expand all | Expand 10 after
879 // TODO(dbeam): why do we need to focus <extensionoptions> before and 879 // TODO(dbeam): why do we need to focus <extensionoptions> before and
880 // after its showing animation? Makes very little sense to me. 880 // after its showing animation? Makes very little sense to me.
881 overlay.setInitialFocus(); 881 overlay.setInitialFocus();
882 }, 882 },
883 }; 883 };
884 884
885 return { 885 return {
886 ExtensionsList: ExtensionsList 886 ExtensionsList: ExtensionsList
887 }; 887 };
888 }); 888 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698