diff options
| author | vabr <vabr@chromium.org> | 2014-10-31 02:36:06 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2014-10-31 09:36:21 +0000 |
| commit | 86cc798cc2947dd88aa73505d716f93538dffa5b (patch) | |
| tree | 9f062f05f5231c4b313b500c58419ce17fb8bceb /components/autofill/content/common | |
| parent | a52fbd252fdaea6395a31e6336608c3b26794aaa (diff) | |
| download | chromium_src-86cc798cc2947dd88aa73505d716f93538dffa5b.zip chromium_src-86cc798cc2947dd88aa73505d716f93538dffa5b.tar.gz chromium_src-86cc798cc2947dd88aa73505d716f93538dffa5b.tar.bz2 | |
Do not haul suggestions back to browser in AutofillHostMsg_ShowPasswordSuggestions
Currently, PasswordAutofillAgent (in renderer) prepares password autofill suggestions for PasswordAutofillManager (in browser) and sends them via IPC.
But the manager has the data to generate the suggestions itself, so this CL makes it derive the suggestions from that data instead of hauling it through IPC.
This CL also adds one more test case to the PasswordAutofillManager unittest, to compensate for the coverage lost in the PasswordAutofillAgent browsertest by moving some logic out of the agent to the manager.
BUG=400186, 377422, 118601
Review URL: https://codereview.chromium.org/688633004
Cr-Commit-Position: refs/heads/master@{#302245}
Diffstat (limited to 'components/autofill/content/common')
| -rw-r--r-- | components/autofill/content/common/autofill_messages.h | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/components/autofill/content/common/autofill_messages.h b/components/autofill/content/common/autofill_messages.h index 25d8277..e339a64 100644 --- a/components/autofill/content/common/autofill_messages.h +++ b/components/autofill/content/common/autofill_messages.h @@ -285,13 +285,12 @@ IPC_MESSAGE_ROUTED2(AutofillHostMsg_AddPasswordFormMapping, autofill::FormFieldData, /* the user name field */ autofill::PasswordFormFillData /* password pairings */) -// Instruct the browser to show a popup with the following suggestions from the -// password manager. +// Instruct the browser to show a popup with suggestions for the form field. IPC_MESSAGE_ROUTED4(AutofillHostMsg_ShowPasswordSuggestions, autofill::FormFieldData /* the form field */, - gfx::RectF /* input field bounds, window-relative */, - std::vector<base::string16> /* suggestions */, - std::vector<base::string16> /* realms */) + base::string16 /* username typed by user */, + bool /* show all suggestions */, + gfx::RectF /* input field bounds, window-relative */) // Inform browser of data list values for the curent field. IPC_MESSAGE_ROUTED2(AutofillHostMsg_SetDataList, |