Start the migration of passwords from the Keychain.

components/password_manager/core/browser/password_manager.cc

Index: components/password_manager/core/browser/password_manager.cc
diff --git a/components/password_manager/core/browser/password_manager.cc b/components/password_manager/core/browser/password_manager.cc
index fb8178bc02c49ada89127b90f3aa58ee54d92117..9d17efed7e6fdea08db1a4cb5bb6a33e754daab8 100644
--- a/components/password_manager/core/browser/password_manager.cc
+++ b/components/password_manager/core/browser/password_manager.cc
@@ -142,6 +142,9 @@ void PasswordManager::RegisterProfilePrefs(
   registry->RegisterBooleanPref(prefs::kPasswordManagerAllowShowPasswords,
                                 true);
   registry->RegisterListPref(prefs::kPasswordManagerGroupsForDomains);
+#if defined(OS_MACOSX)
+  registry->RegisterIntegerPref(prefs::kKeychainMigrationStatus, 0);

[vabr (Chromium), 2015/07/03 08:40:48]

While not detectable to the compiler, this is a kind of dependency violation --
the 0 here means the proxy's NOT_STARTED constant, which is defined in //chrome
and "used" here in //components.

There are multiple ways to address this, out of which I find the following the
easiest and sufficiently clean:

Move the definition of MigrationStatus to
components/password_manager/core/common/password_manager_pref_names.h. The pref
name is already there, so it makes sense to define the pref's values in the same
place. Then you can refer NOT_STARTED here directly, and there will be no
dependency violation.

[vasilii, 2015/07/03 12:59:11]

Done.

+#endif
 }
 
 #if defined(OS_WIN)