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

Unified Diff: chrome/browser/autofill/form_structure.h

Issue 11539003: Pop up requestAutocomplete UI when autofill server hints chrome client that it is in a multipage au… (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Updated string. Pop the UI only on first page of the flow. Created 8 years 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/form_structure.h
diff --git a/chrome/browser/autofill/form_structure.h b/chrome/browser/autofill/form_structure.h
index 45cf2bbcc365e51c6ed7d834dd741a4181d81d3c..0f0f85e44afdec4634c7316d9e6e0f034600fa24 100644
--- a/chrome/browser/autofill/form_structure.h
+++ b/chrome/browser/autofill/form_structure.h
@@ -143,6 +143,14 @@ class FormStructure {
UploadRequired upload_required() const { return upload_required_; }
+ bool is_start_autofillable_flow() const { return page_no_ == 1; }
Ilya Sherman 2012/12/13 02:29:23 nit: This method is not just doing a simple variab
Raman Kakilate 2012/12/13 21:34:34 Done.
+
+ bool is_in_autofillable_flow() const { return page_no_ < total_pages_; }
+
+ void set_page_no(int page_no) { page_no_ = page_no; }
+
+ void set_total_pages(int total_pages) { total_pages_ = total_pages; }
+
virtual std::string server_experiment_id() const;
// Returns a FormData containing the data this form structure knows about.
@@ -208,6 +216,14 @@ class FormStructure {
// GET or POST.
RequestMethod method_;
+ // Page number of the autofill flow, this form belongs to. If this form
+ // doesn't belong to any autofill flow, it is set to 0.
+ int page_no_;
+
+ // Total number of pages in the autofill flow. If this form doesn't belong
+ // to any autofill flow, it is set to 0.
+ int total_pages_;
+
// Whether the form includes any field types explicitly specified by the site
// author, via the |autocompletetype| attribute.
bool has_author_specified_types_;

Powered by Google App Engine
This is Rietveld 408576698