Enable import/export of passwords into/from Password Manager

components/password_manager/core/common/password_manager_features.cc

Index: components/password_manager/core/common/password_manager_features.cc
diff --git a/components/password_manager/core/common/password_manager_features.cc b/components/password_manager/core/common/password_manager_features.cc
index c1f9f4d9ca88cbe09a023a73293ff47dadb258a3..e09c409b2b0ee9124a1c3939f39eb9c91b979512 100644
--- a/components/password_manager/core/common/password_manager_features.cc
+++ b/components/password_manager/core/common/password_manager_features.cc
@@ -27,6 +27,9 @@ const base::Feature kEnablePasswordForceSaving = {
 extern const base::Feature kEnableManualPasswordGeneration = {
     "enable-manual-password-generation", base::FEATURE_DISABLED_BY_DEFAULT};
 
+const base::Feature kEnablePasswordImportExport = {
+    "enable-password-import-export", base::FEATURE_DISABLED_BY_DEFAULT};

[vabr (Chromium), 2016/03/10 10:53:33]

While the "enable" prefix matches the rest of the features here, it is a bad
practice (and we are working on fixing it for the existing ones).

The feature is just "password-import-export", enabling or disabling it is an
action one can take on any feature. So please drop the "enable" both from the
feature name and the constants name.

[xunlu, 2016/03/16 07:23:59]

Done.

+
 }  // namespace features
 
 }  // namespace password_manager