Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(277)

Unified Diff: chrome/browser/sync/test/integration/sync_app_helper.cc

Issue 9595001: Apps on NTP should be in order of installation (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: More fixes Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/sync/test/integration/sync_app_helper.cc
diff --git a/chrome/browser/sync/test/integration/sync_app_helper.cc b/chrome/browser/sync/test/integration/sync_app_helper.cc
index efa61c74dff62b10434b0ce2c80194b7039b1f02..95868109682f18fc3796453f63d2ed1587025318 100644
--- a/chrome/browser/sync/test/integration/sync_app_helper.cc
+++ b/chrome/browser/sync/test/integration/sync_app_helper.cc
@@ -69,10 +69,10 @@ AppStateMap GetAppStates(Profile* profile) {
const PendingExtensionManager* pending_extension_manager =
extension_service->pending_extension_manager();
- std::set<std::string> pending_crx_ids;
+ std::list<std::string> pending_crx_ids;
pending_extension_manager->GetPendingIdsForUpdateCheck(&pending_crx_ids);
- for (std::set<std::string>::const_iterator id = pending_crx_ids.begin();
+ for (std::list<std::string>::const_iterator id = pending_crx_ids.begin();
id != pending_crx_ids.end(); ++id) {
LoadApp(extension_service, *id, &(app_state_map[*id]));
}

Powered by Google App Engine
This is Rietveld 408576698