Chromium Code Reviews| Index: components/autofill/content/renderer/password_autofill_agent.cc |
| diff --git a/components/autofill/content/renderer/password_autofill_agent.cc b/components/autofill/content/renderer/password_autofill_agent.cc |
| index d59aeefe9761283cca12ef84d1080f9a578a23ac..7e8e1c3a540917a69a6a2ebdb31ca3574510e367 100644 |
| --- a/components/autofill/content/renderer/password_autofill_agent.cc |
| +++ b/components/autofill/content/renderer/password_autofill_agent.cc |
| @@ -612,6 +612,10 @@ bool PasswordAutofillAgent::ShowSuggestionPopup( |
| FindFormAndFieldForInputElement( |
| user_input, &form, &field, REQUIRE_NONE); |
| + scoped_ptr<PasswordForm> password_form = |
| + CreatePasswordForm(user_input.form()); |
| + DCHECK(password_form.get()); |
| + |
| blink::WebInputElement selected_element = user_input; |
| gfx::Rect bounding_box(selected_element.boundsInViewportSpace()); |
| @@ -622,6 +626,7 @@ bool PasswordAutofillAgent::ShowSuggestionPopup( |
| bounding_box.height() * scale); |
| Send(new AutofillHostMsg_ShowPasswordSuggestions(routing_id(), |
| field, |
| + *password_form.get(), |
|
vabr (Chromium)
2014/01/13 14:46:29
The ".get()" should not be needed.
riadh.chtara
2014/01/13 20:34:10
Done.
vabr (Chromium)
2014/01/14 14:11:02
Again, not done -- ".get()" is still there.
|
| bounding_box_scaled, |
| suggestions, |
| realms)); |