Enable import/export of passwords into/from Password Manager

components/password_manager/core/common/password_manager_features.cc

 // Copyright 2015 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/password_manager/core/common/password_manager_features.h"
 
 namespace password_manager {
 
 namespace features {
 
 // Disables the save-password prompt. Passwords are then saved automatically,
 // without asking the user.
 const base::Feature kEnableAutomaticPasswordSaving = {
     "enable-automatic-password-saving", base::FEATURE_DISABLED_BY_DEFAULT};
 
 // Enable supporting of updating password in the password manager on a password
 // change form submit.
 const base::Feature kEnablePasswordChangeSupport = {
     "enable-password-change-support", base::FEATURE_DISABLED_BY_DEFAULT};
 
 // Enable a context menu item in the password field that allows the user
 // to manually enforce saving of their password.
 const base::Feature kEnablePasswordForceSaving = {
     "enable-password-force-saving", base::FEATURE_DISABLED_BY_DEFAULT};
 
 // Enable the user to trigger password generation manually.
 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