diff options
| author | csharp@chromium.org <csharp@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-03 15:04:40 +0000 |
|---|---|---|
| committer | csharp@chromium.org <csharp@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-03 15:04:40 +0000 |
| commit | 6aec8932ddfb5771d227a2103e93b9e67de2170a (patch) | |
| tree | 719004e8e0a67a9e13d3815cb5d5eb7886730164 /components/autofill/core | |
| parent | 1f9fd547750a691719ec8ef27ce285c92d4f91e4 (diff) | |
| download | chromium_src-6aec8932ddfb5771d227a2103e93b9e67de2170a.zip chromium_src-6aec8932ddfb5771d227a2103e93b9e67de2170a.tar.gz chromium_src-6aec8932ddfb5771d227a2103e93b9e67de2170a.tar.bz2 | |
[Autofill] Remove IsNativeUiEnabled
The native UI is now always on and the flag to revert to the old
UI has been removed, so this will now always be true.
TBR=isherman@chromium.org
BUG=255625
Review URL: https://chromiumcodereview.appspot.com/18173006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@209984 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'components/autofill/core')
| -rw-r--r-- | components/autofill/core/browser/autofill_manager.cc | 7 | ||||
| -rw-r--r-- | components/autofill/core/browser/autofill_manager.h | 3 | ||||
| -rw-r--r-- | components/autofill/core/common/autofill_messages.h | 5 |
3 files changed, 3 insertions, 12 deletions
diff --git a/components/autofill/core/browser/autofill_manager.cc b/components/autofill/core/browser/autofill_manager.cc index 7912bb8..7a1d925 100644 --- a/components/autofill/core/browser/autofill_manager.cc +++ b/components/autofill/core/browser/autofill_manager.cc @@ -246,10 +246,6 @@ void AutofillManager::SetExternalDelegate(AutofillExternalDelegate* delegate) { autocomplete_history_manager_->SetExternalDelegate(delegate); } -bool AutofillManager::IsNativeUiEnabled() { - return external_delegate_ != NULL; -} - bool AutofillManager::OnFormSubmitted(const FormData& form, const TimeTicks& timestamp) { // Let Autocomplete know as well. @@ -620,8 +616,7 @@ void AutofillManager::OnDidShowAutofillSuggestions(bool is_new_popup) { } void AutofillManager::OnHideAutofillUi() { - if (IsNativeUiEnabled()) - manager_delegate_->HideAutofillPopup(); + manager_delegate_->HideAutofillPopup(); manager_delegate_->HideAutocheckoutBubble(); } diff --git a/components/autofill/core/browser/autofill_manager.h b/components/autofill/core/browser/autofill_manager.h index a4f0194..496a2a2 100644 --- a/components/autofill/core/browser/autofill_manager.h +++ b/components/autofill/core/browser/autofill_manager.h @@ -87,9 +87,6 @@ class AutofillManager : public AutofillDownloadManager::Observer { // Sets an external delegate. void SetExternalDelegate(AutofillExternalDelegate* delegate); - // Whether browser process will create and own the Autofill popup UI. - bool IsNativeUiEnabled(); - // Called from our external delegate so they cannot be private. virtual void OnFillAutofillFormData(int query_id, const FormData& form, diff --git a/components/autofill/core/common/autofill_messages.h b/components/autofill/core/common/autofill_messages.h index 2e94713..d504855 100644 --- a/components/autofill/core/common/autofill_messages.h +++ b/components/autofill/core/common/autofill_messages.h @@ -125,9 +125,8 @@ IPC_MESSAGE_ROUTED2(AutofillMsg_FormDataFilled, // Fill a password form and prepare field autocomplete for multiple // matching logins. Lets the renderer know if it should disable the popup // because the browser process will own the popup UI. -IPC_MESSAGE_ROUTED2(AutofillMsg_FillPasswordForm, - autofill::PasswordFormFillData, /* the fill form data*/ - bool /* disable popup */ ) +IPC_MESSAGE_ROUTED1(AutofillMsg_FillPasswordForm, + autofill::PasswordFormFillData /* the fill form data*/) // Send the heuristic and server field type predictions to the renderer. IPC_MESSAGE_ROUTED1( |