Chromium Code Reviews

Issue 3611014: Fix 57090: NTP apps sorted in random-looking order.... (Closed)

Created:
10 years, 2 months ago by Finnur
Modified:
9 years, 6 months ago
Reviewers:
arv (Not doing code reviews)
CC:
chromium-reviews, ben+cc_chromium.org, Erik does not do reviews, Aaron Boodman, arv (Not doing code reviews), pam+watch_chromium.org, Paweł Hajdan Jr.
Visibility:
Public.

Description

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 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=62112

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Stats (+94 lines, -0 lines)
M chrome/browser/dom_ui/app_launcher_handler.cc View 1 chunk +8 lines, -0 lines 0 comments
M chrome/browser/extensions/extension_prefs.h View 1 chunk +13 lines, -0 lines 0 comments
M chrome/browser/extensions/extension_prefs.cc View 3 chunks +36 lines, -0 lines 0 comments
M chrome/browser/extensions/extension_prefs_unittest.cc View 1 chunk +33 lines, -0 lines 0 comments
M chrome/browser/resources/ntp/apps.js View 1 chunk +4 lines, -0 lines 0 comments

Messages

Total messages: 2 (0 generated)
Finnur
10 years, 2 months ago (2010-10-08 15:18:00 UTC) #1
arv (Not doing code reviews)
10 years, 2 months ago (2010-10-08 16:50:23 UTC) #2 
LGTM

Powered by Google App Engine