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

Unified Diff: chrome/browser/ui/app_list/app_list_syncable_service.h

Issue 2318303002: Remove stl_util's STLDeleteContainerPairSecondPointers. (Closed)
Patch Set: fix Created 4 years, 3 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/ui/app_list/app_list_syncable_service.h
diff --git a/chrome/browser/ui/app_list/app_list_syncable_service.h b/chrome/browser/ui/app_list/app_list_syncable_service.h
index 6359b30f767ad4dbcdf550bd3074161db849b413..6ba6fad7957639d675a3fa1fd9628f30bb6a380b 100644
--- a/chrome/browser/ui/app_list/app_list_syncable_service.h
+++ b/chrome/browser/ui/app_list/app_list_syncable_service.h
@@ -74,7 +74,7 @@ class AppListSyncableService : public syncer::SyncableService,
virtual ~Observer() = default;
};
- using SyncItemMap = std::map<std::string, SyncItem*>;
+ using SyncItemMap = std::map<std::string, std::unique_ptr<SyncItem>>;
// Populates the model when |extension_system| is ready.
AppListSyncableService(Profile* profile,
@@ -173,7 +173,7 @@ class AppListSyncableService : public syncer::SyncableService,
bool RemoveDefaultApp(AppListItem* item, SyncItem* sync_item);
// Deletes a sync item from |sync_items_| and sends a DELETE action.
- void DeleteSyncItem(SyncItem* sync_item);
+ void DeleteSyncItem(const std::string& item_id);
// Updates existing entry in |sync_items_| from |app_item|.
void UpdateSyncItem(AppListItem* app_item);
« no previous file with comments | « chrome/browser/signin/easy_unlock_service_signin_chromeos.cc ('k') | chrome/browser/ui/app_list/app_list_syncable_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698