Apps on NTP should be in order of installation

chrome/browser/extensions/pending_extension_info.h

Index: chrome/browser/extensions/pending_extension_info.h
diff --git a/chrome/browser/extensions/pending_extension_info.h b/chrome/browser/extensions/pending_extension_info.h
index 7ab82d4dd8fa56e0ec9e1f2f287fbe394866b2d7..13d6d3f9f2be117f18409506a6a53670d7e35960 100644
--- a/chrome/browser/extensions/pending_extension_info.h
+++ b/chrome/browser/extensions/pending_extension_info.h
@@ -21,6 +21,7 @@ class PendingExtensionInfo {
   typedef bool (*ShouldAllowInstallPredicate)(const Extension&);
 
   PendingExtensionInfo(
+      const std::string& id,
       const GURL& update_url,
       ShouldAllowInstallPredicate should_allow_install,
       bool is_from_sync,
@@ -30,6 +31,7 @@ class PendingExtensionInfo {
   // Required for STL container membership.  Should not be used directly.
   PendingExtensionInfo();
 
+  const std::string& id() const { return id_; }
   const GURL& update_url() const { return update_url_; }
 
   // ShouldAllowInstall() returns the result of running constructor argument
@@ -46,6 +48,8 @@ class PendingExtensionInfo {
   Extension::Location install_source() const { return install_source_; }
 
  private:
+  const std::string& id_;

[Aaron Boodman, 2012/05/01 15:50:22]

No need for this to be const since it is private.

[mitchellwrosen, 2012/05/11 05:45:03]

Compiler barks about initializing a non-const std::string& from a const
std::string& in the constructor

[Aaron Boodman, 2012/05/11 19:23:55]

It should work if its non-reference. Just plain old std::string. In fact I'm not
really sure what it means to have a reference member.

+
   GURL update_url_;
 
   // When the extension is about to be installed, this function is