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

Unified Diff: chrome/browser/safe_browsing/browser_feature_extractor.h

Issue 21170004: Store and return urls instead of hosts for the malware IP matching. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 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
Index: chrome/browser/safe_browsing/browser_feature_extractor.h
diff --git a/chrome/browser/safe_browsing/browser_feature_extractor.h b/chrome/browser/safe_browsing/browser_feature_extractor.h
index 086e9bb76f08ecb986a5e748b04f569c4e940e8d..9a6fbf5db10ef71528737c3e3c4ce22840eb0a12 100644
--- a/chrome/browser/safe_browsing/browser_feature_extractor.h
+++ b/chrome/browser/safe_browsing/browser_feature_extractor.h
@@ -39,12 +39,12 @@ class ClientMalwareRequest;
class ClientPhishingRequest;
class ClientSideDetectionService;
-typedef std::map<std::string, std::set<std::string> > IPHostMap;
+typedef std::map<std::string, std::set<std::string> > IPUrlMap;
struct BrowseInfo {
// List of IPv4 and IPv6 addresses from which content was requested
// together with the hosts on it, while browsing to the |url|.
- IPHostMap ips;
+ IPUrlMap ips;
// If a SafeBrowsing interstitial was shown for the current URL
// this will contain the UnsafeResource struct for that URL.
@@ -172,6 +172,9 @@ class BrowserFeatureExtractor {
// the history callback hasn't been invoked yet).
PendingQueriesMap pending_queries_;
+ // Max number of malware IPs can be sent in one malware request
+ static const int kMaxMalwareIPPerRequest;
+
DISALLOW_COPY_AND_ASSIGN(BrowserFeatureExtractor);
};

Powered by Google App Engine
This is Rietveld 408576698