Merge autofill::FormFieldData::is_checkable and is_checked

components/autofill/core/common/form_field_data.h

Index: components/autofill/core/common/form_field_data.h
diff --git a/components/autofill/core/common/form_field_data.h b/components/autofill/core/common/form_field_data.h
index 5a9bb8a89356c48dba81b25d492b3836333a9d39..9b63b6e086005f940d4d9f5dd404a412ec280687 100644
--- a/components/autofill/core/common/form_field_data.h
+++ b/components/autofill/core/common/form_field_data.h
@@ -29,6 +29,12 @@ struct FormFieldData {
     ROLE_ATTRIBUTE_OTHER,
   };
 
+  enum CheckStatus {

[vabr (Chromium), 2016/05/31 15:48:25]

Please use enum class instead of enum, to get better protection against unwanted
casts and also to isolate the values in their own namespace.

Once you do that, please drop the CHECK_STATUS_ prefix from the values, it will
be replaced by the namespace.

[Hwanseung Lee(hs1217.lee), 2016/06/09 15:41:59]

if enum class used instead of enum, it is complex to serialize & deserialize.
could you teach me how to use easily serialize & deserialize when use enum
class?

+    CHECK_STATUS_NOT_CHECKABLE,
+    CHECK_STATUS_CHECKABLE_BUT_UNCHECKED,
+    CHECK_STATUS_CHECKED,
+  };
+
   FormFieldData();
   FormFieldData(const FormFieldData& other);
   ~FormFieldData();
@@ -53,8 +59,7 @@ struct FormFieldData {
   // (base::Pickle used to serialize that as 64 bit).
   uint64_t max_length;
   bool is_autofilled;
-  bool is_checked;
-  bool is_checkable;
+  CheckStatus check_status;
   bool is_focusable;
   bool should_autocomplete;
   RoleAttribute role;