Support Android username-only credentials.

components/password_manager/core/browser/affiliated_match_helper.h

Index: components/password_manager/core/browser/affiliated_match_helper.h
diff --git a/components/password_manager/core/browser/affiliated_match_helper.h b/components/password_manager/core/browser/affiliated_match_helper.h
index 5197174222baa1225450fb4e812315b7fd6d71b4..dfb6c100f230805f535a9d7f825713adc8fcd61c 100644
--- a/components/password_manager/core/browser/affiliated_match_helper.h
+++ b/components/password_manager/core/browser/affiliated_match_helper.h
@@ -90,6 +90,11 @@ class AffiliatedMatchHelper : public PasswordStore::Observer,
   // purposes of affiliation-based matching.
   static bool IsValidWebCredential(const autofill::PasswordForm& form);
 
+  // Removes Android username-only credentials from |android_credentials|.
+  // Transforms federated credentials into non zero-click ones.
+  static void TrimAndroidUsernameOnlyCredentials(
+      ScopedVector<autofill::PasswordForm>* android_credentials);

[engedy, 2015/10/13 12:11:33]

What is the reason for passing a pointer to the ScopedVector, as opposed to
taking and returning a ScopedVector by value? That solution would allow
functional-style chaining of filters.

[vasilii, 2015/10/13 18:05:26]

It's used in 2 places. Both of them benefit from modifying the parameter
directly.
I'd agree with the functional style if we didn't need to call Pass() every time.

+
   // Sets the task runner to be used to delay I/O heavy initialization. Should
   // be called before Initialize(). Used only for testing.
   void SetTaskRunnerUsedForWaitingForTesting(