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

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

Issue 6286003: Refactoring of upload, form saving, and UMA logging. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move upload. Created 9 years, 11 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 | chrome/browser/autofill/autofill_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/autofill/autofill_manager.h
diff --git a/chrome/browser/autofill/autofill_manager.h b/chrome/browser/autofill/autofill_manager.h
index 76ed0d7f388be11ed68215f9642d1acddba778f2..bbf219eac3f64250bcfa9cdb6fa9d34f780620fe 100644
--- a/chrome/browser/autofill/autofill_manager.h
+++ b/chrome/browser/autofill/autofill_manager.h
@@ -67,11 +67,11 @@ class AutoFillManager : public IPC::Channel::Listener,
// Returns the value of the AutoFillEnabled pref.
virtual bool IsAutoFillEnabled() const;
- // Handles the form data submitted by the user.
- void HandleSubmit();
+ // Imports the form data, submitted by the user, into |personal_data_|.
+ void ImportFormData(const FormStructure& submitted_form);
// Uploads the form data to the AutoFill server.
- void UploadFormData();
+ void UploadFormData(const FormStructure& submitted_form);
// Reset cache.
void Reset();
@@ -173,9 +173,11 @@ class AutoFillManager : public IPC::Channel::Listener,
void ParseForms(const std::vector<webkit_glue::FormData>& forms);
// Uses existing personal data to determine possible field types for the
- // |upload_form_structure_|.
- void DeterminePossibleFieldTypesForUpload(
- const FormStructure* cached_upload_form_structure);
+ // |submitted_form|.
+ void DeterminePossibleFieldTypesForUpload(FormStructure* submitted_form);
+
+ void LogMetricsAboutSubmittedForm(const webkit_glue::FormData& form,
+ const FormStructure* submitted_form);
// The TabContents hosting this AutoFillManager.
// Weak reference.
@@ -204,9 +206,6 @@ class AutoFillManager : public IPC::Channel::Listener,
// Our copy of the form data.
ScopedVector<FormStructure> form_structures_;
- // The form data the user has submitted.
- scoped_ptr<FormStructure> upload_form_structure_;
-
// The InfoBar that asks for permission to store credit card information.
// Deletes itself when closed.
AutoFillCCInfoBarDelegate* cc_infobar_;
« no previous file with comments | « no previous file | chrome/browser/autofill/autofill_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698