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

Side by Side Diff: chrome/browser/ui/app_list/arc/arc_package_syncable_service.cc

Issue 2230203002: chrome: Use stl utilities from the base namespace (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed accidental components/ change Created 4 years, 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/app_list/arc/arc_package_syncable_service.h" 5 #include "chrome/browser/ui/app_list/arc/arc_package_syncable_service.h"
6 6
7 #include <unordered_set> 7 #include <unordered_set>
8 8
9 #include "chrome/browser/profiles/profile.h" 9 #include "chrome/browser/profiles/profile.h"
10 #include "chrome/browser/ui/app_list/arc/arc_app_list_prefs.h" 10 #include "chrome/browser/ui/app_list/arc/arc_app_list_prefs.h"
(...skipping 116 matching lines...) Expand 10 before | Expand all | Expand 10 after
127 const std::vector<std::string> local_packages = 127 const std::vector<std::string> local_packages =
128 prefs_->GetPackagesFromPrefs(); 128 prefs_->GetPackagesFromPrefs();
129 const std::unordered_set<std::string> local_package_set( 129 const std::unordered_set<std::string> local_package_set(
130 local_packages.begin(), local_packages.end()); 130 local_packages.begin(), local_packages.end());
131 131
132 // Creates sync items from synced data. 132 // Creates sync items from synced data.
133 for (const syncer::SyncData& sync_data : initial_sync_data) { 133 for (const syncer::SyncData& sync_data : initial_sync_data) {
134 std::unique_ptr<ArcSyncItem> sync_item( 134 std::unique_ptr<ArcSyncItem> sync_item(
135 CreateSyncItemFromSyncData(sync_data)); 135 CreateSyncItemFromSyncData(sync_data));
136 const std::string& package_name = sync_item->package_name; 136 const std::string& package_name = sync_item->package_name;
137 if (!ContainsKey(local_package_set, package_name)) { 137 if (!base::ContainsKey(local_package_set, package_name)) {
138 pending_install_items_[package_name] = std::move(sync_item); 138 pending_install_items_[package_name] = std::move(sync_item);
139 InstallPackage(pending_install_items_[package_name].get()); 139 InstallPackage(pending_install_items_[package_name].get());
140 } else { 140 } else {
141 // TODO(lgcheng@) may need to handle update exsiting package here. 141 // TODO(lgcheng@) may need to handle update exsiting package here.
142 sync_items_[package_name] = std::move(sync_item); 142 sync_items_[package_name] = std::move(sync_item);
143 } 143 }
144 } 144 }
145 145
146 // Creates sync items for local unsynced packages. 146 // Creates sync items for local unsynced packages.
147 syncer::SyncChangeList change_list; 147 syncer::SyncChangeList change_list;
148 for (const auto& local_package_name : local_packages) { 148 for (const auto& local_package_name : local_packages) {
149 if (ContainsKey(sync_items_, local_package_name)) 149 if (base::ContainsKey(sync_items_, local_package_name))
150 continue; 150 continue;
151 151
152 if (!ShouldSyncPackage(local_package_name)) 152 if (!ShouldSyncPackage(local_package_name))
153 continue; 153 continue;
154 154
155 std::unique_ptr<ArcSyncItem> sync_item( 155 std::unique_ptr<ArcSyncItem> sync_item(
156 CreateSyncItemFromPrefs(prefs_->GetPackage(local_package_name))); 156 CreateSyncItemFromPrefs(prefs_->GetPackage(local_package_name)));
157 change_list.push_back(SyncChange(FROM_HERE, SyncChange::ACTION_ADD, 157 change_list.push_back(SyncChange(FROM_HERE, SyncChange::ACTION_ADD,
158 GetSyncDataFromSyncItem(sync_item.get()))); 158 GetSyncDataFromSyncItem(sync_item.get())));
159 sync_items_[local_package_name] = std::move(sync_item); 159 sync_items_[local_package_name] = std::move(sync_item);
(...skipping 255 matching lines...) Expand 10 before | Expand all | Expand 10 after
415 415
416 bool ArcPackageSyncableService::ShouldSyncPackage( 416 bool ArcPackageSyncableService::ShouldSyncPackage(
417 const std::string& package_name) const { 417 const std::string& package_name) const {
418 std::unique_ptr<ArcAppListPrefs::PackageInfo> package( 418 std::unique_ptr<ArcAppListPrefs::PackageInfo> package(
419 prefs_->GetPackage(package_name)); 419 prefs_->GetPackage(package_name));
420 DCHECK(package.get()); 420 DCHECK(package.get());
421 return package->should_sync; 421 return package->should_sync;
422 } 422 }
423 423
424 } // namespace arc 424 } // namespace arc
OLDNEW
« no previous file with comments | « chrome/browser/ui/app_list/app_list_syncable_service.cc ('k') | chrome/browser/ui/ash/launcher/browser_status_monitor.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698