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

Unified Diff: components/autofill/core/browser/autofill_manager.cc

Issue 16611003: Ignore ajax on specified pages. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: yay for scoped pointers Created 7 years, 6 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: components/autofill/core/browser/autofill_manager.cc
diff --git a/components/autofill/core/browser/autofill_manager.cc b/components/autofill/core/browser/autofill_manager.cc
index 4cbc8afa86a6ed88d1ba6e88b50af66b438d1c9c..4da72e1ce3bab65c2c112853f73e1c7ec34e5291 100644
--- a/components/autofill/core/browser/autofill_manager.cc
+++ b/components/autofill/core/browser/autofill_manager.cc
@@ -332,9 +332,14 @@ void AutofillManager::OnFormsSeen(const std::vector<FormData>& forms,
autofill::FormsSeenState state) {
bool is_post_document_load = state == autofill::DYNAMIC_FORMS_SEEN;
bool has_more_forms = state == autofill::PARTIAL_FORMS_SEEN;
- // If new forms were added via AJAX or DHML, treat as new page.
- if (is_post_document_load)
+ // If new forms were added dynamically, and the autocheckout manager
+ // doesn't tell us to ignore ajax on this page, treat as a new page.
+ if (is_post_document_load) {
+ if (autocheckout_manager_.ShouldIgnoreAjax())
+ return;
+
Reset();
+ }
RenderViewHost* host = driver_->GetWebContents()->GetRenderViewHost();
if (!host)

Powered by Google App Engine
This is Rietveld 408576698