ntp4: default to 18 apps per page for new installs

chrome/browser/extensions/extension_prefs.h

Index: chrome/browser/extensions/extension_prefs.h
diff --git a/chrome/browser/extensions/extension_prefs.h b/chrome/browser/extensions/extension_prefs.h
index cd644633d8fdf5b2840ffcd916d279ad631c4ec4..e17f73933c645cd479c640cc9baecccc31e911d2 100644
--- a/chrome/browser/extensions/extension_prefs.h
+++ b/chrome/browser/extensions/extension_prefs.h
@@ -101,6 +101,7 @@ class ExtensionPrefs : public ExtensionContentSettingsStore::Observer {
   void SetToolbarOrder(const std::vector<std::string>& extension_ids);
 
   // Called when an extension is installed, so that prefs get created.
+  // If |page_index| is -1, and the then a page will be found for the App.

[Mihai Parparita -not on Chrome, 2011/08/25 18:27:08]

Nit: "and the then" doesn't sound quite right and "App" probably shouldn't be
capitalized.

[Evan Stade, 2011/08/25 18:50:49]

you're right on both counts.

   void OnExtensionInstalled(const Extension* extension,
                             Extension::State initial_state,
                             bool from_webstore,
@@ -277,6 +278,10 @@ class ExtensionPrefs : public ExtensionContentSettingsStore::Observer {
   // highest current application launch index found for the page |on_page|.
   int GetNextAppLaunchIndex(int on_page);
 
+  // Gets the page a new app should install to. Starts on page 0, and if there
+  // are N or more apps on it, tries to install on the next page.
+  int GetNaturalAppPageIndex();
+
   // Sets the order the apps should be displayed in the app launcher.
   void SetAppLauncherOrder(const std::vector<std::string>& extension_ids);