diff options
author | erikchen@chromium.org <erikchen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-31 05:14:36 +0000 |
---|---|---|
committer | erikchen@chromium.org <erikchen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-31 05:14:36 +0000 |
commit | 27cd8ff93067dafdd8aa90eceb83625fb1c9e4fd (patch) | |
tree | b4b7eff44bd28deb757714f45d23afe46d6c7935 /components/autofill/core/browser/autofill_driver.h | |
parent | ae37c4561a23368438671ce73c76c8f5e85f2068 (diff) | |
download | chromium_src-27cd8ff93067dafdd8aa90eceb83625fb1c9e4fd.zip chromium_src-27cd8ff93067dafdd8aa90eceb83625fb1c9e4fd.tar.gz chromium_src-27cd8ff93067dafdd8aa90eceb83625fb1c9e4fd.tar.bz2 |
Mac: Autofill should not immediately request access to address book.
If the browser does not yet have permission to access the user's contacts, one
of the Autofill suggestions has the text "Enable Autofill using Contacts". If
the user selects that suggestion, the browser will prompt the user for access
to the user's address book.
The act of clicking the permissions dialog causes Blink to lose its focus on
the text field, which also dismisses the Autofill popup. If the user has
granted Chrome access to the address book, the autofill popup will be presented
again, and filled in.
This CL does not include the image asset, nor does it include the logic to
display the image asset on the left of the autofill entry.
BUG=139154
TEST=Run the command `tccutil reset AddressBook` to reset AddressBook
permissions for all applications. Clear the default Chromium profile:
`rm -rf ~/Library/Application\ Support/Chromium/Default/`. Launch Chromium by
double clicking on it from Finder. Navigate to
`https://www.mycontactform.com/samples/change_address.php`. Double click the
"First Name" field. There should not be a prompt to access your "Contacts".
There should be exactly 1 autofill entry: "Enable Autofill using Contacts...".
Clicking that autofill entry should open a system prompt that asks for Contacts
permissions for Chromium.
Review URL: https://codereview.chromium.org/286243002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@274040 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'components/autofill/core/browser/autofill_driver.h')
-rw-r--r-- | components/autofill/core/browser/autofill_driver.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/components/autofill/core/browser/autofill_driver.h b/components/autofill/core/browser/autofill_driver.h index efc5696..748ad6e 100644 --- a/components/autofill/core/browser/autofill_driver.h +++ b/components/autofill/core/browser/autofill_driver.h @@ -58,6 +58,9 @@ class AutofillDriver { RendererFormDataAction action, const FormData& data) = 0; + // Pings renderer. The renderer will return an IPC acknowledging the ping. + virtual void PingRenderer() = 0; + // Sends the field type predictions specified in |forms| to the renderer. This // method is a no-op if the renderer is not available or the appropriate // command-line flag is not set. @@ -81,7 +84,6 @@ class AutofillDriver { // Tells the renderer to preview the node with suggested text. virtual void RendererShouldPreviewFieldWithValue( const base::string16& value) = 0; - }; } // namespace autofill |