| Index: components/autofill/content/renderer/password_generation_agent.cc
|
| diff --git a/components/autofill/content/renderer/password_generation_agent.cc b/components/autofill/content/renderer/password_generation_agent.cc
|
| index 83146b4feb98293dc2a9390b1d280f8a3a455283..74a6113d0116924139ac304ff72777ab7c4e183b 100644
|
| --- a/components/autofill/content/renderer/password_generation_agent.cc
|
| +++ b/components/autofill/content/renderer/password_generation_agent.cc
|
| @@ -33,8 +33,6 @@
|
|
|
| namespace autofill {
|
|
|
| -using form_util::StripAuthAndParams;
|
| -
|
| namespace {
|
|
|
| // Returns true if we think that this form is for account creation. |passwords|
|
| @@ -58,18 +56,11 @@ bool ContainsURL(const std::vector<GURL>& urls, const GURL& url) {
|
| }
|
|
|
| // Finds a form in |forms| that has the same action and name as |form|.
|
| -// If the action of a form in |forms| is empty, it uses |base_url| as action. It
|
| -// also strips parameters of the action.
|
| const PasswordFormGenerationData* FindFormGenerationData(
|
| const std::vector<PasswordFormGenerationData>& forms,
|
| - const PasswordForm& form,
|
| - const GURL& base_url) {
|
| + const PasswordForm& form) {
|
| for (const auto& form_it : forms) {
|
| - GURL action = form_it.action;
|
| - if (action.is_empty())
|
| - action = base_url;
|
| - action = form_util::StripAuthAndParams(action);
|
| - if (form_it.name == form.form_data.name && action == form.action)
|
| + if (form_it.name == form.form_data.name && form_it.action == form.action)
|
| return &form_it;
|
| }
|
| return nullptr;
|
| @@ -391,9 +382,8 @@ void PasswordGenerationAgent::DetermineGenerationElement() {
|
| << "blacklisted";
|
| continue;
|
| } else {
|
| - generation_data = FindFormGenerationData(
|
| - generation_enabled_forms_, *possible_password_form,
|
| - render_frame()->GetWebFrame()->document().baseURL());
|
| + generation_data = FindFormGenerationData(generation_enabled_forms_,
|
| + *possible_password_form);
|
| if (!generation_data) {
|
| if (AutocompleteAttributesSetForGeneration(*possible_password_form)) {
|
| VLOG(2) << "Ignoring lack of Autofill signal due to Autocomplete "
|
|
|