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

Unified Diff: chrome/browser/resources/task_manager/includes.js

Issue 8993007: WebUI TaskManager: Delay scripts loading. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review fix Created 9 years 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/task_manager/includes.js
diff --git a/chrome/browser/resources/task_manager/includes.js b/chrome/browser/resources/task_manager/includes.js
index 75708b12c5929a0de7e0b4302400072f983d8e16..94bc2dfb6f0da169187f046f8fd12988c971580a 100644
--- a/chrome/browser/resources/task_manager/includes.js
+++ b/chrome/browser/resources/task_manager/includes.js
@@ -39,7 +39,9 @@ var script = [
'cr/ui/table.js',
'cr/ui/grid.js',
+];
+var scriptDelayed = [
'cr/ui/command.js',
'cr/ui/position_util.js',
'cr/ui/menu_item.js',
@@ -47,6 +49,8 @@ var script = [
'cr/ui/context_menu_handler.js',
];
+var loadDelayedIncludes;
+
(function() {
// Switch to 'test harness' mode when loading from a file url.
var isHarness = document.location.protocol == 'file:';
@@ -63,4 +67,25 @@ var script = [
document.write('<script src="' + prefix + 'js/' + script[i] +
'"><\/script>');
}
+
+ /**
+ * Loads delayed scripts.
+ * This function is called by TaskManager::initalize() in main.js.
+ */
+ loadDelayedIncludes = function (taskmanager) {
mazda 2011/12/21 05:56:53 No space between 'function' and '('.
yoshiki 2011/12/21 06:08:48 Done.
+ // Number of remaining scripts to load.
+ var remain = scriptDelayed.length;
+
+ // Waits for initialization of task manager.
+ for (var i = 0; i < scriptDelayed.length; ++i) {
+ var s = document.createElement('script');
+ s.onload = function(e) {
+ if (!--remain)
+ taskmanager.delayedInitialize();
+ };
+ s.type = 'text/javascript';
+ s.src = prefix + 'js/' + scriptDelayed[i];
+ document.body.appendChild(s);
+ }
+ };
})();

Powered by Google App Engine
This is Rietveld 408576698