| Index: components/password_manager/core/browser/password_store.h
|
| diff --git a/components/password_manager/core/browser/password_store.h b/components/password_manager/core/browser/password_store.h
|
| index ccc2353266a64eb8b0e1e041d2dcb64ca4739eb5..02df82bc04671ec11bb4bec43d07dfc9f12d3c5d 100644
|
| --- a/components/password_manager/core/browser/password_store.h
|
| +++ b/components/password_manager/core/browser/password_store.h
|
| @@ -55,10 +55,6 @@ struct InteractionsStats;
|
| class PasswordStore : protected PasswordStoreSync,
|
| public RefcountedKeyedService {
|
| public:
|
| - // Whether or not it's acceptable for Chrome to request access to locked
|
| - // passwords, which requires prompting the user for permission.
|
| - enum AuthorizationPromptPolicy { ALLOW_PROMPT, DISALLOW_PROMPT };
|
| -
|
| // An interface used to notify clients (observers) of this object that data in
|
| // the password store has changed. Register the observer via
|
| // PasswordStore::AddObserver.
|
| @@ -148,16 +144,10 @@ class PasswordStore : protected PasswordStoreSync,
|
|
|
| // Searches for a matching PasswordForm, and notifies |consumer| on
|
| // completion. The request will be cancelled if the consumer is destroyed.
|
| - // |prompt_policy| indicates whether it's permissible to prompt the user to
|
| - // authorize access to locked passwords. This argument is only used on
|
| - // platforms that support prompting the user for access (such as Mac OS).
|
| - // NOTE: This means that this method can return different results depending
|
| - // on the value of |prompt_policy|.
|
| // TODO(engedy): Currently, this will not return federated logins saved from
|
| // Android applications that are affiliated with the realm of |form|. Need to
|
| // decide if this is the desired behavior. See: https://crbug.com/539844.
|
| virtual void GetLogins(const autofill::PasswordForm& form,
|
| - AuthorizationPromptPolicy prompt_policy,
|
| PasswordStoreConsumer* consumer);
|
|
|
| // Gets the complete list of PasswordForms that are not blacklist entries--and
|
| @@ -272,7 +262,6 @@ class PasswordStore : protected PasswordStoreSync,
|
| // to be virtual only because asynchronous behavior in PasswordStoreWin.
|
| // TODO(engedy): Make this non-virtual once https://crbug.com/78830 is fixed.
|
| virtual void GetLoginsImpl(const autofill::PasswordForm& form,
|
| - AuthorizationPromptPolicy prompt_policy,
|
| scoped_ptr<GetLoginsRequest> request);
|
|
|
| // Synchronous implementation provided by subclasses to add the given login.
|
| @@ -292,8 +281,7 @@ class PasswordStore : protected PasswordStoreSync,
|
| // Finds and returns all PasswordForms with the same signon_realm as |form|,
|
| // or with a signon_realm that is a PSL-match to that of |form|.
|
| virtual ScopedVector<autofill::PasswordForm> FillMatchingLogins(
|
| - const autofill::PasswordForm& form,
|
| - AuthorizationPromptPolicy prompt_policy) = 0;
|
| + const autofill::PasswordForm& form) = 0;
|
|
|
| // Synchronous implementation for manipulating with statistics.
|
| virtual void AddSiteStatsImpl(const InteractionsStats& stats) = 0;
|
| @@ -386,14 +374,12 @@ class PasswordStore : protected PasswordStoreSync,
|
| // and takes care of notifying the consumer with the results when done.
|
| void GetLoginsWithAffiliationsImpl(
|
| const autofill::PasswordForm& form,
|
| - AuthorizationPromptPolicy prompt_policy,
|
| scoped_ptr<GetLoginsRequest> request,
|
| const std::vector<std::string>& additional_android_realms);
|
|
|
| // Schedules GetLoginsWithAffiliationsImpl() to be run on the DB thread.
|
| void ScheduleGetLoginsWithAffiliations(
|
| const autofill::PasswordForm& form,
|
| - AuthorizationPromptPolicy prompt_policy,
|
| scoped_ptr<GetLoginsRequest> request,
|
| const std::vector<std::string>& additional_android_realms);
|
|
|
|
|