Apps on NTP should be in order of installation

chrome/browser/extensions/updater/extension_updater.h

Index: chrome/browser/extensions/updater/extension_updater.h
diff --git a/chrome/browser/extensions/updater/extension_updater.h b/chrome/browser/extensions/updater/extension_updater.h
index 06a4ae6e82dca55bbcec2f24515b4c5db4e04ade..45fc0edff2c9877ba613919053f090440d8c98ce 100644
--- a/chrome/browser/extensions/updater/extension_updater.h
+++ b/chrome/browser/extensions/updater/extension_updater.h
@@ -6,6 +6,7 @@
 #define CHROME_BROWSER_EXTENSIONS_UPDATER_EXTENSION_UPDATER_H_
 #pragma once
 
+#include <list>
 #include <set>

[Aaron Boodman, 2012/04/09 17:02:14]

Is the set include still necessary?

[mitchellwrosen, 2012/04/13 22:20:18]

Nope!

 #include <stack>
 #include <string>
@@ -113,7 +114,7 @@ class ExtensionUpdater : public ExtensionDownloaderDelegate,
   // Add fetch records for extensions that are installed to the downloader,
   // ignoring |pending_ids| so the extension isn't fetched again.
   void AddToDownloader(const ExtensionSet* extensions,
-                       const std::set<std::string>& pending_ids);
+                       const std::list<std::string>& pending_ids);
 
   // BaseTimer::ReceiverMethod callback.
   void TimerFired();
@@ -184,7 +185,7 @@ class ExtensionUpdater : public ExtensionDownloaderDelegate,
   bool blacklist_checks_enabled_;
 
   // The ids of extensions that have in-progress update checks.
-  std::set<std::string> in_progress_ids_;
+  std::list<std::string> in_progress_ids_;
 
   // Observes CRX installs we initiate.
   content::NotificationRegistrar registrar_;