Implement support for checking bad IPs aka MatchMalwareIP

components/safe_browsing_db/v4_local_database_manager.h

Index: components/safe_browsing_db/v4_local_database_manager.h
diff --git a/components/safe_browsing_db/v4_local_database_manager.h b/components/safe_browsing_db/v4_local_database_manager.h
index 297fdf058a9a9c51d11980a68e2003049cd057e9..e704569e5caa1b623045d20c1edd38bc92c8a0b7 100644
--- a/components/safe_browsing_db/v4_local_database_manager.h
+++ b/components/safe_browsing_db/v4_local_database_manager.h
@@ -95,9 +95,13 @@ class V4LocalDatabaseManager : public SafeBrowsingDatabaseManager {
     // unsafe resource.
     CHECK_RESOURCE_URL = 2,
 
-    // This represents the case where we're trying to determine if a Chrome
+    // This represents the case when we're trying to determine if a Chrome
     // extension is a unsafe.
-    CHECK_EXTENSION_IDS = 3
+    CHECK_EXTENSION_IDS = 3,
+
+    // This represents the case when we're trying to determing if an IP address
+    // is unsafe due to hosting Malware.
+    CHECK_MALWARE_IP = 4
   };
 
   // The information we need to process a URL safety reputation request and
@@ -194,9 +198,16 @@ class V4LocalDatabaseManager : public SafeBrowsingDatabaseManager {
   // Queues the check for async response if the database isn't ready yet.
   // If the database is ready, checks the database for prefix matches and
   // returns true immediately if there's no match. If a match is found, it
-  // schedules a task to perform full hash check and returns false.
+  // schedules a task to perform full hash check and returns false. If
+  // |synchronous_response| is false, an asynchronous full hash request to

[Nathan Parker, 2016/11/15 00:54:46]

Update comments, since |synchronous_response| isn't there anymore

[vakh (use Gerrit instead), 2016/11/15 01:03:41]

Done.

+  // fetch full hashes for the matching hash prefixes is sent to the
+  // SafeBrowsing service.
   bool HandleCheck(std::unique_ptr<PendingCheck> check);
 
+  // Checks the database for prefix matches. Returns true if the database isn't
+  // ready or if there's no match; false otherwise.

[Nathan Parker, 2016/11/15 00:54:46]

Maybe add a comment to say what this is used for: For things where a full-hash
check isn't needed

[vakh (use Gerrit instead), 2016/11/15 01:03:41]

Done.

+  bool HandleCheckSynchronously(std::unique_ptr<PendingCheck> check);
+
   // Called when the |v4_get_hash_protocol_manager_| has the full hash response
   // available for the URL that we requested. It determines the severest
   // threat type and responds to the |client| with that information.