[Autofill] No longer pass FormStructure pointers in ParseQueryResponse.

components/autofill/core/browser/form_structure.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_AUTOFILL_CORE_BROWSER_FORM_STRUCTURE_H_
 #define COMPONENTS_AUTOFILL_CORE_BROWSER_FORM_STRUCTURE_H_
 
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 53 matching lines @@
 
   // Encodes a XML block contains autofill field type from this FormStructure.
   // This XML will be written VLOG only, never be sent to server. It will
   // help make FieldAssignments and feed back to autofill server as
   // experiment data.
   bool EncodeFieldAssignments(const ServerFieldTypeSet& available_field_types,
                               std::string* encoded_xml) const;
 
   // Encodes the XML query request for the set of |forms| that are valid (see
   // implementation for details on which forms are not included in the query).
-  // The forms and form signatures used in the Query request are output in
-  // |queried_forms| and |encoded_signatures|, respectively. All valid fields
-  // are encoded in |encoded_xml|. For example, there are three valid forms,
-  // with 2, 4, and 3 fields. The returned XML would have type info for 9
-  // fields, first two of which would be for the first form, next 4 for the
-  // second, and the rest is for the third.
+  // The form signatures used in the Query request are output in
+  // |encoded_signatures|, respectively. All valid fields are encoded in

[vabr (Chromium), 2015/11/19 07:56:04]

nit: No need for 'respectively' any more.

[Mathieu, 2015/11/19 13:20:55]

Done.

+  // |encoded_xml|. For example, there are three valid forms, with 2, 4, and 3
+  // fields. The returned XML would have type info for 9 fields, first two of
+  // which would be for the first form, next 4 for the second, and the rest is
+  // for the third.
   static bool EncodeQueryRequest(const std::vector<FormStructure*>& forms,
                                  std::vector<std::string>* encoded_signatures,
-                                 std::vector<FormStructure*>* queried_forms,
                                  std::string* encoded_xml);
 
   // Parses the field types from the server query response. |forms| must be the
   // same as the one passed to EncodeQueryRequest when constructing the query.
   // |rappor_service| may be null.
   static void ParseQueryResponse(const std::string& response_xml,
                                  const std::vector<FormStructure*>& forms,
                                  rappor::RapporService* rappor_service);
 
   // Returns predictions using the details from the given |form_structures| and
@@ skipping 180 matching lines @@
 
   // True if the form is a <form>.
   bool is_form_tag_;
 
   DISALLOW_COPY_AND_ASSIGN(FormStructure);
 };
 
 }  // namespace autofill
 
 #endif  // COMPONENTS_AUTOFILL_CORE_BROWSER_FORM_STRUCTURE_H_