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

Unified Diff: chrome/browser/resources/file_manager/js/main.js

Issue 8565009: Improving file manager load time (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge Created 9 years, 1 month 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/file_manager/js/main.js
diff --git a/chrome/browser/resources/file_manager/js/main.js b/chrome/browser/resources/file_manager/js/main.js
index 3e0f1bfb8f6e864c72dc72323b5e88d44c961add..73bedb9bb201552ef737cda76a4ae6b76435ed3b 100644
--- a/chrome/browser/resources/file_manager/js/main.js
+++ b/chrome/browser/resources/file_manager/js/main.js
@@ -13,54 +13,13 @@ var fileManager;
* Called by main.html after the dom has been parsed.
*/
function init() {
- var rootPaths = ['Downloads', 'removable', 'archive', 'tmp'];
-
- function onEntriesFound(filesystem, entries) {
- metrics.recordTime('EnumerateRoots');
- FileManager.initStrings(function () {
- metrics.startInterval('Construct');
- fileManager = new FileManager(document.body, filesystem, entries);
- metrics.recordTime('Construct');
- metrics.recordTime('TotalLoad');
- // We're ready to run. Tests can monitor for this state with
- // ExtensionTestMessageListener listener("ready");
- // ASSERT_TRUE(listener.WaitUntilSatisfied());
- chrome.test.sendMessage('ready');
- });
- }
-
- function onFileSystemFound(filesystem) {
- metrics.recordTime('RequestLocalFileSystem');
- console.log('Found filesystem: ' + filesystem.name, filesystem);
-
- var entries = [];
-
- function onPathError(path, err) {
- console.error('Error locating root path: ' + path + ': ' + err);
- }
-
- function onEntryFound(entry) {
- if (entry) {
- entries.push(entry);
- } else {
- onEntriesFound(filesystem, entries);
- }
- }
-
- metrics.startInterval('EnumerateRoots');
- if (filesystem.name.match(/^chrome-extension_\S+:external/i)) {
- // We've been handed the local filesystem, whose root directory
- // cannot be enumerated.
- util.getDirectories(filesystem.root, {create: false}, rootPaths,
- onEntryFound, onPathError);
- } else {
- util.forEachDirEntry(filesystem.root, onEntryFound);
- }
- };
-
- util.installFileErrorToString();
-
- console.log('Requesting filesystem.');
- metrics.startInterval('RequestLocalFileSystem');
- chrome.fileBrowserPrivate.requestLocalFileSystem(onFileSystemFound);
+ FileManager.initStrings(function () {
+ metrics.startInterval('Construct');
+ fileManager = new FileManager(document.body);
+ metrics.recordTime('Construct');
+ // We're ready to run. Tests can monitor for this state with
+ // ExtensionTestMessageListener listener("ready");
+ // ASSERT_TRUE(listener.WaitUntilSatisfied());
+ chrome.test.sendMessage('ready');
+ });
}
« no previous file with comments | « chrome/browser/resources/file_manager/js/file_manager.js ('k') | chrome/browser/resources/file_manager/main.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698