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

Unified Diff: chrome/renderer/autofill/form_autofill_util.cc

Issue 11000016: Move forms/ out of webkit/. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Response to review Created 8 years, 2 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/renderer/autofill/form_autofill_util.cc
diff --git a/chrome/renderer/autofill/form_autofill_util.cc b/chrome/renderer/autofill/form_autofill_util.cc
index 105c89dec33683700874f1617ebe65f87acd9fd7..51bf8e39381286e967badac66d5e70cce2310dcc 100644
--- a/chrome/renderer/autofill/form_autofill_util.cc
+++ b/chrome/renderer/autofill/form_autofill_util.cc
@@ -10,6 +10,8 @@
#include "base/memory/scoped_vector.h"
#include "base/string_util.h"
#include "base/utf_string_conversions.h"
+#include "chrome/common/form_data.h"
+#include "chrome/common/form_field.h"
#include "third_party/WebKit/Source/WebKit/chromium/public/WebDocument.h"
#include "third_party/WebKit/Source/WebKit/chromium/public/WebElement.h"
#include "third_party/WebKit/Source/WebKit/chromium/public/WebFormControlElement.h"
@@ -23,9 +25,9 @@
#include "third_party/WebKit/Source/WebKit/chromium/public/WebSelectElement.h"
#include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebString.h"
#include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebVector.h"
-#include "webkit/forms/form_data.h"
-#include "webkit/forms/form_field.h"
+using chrome::FormData;
+using chrome::FormField;
using WebKit::WebElement;
using WebKit::WebFormControlElement;
using WebKit::WebFormElement;
@@ -38,8 +40,6 @@ using WebKit::WebOptionElement;
using WebKit::WebSelectElement;
using WebKit::WebString;
using WebKit::WebVector;
-using webkit::forms::FormData;
-using webkit::forms::FormField;
namespace {
@@ -416,7 +416,7 @@ void GetOptionStringsFromElement(const WebSelectElement& select_element,
// The callback type used by |ForEachMatchingFormField()|.
typedef void (*Callback)(WebKit::WebFormControlElement*,
- const webkit::forms::FormField*,
+ const chrome::FormField*,
bool);
// For each autofillable field in |data| that matches a field in the |form|,
@@ -478,7 +478,7 @@ void ForEachMatchingFormField(const WebFormElement& form_element,
// Sets the |field|'s value to the value in |data|.
// Also sets the "autofilled" attribute, causing the background to be yellow.
void FillFormField(WebKit::WebFormControlElement* field,
- const webkit::forms::FormField* data,
+ const chrome::FormField* data,
bool is_initiating_node) {
// Nothing to fill.
if (data->value.empty())
@@ -508,7 +508,7 @@ void FillFormField(WebKit::WebFormControlElement* field,
// Sets the |field|'s "suggested" (non JS visible) value to the value in |data|.
// Also sets the "autofilled" attribute, causing the background to be yellow.
void PreviewFormField(WebKit::WebFormControlElement* field,
- const webkit::forms::FormField* data,
+ const chrome::FormField* data,
bool is_initiating_node) {
// Nothing to preview.
if (data->value.empty())
@@ -663,8 +663,8 @@ bool WebFormElementToFormData(
const WebKit::WebFormControlElement& form_control_element,
RequirementsMask requirements,
ExtractMask extract_mask,
- webkit::forms::FormData* form,
- webkit::forms::FormField* field) {
+ chrome::FormData* form,
+ chrome::FormField* field) {
const WebFrame* frame = form_element.document().frame();
if (!frame)
return false;
@@ -792,7 +792,7 @@ bool WebFormElementToFormData(
bool FindFormAndFieldForInputElement(const WebInputElement& element,
FormData* form,
- webkit::forms::FormField* field,
+ chrome::FormField* field,
RequirementsMask requirements) {
if (!IsAutofillableElement(element))
return false;

Powered by Google App Engine
This is Rietveld 408576698