| Index: components/autofill/core/common/password_form.h
|
| diff --git a/components/autofill/core/common/password_form.h b/components/autofill/core/common/password_form.h
|
| index 88a7840fd84cfe3dd91376e2ea96956689be044e..67da0ec36246b97efbd3b0ebedbb1cf8757c41ae 100644
|
| --- a/components/autofill/core/common/password_form.h
|
| +++ b/components/autofill/core/common/password_form.h
|
| @@ -246,11 +246,13 @@ struct PasswordForm {
|
| // User friendly name to show in the UI.
|
| base::string16 display_name;
|
|
|
| - // The URL of the user's avatar to display in the UI. Note that the
|
| - // corresponding property in the Credential Manager is called icon URL.
|
| - // TODO(msramek): Rename |avatar_url| to |icon_url| to match the naming
|
| - // in Credential Manager.
|
| - GURL avatar_url;
|
| + // The URL of this credential's icon, such as the user's avatar, to display
|
| + // in the UI.
|
| + // TODO(msramek): This field was previously named |avatar_url|. It is still
|
| + // named this way in the password store backends (e.g. the avatar_url column
|
| + // in the SQL DB of LoginDatabase) and for the purposes of syncing
|
| + // (i.e in PasswordSpecificsData). Rename these occurrences.
|
| + GURL icon_url;
|
|
|
| // The URL of identity provider used for federated login.
|
| GURL federation_url;
|
|
|