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

Unified Diff: components/safe_browsing_db/v4_get_hash_protocol_manager.cc

Issue 2345573002: Each DatabaseManager gets to decide which stores to track (Closed)
Patch Set: Incorporated nparker@ feedback 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: components/safe_browsing_db/v4_get_hash_protocol_manager.cc
diff --git a/components/safe_browsing_db/v4_get_hash_protocol_manager.cc b/components/safe_browsing_db/v4_get_hash_protocol_manager.cc
index 8d5191293842c55d37cbcfe1ec0019dde99319cf..d9e487d881494e606d9f2fa6d6e647e59a14de95 100644
--- a/components/safe_browsing_db/v4_get_hash_protocol_manager.cc
+++ b/components/safe_browsing_db/v4_get_hash_protocol_manager.cc
@@ -140,7 +140,7 @@ class V4GetHashProtocolManagerFactoryImpl
~V4GetHashProtocolManagerFactoryImpl() override {}
std::unique_ptr<V4GetHashProtocolManager> CreateProtocolManager(
net::URLRequestContextGetter* request_context_getter,
- const base::hash_set<UpdateListIdentifier>& stores_to_request,
+ const std::unordered_set<UpdateListIdentifier>& stores_to_request,
const V4ProtocolConfig& config) override {
return base::WrapUnique(new V4GetHashProtocolManager(
request_context_getter, stores_to_request, config));
@@ -209,7 +209,7 @@ V4GetHashProtocolManagerFactory* V4GetHashProtocolManager::factory_ = NULL;
// static
std::unique_ptr<V4GetHashProtocolManager> V4GetHashProtocolManager::Create(
net::URLRequestContextGetter* request_context_getter,
- const base::hash_set<UpdateListIdentifier>& stores_to_request,
+ const std::unordered_set<UpdateListIdentifier>& stores_to_request,
const V4ProtocolConfig& config) {
if (!factory_)
factory_ = new V4GetHashProtocolManagerFactoryImpl();
@@ -227,7 +227,7 @@ void V4GetHashProtocolManager::RegisterFactory(
V4GetHashProtocolManager::V4GetHashProtocolManager(
net::URLRequestContextGetter* request_context_getter,
- const base::hash_set<UpdateListIdentifier>& stores_to_request,
+ const std::unordered_set<UpdateListIdentifier>& stores_to_request,
const V4ProtocolConfig& config)
: gethash_error_count_(0),
gethash_back_off_mult_(1),
@@ -308,7 +308,7 @@ void V4GetHashProtocolManager::GetFullHashesWithApis(
ThreatMetadataForApiCallback api_callback) {
DCHECK(url.SchemeIs(url::kHttpScheme) || url.SchemeIs(url::kHttpsScheme));
- base::hash_set<FullHash> full_hashes;
+ std::unordered_set<FullHash> full_hashes;
V4ProtocolManagerUtil::UrlToFullHashes(url, &full_hashes);
FullHashToStoreAndHashPrefixesMap full_hash_to_store_and_hash_prefixes;
@@ -365,7 +365,7 @@ void V4GetHashProtocolManager::GetFullHashCachedResults(
// cache entry if they expire AND their expire time is after the negative
// cache expire time.
- base::hash_set<HashPrefix> unique_prefixes_to_request;
+ std::unordered_set<HashPrefix> unique_prefixes_to_request;
for (const auto& it : full_hash_to_store_and_hash_prefixes) {
const FullHash& full_hash = it.first;
const StoreAndHashPrefixes& matched = it.second;
@@ -464,7 +464,7 @@ void V4GetHashProtocolManager::HandleGetHashError(const Time& now) {
void V4GetHashProtocolManager::OnFullHashForApi(
const ThreatMetadataForApiCallback& api_callback,
- const base::hash_set<FullHash>& full_hashes,
+ const std::unordered_set<FullHash>& full_hashes,
const std::vector<FullHashInfo>& full_hash_infos) {
ThreatMetadata md;
for (const FullHashInfo& full_hash_info : full_hash_infos) {

Powered by Google App Engine
This is Rietveld 408576698