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

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

Issue 8680040: Group forms-related files in webkit/glue in a forms/ subdirectory. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase + another build fix Created 9 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/autofill_external_delegate.h
diff --git a/chrome/browser/autofill/autofill_external_delegate.h b/chrome/browser/autofill/autofill_external_delegate.h
index c9bc4dcabc6d180a989829974d728ff09ff337ca..48905653772c9ed9fb849e39cfda4f7922a24af3 100644
--- a/chrome/browser/autofill/autofill_external_delegate.h
+++ b/chrome/browser/autofill/autofill_external_delegate.h
@@ -9,7 +9,7 @@
#include <vector>
#include "base/string16.h"
-#include "webkit/glue/form_field.h"
+#include "webkit/forms/form_field.h"
class AutofillManager;
class TabContentsWrapper;
@@ -18,9 +18,11 @@ namespace gfx {
class Rect;
}
-namespace webkit_glue {
+namespace webkit {
+namespace forms {
struct FormData;
-} // namespace webkit_glue
+}
+}
// TODO(csharp): A lot of the logic in this class is copied from autofillagent.
// Once Autofill is moved out of WebKit this class should be the only home for
@@ -45,8 +47,8 @@ class AutofillExternalDelegate {
// Autocomplete because they have their own popup, and showing our popup
// on to of theirs would be a poor user experience.
virtual void OnQuery(int query_id,
- const webkit_glue::FormData& form,
- const webkit_glue::FormField& field,
+ const webkit::forms::FormData& form,
+ const webkit::forms::FormField& field,
const gfx::Rect& bounds,
bool display_warning_if_disabled);
@@ -87,9 +89,10 @@ class AutofillExternalDelegate {
int separator_index) = 0;
// Handle instance specific OnQueryCode.
- virtual void OnQueryPlatformSpecific(int query_id,
- const webkit_glue::FormData& form,
- const webkit_glue::FormField& field) = 0;
+ virtual void OnQueryPlatformSpecific(
+ int query_id,
+ const webkit::forms::FormData& form,
+ const webkit::forms::FormField& field) = 0;
private:
TabContentsWrapper* tab_contents_wrapper_; // weak; owns me.
@@ -99,7 +102,7 @@ class AutofillExternalDelegate {
int autofill_query_id_;
// The current field selected by Autofill.
- webkit_glue::FormField autofill_query_field_;
+ webkit::forms::FormField autofill_query_field_;
// Should we display a warning if Autofill is disabled?
bool display_warning_if_disabled_;
« no previous file with comments | « chrome/browser/autofill/autofill_download_unittest.cc ('k') | chrome/browser/autofill/autofill_external_delegate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698