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

Unified Diff: components/update_client/update_checker.cc

Issue 2110663002: components: Change auto to not deduce raw pointers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase+one fix Created 4 years, 5 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
« no previous file with comments | « components/update_client/request_sender.cc ('k') | components/update_client/update_client.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/update_client/update_checker.cc
diff --git a/components/update_client/update_checker.cc b/components/update_client/update_checker.cc
index 6080b13a6c687e7df75cca9d83c46c1894bf3048..612024bca8d0ab7875afc9b1bcd9f0bfe581b5e9 100644
--- a/components/update_client/update_checker.cc
+++ b/components/update_client/update_checker.cc
@@ -48,7 +48,7 @@ update_client::InstallerAttributes SanitizeInstallerAttributes(
// Returns true if at least one item requires network encryption.
bool IsEncryptionRequired(const std::vector<CrxUpdateItem*>& items) {
- for (const auto& item : items) {
+ for (const auto* item : items) {
if (item->component.requires_network_encryption)
return true;
}
@@ -169,7 +169,7 @@ bool UpdateCheckerImpl::CheckForUpdates(
std::unique_ptr<std::vector<std::string>> ids_checked(
new std::vector<std::string>());
- for (auto crx : items_to_check)
+ for (auto* crx : items_to_check)
ids_checked->push_back(crx->id);
request_sender_.reset(new RequestSender(config_));
request_sender_->Send(
« no previous file with comments | « components/update_client/request_sender.cc ('k') | components/update_client/update_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698