Sending generated vote on password generation.

components/autofill/core/browser/form_structure.cc

 // 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.
 
 #include "components/autofill/core/browser/form_structure.h"
 
 #include <stdint.h>
 
 #include <algorithm>
 #include <map>
@@ skipping 1088 matching lines @@
       continue;
 
     const ServerFieldTypeSet& types = field->possible_types();
     for (const auto& field_type : types) {
       // Add the same field elements as the query and a few more below.
       if (ShouldSkipField(*field))
         continue;
 
       AutofillUploadContents::Field* added_field = upload->add_field();
       added_field->set_autofill_type(field_type);
+      if (field->generation_type())
+        added_field->set_generation_type(
+            static_cast<
+                ::autofill::
+                    AutofillUploadContents_Field_PasswordGenerationType>(
+                field->generation_type()));

[dvadym, 2016/02/19 16:25:06]

Mathieu@: is it correct way to use proto enums?

[Mathieu, 2016/02/22 13:25:20]

if |generation_type| is of type
AutofillUploadContents::Field::PasswordGenerationType, you wouldn't need the
cast here (probably also wouldn't need the ::autofill:: since you are already in
the autofill namespace?)

[dvadym, 2016/02/22 14:52:32]

Done.

 
       unsigned sig = 0;
       // The signature is a required field. If it can't be parsed, the proto
       // would not serialize.
       if (!base::StringToUint(field->FieldSignature(), &sig))
         continue;
       added_field->set_signature(sig);
 
       if (IsAutofillFieldMetadataEnabled()) {
         added_field->set_type(field->form_control_type);
@@ skipping 179 matching lines @@
           strings[i].at(prefix_len) != strings[0].at(prefix_len)) {
         // Mismatch found.
         return base::StringPiece16(strings[i].data(), prefix_len);
       }
     }
   }
   return strings[0];
 }
 
 }  // namespace autofill