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

Unified Diff: components/autofill/browser/autocheckout_page_meta_data.h

Issue 15487004: Autocheckout: parse multiple clicks setting in autofill response. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Ilya's comments Created 7 years, 7 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
« no previous file with comments | « no previous file | components/autofill/browser/autofill_xml_parser.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/autofill/browser/autocheckout_page_meta_data.h
diff --git a/components/autofill/browser/autocheckout_page_meta_data.h b/components/autofill/browser/autocheckout_page_meta_data.h
index f1b9a7ad059200c5779071dbdedcc46ec2a9c06a..997a66e9f60be8624f7a81d2d5420d93f36a7b90 100644
--- a/components/autofill/browser/autocheckout_page_meta_data.h
+++ b/components/autofill/browser/autocheckout_page_meta_data.h
@@ -5,6 +5,8 @@
#ifndef COMPONENTS_AUTOFILL_BROWSER_AUTOCHECKOUT_PAGE_META_DATA_H_
#define COMPONENTS_AUTOFILL_BROWSER_AUTOCHECKOUT_PAGE_META_DATA_H_
+#include <vector>
+
#include "base/basictypes.h"
#include "components/autofill/common/web_element_descriptor.h"
@@ -36,8 +38,23 @@ struct AutocheckoutPageMetaData {
// belong to any autofill flow, it is set to -1.
int total_pages;
+ // A list of elements to click before filling form fields. Elements have to be
+ // clicked in order.
+ std::vector<WebElementDescriptor> click_elements_before_form_fill;
+
+ // A list of elements to click after filling form fields, and before clicking
+ // page_advance_button. Elements have to be clicked in order.
+ std::vector<WebElementDescriptor> click_elements_after_form_fill;
+
// The proceed element of the multipage Autofill flow. It can be empty
// if current page is the last page of a flow or isn't a member of a flow.
+ //
+ // We do expect page navigation when click on |proceed_element_descriptor|,
+ // and report an error if it doesn't. Oppositely, we do not expect page
+ // navigation when click elements in |click_elements_before_form_fill| and
+ // |click_elements_after_form_fill|. Because of this behavior difference and
+ // |proceed_element_descriptor| is optional, we separate it from
+ // |click_elements_after_form_fill|.
WebElementDescriptor proceed_element_descriptor;
private:
« no previous file with comments | « no previous file | components/autofill/browser/autofill_xml_parser.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698