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

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

Issue 1049483003: [Extensions] Update extensions UI to observe events and add test (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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="../../../../ui/webui/resources/js/cr/ui/focus_row.js"> 5 <include src="../../../../ui/webui/resources/js/cr/ui/focus_row.js">
6 <include src="../../../../ui/webui/resources/js/cr/ui/focus_grid.js"> 6 <include src="../../../../ui/webui/resources/js/cr/ui/focus_grid.js">
7 <include src="../uber/uber_utils.js"> 7 <include src="../uber/uber_utils.js">
8 <include src="extension_code.js"> 8 <include src="extension_code.js">
9 <include src="extension_commands_overlay.js"> 9 <include src="extension_commands_overlay.js">
10 <include src="extension_error_overlay.js"> 10 <include src="extension_error_overlay.js">
(...skipping 301 matching lines...) Expand 10 before | Expand all | Expand 10 after
312 pageDiv.classList.toggle('showing-banner', supervised); 312 pageDiv.classList.toggle('showing-banner', supervised);
313 313
314 var devControlsCheckbox = $('toggle-dev-on'); 314 var devControlsCheckbox = $('toggle-dev-on');
315 devControlsCheckbox.checked = extensionsData.developerMode; 315 devControlsCheckbox.checked = extensionsData.developerMode;
316 devControlsCheckbox.disabled = supervised; 316 devControlsCheckbox.disabled = supervised;
317 317
318 var instance = ExtensionSettings.getInstance(); 318 var instance = ExtensionSettings.getInstance();
319 instance.displayPromo_ = extensionsData.promoteAppsDevTools; 319 instance.displayPromo_ = extensionsData.promoteAppsDevTools;
320 instance.updateDevControlsVisibility_(false); 320 instance.updateDevControlsVisibility_(false);
321 321
322 document.documentElement.classList.add('loading');
322 $('load-unpacked').disabled = extensionsData.loadUnpackedDisabled; 323 $('load-unpacked').disabled = extensionsData.loadUnpackedDisabled;
323 var extensionList = $('extension-settings-list'); 324 var extensionList = $('extension-settings-list');
324 extensionList.updateExtensionsData( 325 extensionList.updateExtensionsData(
325 extensionsData.incognitoAvailable, 326 extensionsData.incognitoAvailable,
326 extensionsData.enableAppInfoDialog).then(function() { 327 extensionsData.enableAppInfoDialog).then(function() {
327 // We can get called many times in short order, thus we need to 328 // We can get called many times in short order, thus we need to
328 // be careful to remove the 'finished loading' timeout. 329 // be careful to remove the 'finished loading' timeout.
329 if (this.loadingTimeout_) 330 if (this.loadingTimeout_)
330 window.clearTimeout(this.loadingTimeout_); 331 window.clearTimeout(this.loadingTimeout_);
331 document.documentElement.classList.add('loading');
332 this.loadingTimeout_ = window.setTimeout(function() { 332 this.loadingTimeout_ = window.setTimeout(function() {
333 document.documentElement.classList.remove('loading'); 333 document.documentElement.classList.remove('loading');
334 }, 0); 334 }, 0);
335 335
336 var hasExtensions = extensionList.getNumExtensions() != 0; 336 var hasExtensions = extensionList.getNumExtensions() != 0;
337 $('no-extensions').hidden = hasExtensions; 337 $('no-extensions').hidden = hasExtensions;
338 $('extension-list-wrapper').hidden = !hasExtensions; 338 $('extension-list-wrapper').hidden = !hasExtensions;
339 }.bind(this)); 339 }.bind(this));
340 }; 340 };
341 341
(...skipping 95 matching lines...) Expand 10 before | Expand all | Expand 10 after
437 437
438 // Export 438 // Export
439 return { 439 return {
440 ExtensionSettings: ExtensionSettings 440 ExtensionSettings: ExtensionSettings
441 }; 441 };
442 }); 442 });
443 443
444 window.addEventListener('load', function(e) { 444 window.addEventListener('load', function(e) {
445 extensions.ExtensionSettings.getInstance().initialize(); 445 extensions.ExtensionSettings.getInstance().initialize();
446 }); 446 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698