SafeBrowsing PVer4: Send mutable response to the database and the stores

components/safe_browsing_db/v4_update_protocol_manager.h

Index: components/safe_browsing_db/v4_update_protocol_manager.h
diff --git a/components/safe_browsing_db/v4_update_protocol_manager.h b/components/safe_browsing_db/v4_update_protocol_manager.h
index 1439cc65900af85b1a5eaf2067449405afff3ecd..053229824a50833289cc330896fec6c48358b52c 100644
--- a/components/safe_browsing_db/v4_update_protocol_manager.h
+++ b/components/safe_browsing_db/v4_update_protocol_manager.h
@@ -41,7 +41,8 @@ class V4UpdateProtocolManagerFactory;
 // Parameters:
 //   - The vector of update response protobufs received from the server for
 //     each list type.
-typedef base::Callback<void(const std::vector<ListUpdateResponse>&)>
+typedef base::Callback<void(
+    std::unique_ptr<ParsedServerResponse> parsed_server_response)>

[Scott Hess - ex-Googler, 2016/06/28 03:58:43]

Don't need the parameter's name in this context, just the type.

[vakh (use Gerrit instead), 2016/06/28 21:34:14]

Done.

     V4UpdateCallback;
 
 class V4UpdateProtocolManager : public net::URLFetcherDelegate,
@@ -104,9 +105,8 @@ class V4UpdateProtocolManager : public net::URLFetcherDelegate,
   // FetchThreatListUpdatesResponse protobuf and returns each of the
   // ListUpdateResponse protobufs contained in it as a vector.
   // Returns true if parsing is successful, false otherwise.
-  bool ParseUpdateResponse(
-      const std::string& data_base64,
-      std::vector<ListUpdateResponse>* list_update_responses);
+  bool ParseUpdateResponse(const std::string& data_base64,
+                           ParsedServerResponse* parsed_server_response);
 
   // Resets the update error counter and multiplier.
   void ResetUpdateErrors();