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

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

Issue 342003005: Show alert failure for reloading unpacked extensions with bad manifest (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Synchronized page loading, support multiple load errors Created 6 years, 5 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="../uber/uber_utils.js"></include> 5 <include src="../uber/uber_utils.js"></include>
6 <include src="extension_code.js"></include> 6 <include src="extension_code.js"></include>
7 <include src="extension_commands_overlay.js"></include> 7 <include src="extension_commands_overlay.js"></include>
8 <include src="extension_focus_manager.js"></include> 8 <include src="extension_focus_manager.js"></include>
9 <include src="extension_list.js"></include> 9 <include src="extension_list.js"></include>
10 <include src="pack_extension_overlay.js"></include> 10 <include src="pack_extension_overlay.js"></include>
11 <include src="extension_error_overlay.js"></include> 11 <include src="extension_error_overlay.js"></include>
12 <include src="extension_loader.js"></include> 12 <include src="extension_loader.js"></include>
13 13
14 <if expr="chromeos"> 14 <if expr="chromeos">
15 <include src="chromeos/kiosk_apps.js"></include> 15 <include src="chromeos/kiosk_apps.js"></include>
16 </if> 16 </if>
17 17
18 document.addEventListener('DOMContentLoaded', function() {
Devlin 2014/06/27 22:31:08 ExtensionLoader should have as few dependencies on
gpdavis 2014/06/27 23:42:54 I'll give that a try and see if it works the same.
gpdavis 2014/06/28 02:31:17 Looks good! Done.
19 chrome.send('extensionLoaderDisplayFailures');
Devlin 2014/06/27 22:31:09 nit: indentation.
gpdavis 2014/06/28 02:31:17 Done.
20 });
21
22 window.addEventListener('beforeunload', function() {
Devlin 2014/06/27 22:31:09 nit: indentation.
gpdavis 2014/06/28 02:31:17 Done.
23 chrome.send('extensionLoaderSetDisplayLoading');
24 });
25
18 // Used for observing function of the backend datasource for this page by 26 // Used for observing function of the backend datasource for this page by
19 // tests. 27 // tests.
20 var webuiResponded = false; 28 var webuiResponded = false;
21 29
22 cr.define('extensions', function() { 30 cr.define('extensions', function() {
23 var ExtensionsList = options.ExtensionsList; 31 var ExtensionsList = options.ExtensionsList;
24 32
25 // Implements the DragWrapper handler interface. 33 // Implements the DragWrapper handler interface.
26 var dragWrapperHandler = { 34 var dragWrapperHandler = {
27 /** @override */ 35 /** @override */
(...skipping 406 matching lines...) Expand 10 before | Expand all | Expand 10 after
434 442
435 // Export 443 // Export
436 return { 444 return {
437 ExtensionSettings: ExtensionSettings 445 ExtensionSettings: ExtensionSettings
438 }; 446 };
439 }); 447 });
440 448
441 window.addEventListener('load', function(e) { 449 window.addEventListener('load', function(e) {
442 extensions.ExtensionSettings.getInstance().initialize(); 450 extensions.ExtensionSettings.getInstance().initialize();
443 }); 451 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698