diff options
| author | finnur@chromium.org <finnur@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-10 09:17:08 +0000 |
|---|---|---|
| committer | finnur@chromium.org <finnur@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-10 09:17:08 +0000 |
| commit | ebfd4b915548df519779cae0853fe34f31af36da (patch) | |
| tree | 2562b4516607c859c36209bb56239dc0ced93b16 /chrome/browser/dom_ui | |
| parent | d57d13a6fbea0fc868f8a334857e228989cfbfdf (diff) | |
| download | chromium_src-ebfd4b915548df519779cae0853fe34f31af36da.zip chromium_src-ebfd4b915548df519779cae0853fe34f31af36da.tar.gz chromium_src-ebfd4b915548df519779cae0853fe34f31af36da.tar.bz2 | |
Fix 57090: NTP apps sorted in random-looking order.
Add the concept of app_launch_index, stored with the extension that determines an install order which is used to order the apps on the new tab page.
BUG=57090
TEST=None
Review URL: http://codereview.chromium.org/3611014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62112 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/dom_ui')
| -rw-r--r-- | chrome/browser/dom_ui/app_launcher_handler.cc | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/chrome/browser/dom_ui/app_launcher_handler.cc b/chrome/browser/dom_ui/app_launcher_handler.cc index e834754..674d951 100644 --- a/chrome/browser/dom_ui/app_launcher_handler.cc +++ b/chrome/browser/dom_ui/app_launcher_handler.cc @@ -121,6 +121,14 @@ void AppLauncherHandler::CreateAppInfo(Extension* extension, value->SetInteger("launch_container", extension->launch_container()); value->SetInteger("launch_type", extension_prefs->GetLaunchType(extension->id())); + + int app_launch_index = extension_prefs->GetAppLaunchIndex(extension->id()); + if (app_launch_index == -1) { + // Make sure every app has a launch index (some predate the launch index). + app_launch_index = extension_prefs->GetNextAppLaunchIndex(); + extension_prefs->SetAppLaunchIndex(extension->id(), app_launch_index); + } + value->SetInteger("app_launch_index", app_launch_index); } void AppLauncherHandler::FillAppDictionary(DictionaryValue* dictionary) { |