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

Unified Diff: chrome/browser/ui/autofill/autofill_dialog_controller_impl.h

Issue 12212057: [Autofill] Add ability to load Risk fingerprint data in AutofillDialogControllerImpl. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 10 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/browser/ui/autofill/autofill_dialog_controller_impl.h
diff --git a/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h b/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h
index 0d86fa939a3a8d89f3f0832ce6806cddac33e7d6..a85c644ffa2db3b6daaba27365e992b10b171db1 100644
--- a/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h
+++ b/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h
@@ -9,6 +9,7 @@
#include "base/callback.h"
#include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
#include "base/string16.h"
#include "chrome/browser/autofill/field_types.h"
#include "chrome/browser/autofill/form_structure.h"
@@ -39,6 +40,10 @@ namespace autofill {
class AutofillDialogView;
+namespace risk {
+class Fingerprint;
+}
+
// This class drives the dialog that appears when a site uses the imperative
// autocomplete API to fill out a form.
class AutofillDialogControllerImpl : public AutofillDialogController,
@@ -204,6 +209,13 @@ class AutofillDialogControllerImpl : public AutofillDialogController,
// Hides |popup_controller_|'s popup view, if it exists.
void HidePopup();
+ // Asynchronously loads fingerprint data to be sent to the Risk server, and
+ // calls OnDidLoadRiskFingerprintData() with the data once it's loaded.
Evan Stade 2013/02/07 01:34:53 this function does not actually call OnDidLoadRisk
Ilya Sherman 2013/02/07 01:58:29 Done.
+ void LoadRiskFingerprintData();
+
+ // Callback from LoadRiskFingerprintData().
+ void OnDidLoadRiskFingerprintData(scoped_ptr<risk::Fingerprint> fingerprint);
+
// The |profile| for |contents_|.
Profile* const profile_;
@@ -267,6 +279,8 @@ class AutofillDialogControllerImpl : public AutofillDialogController,
// A NotificationRegistrar for tracking the completion of sign-in.
content::NotificationRegistrar registrar_;
+ base::WeakPtrFactory<AutofillDialogControllerImpl> weak_ptr_factory_;
+
DISALLOW_COPY_AND_ASSIGN(AutofillDialogControllerImpl);
};

Powered by Google App Engine
This is Rietveld 408576698