Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3463)

Unified Diff: chrome/browser/autofill/autofill_manager.cc

Issue 11867025: Download autocheckout whitelist and enable autocheckout for whitelisted sites only. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Pass autocheckout url prefix to FormStructure's constructor. Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/autofill/autofill_manager.cc
diff --git a/chrome/browser/autofill/autofill_manager.cc b/chrome/browser/autofill/autofill_manager.cc
index 150d51a7fa39b81fe2e7269606dda8b8fb6b41b8..62643044d818dc3af8bfc59d29abf47e97aed311 100644
--- a/chrome/browser/autofill/autofill_manager.cc
+++ b/chrome/browser/autofill/autofill_manager.cc
@@ -23,6 +23,7 @@
#include "base/utf_string_conversions.h"
#include "chrome/browser/api/infobars/infobar_service.h"
#include "chrome/browser/api/sync/profile_sync_service_base.h"
+#include "chrome/browser/autofill/autocheckout/whitelist_manager.h"
#include "chrome/browser/autofill/autocheckout_manager.h"
#include "chrome/browser/autofill/autocheckout_infobar_delegate.h"
#include "chrome/browser/autofill/autocomplete_history_manager.h"
@@ -393,7 +394,8 @@ bool AutofillManager::OnFormSubmitted(const FormData& form,
return false;
// Grab a copy of the form data.
- scoped_ptr<FormStructure> submitted_form(new FormStructure(form));
+ scoped_ptr<FormStructure> submitted_form(
+ new FormStructure(form, GetAutocheckoutURLPrefix()));
// Disregard forms that we wouldn't ever autofill in the first place.
if (!submitted_form->ShouldBeParsed(true))
@@ -885,6 +887,16 @@ void AutofillManager::OnClickFailed(autofill::AutocheckoutStatus status) {
// TODO(ahutter): Plug into WalletClient.
}
+std::string AutofillManager::GetAutocheckoutURLPrefix() const {
+ if (!web_contents())
+ return false;
+
+ autofill::autocheckout::WhitelistManager* whitelist_manager =
+ autofill::autocheckout::WhitelistManager::GetForBrowserContext(
+ web_contents()->GetBrowserContext());
+ return whitelist_manager->GetMatchedURLPrefix(web_contents()->GetURL());
+}
+
bool AutofillManager::IsAutofillEnabled() const {
return manager_delegate_->GetPrefs()->GetBoolean(prefs::kAutofillEnabled);
}
@@ -1085,7 +1097,7 @@ bool AutofillManager::GetCachedFormAndField(const FormData& form,
// If we do not have this form in our cache but it is parseable, we'll add it
// in the call to |UpdateCachedForm()|.
if (!FindCachedForm(form, form_structure) &&
- !FormStructure(form).ShouldBeParsed(false)) {
+ !FormStructure(form, GetAutocheckoutURLPrefix()).ShouldBeParsed(false)) {
return false;
}
@@ -1132,7 +1144,8 @@ bool AutofillManager::UpdateCachedForm(const FormData& live_form,
return false;
// Add the new or updated form to our cache.
- form_structures_.push_back(new FormStructure(live_form));
+ form_structures_.push_back(
+ new FormStructure(live_form, GetAutocheckoutURLPrefix()));
*updated_form = *form_structures_.rbegin();
(*updated_form)->DetermineHeuristicTypes(*metric_logger_);
@@ -1208,9 +1221,11 @@ void AutofillManager::GetCreditCardSuggestions(
void AutofillManager::ParseForms(const std::vector<FormData>& forms) {
std::vector<FormStructure*> non_queryable_forms;
+ std::string autocheckout_url_prefix = GetAutocheckoutURLPrefix();
for (std::vector<FormData>::const_iterator iter = forms.begin();
iter != forms.end(); ++iter) {
- scoped_ptr<FormStructure> form_structure(new FormStructure(*iter));
+ scoped_ptr<FormStructure> form_structure(
+ new FormStructure(*iter, autocheckout_url_prefix));
if (!form_structure->ShouldBeParsed(false))
continue;

Powered by Google App Engine
This is Rietveld 408576698