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

Unified Diff: chrome/browser/autofill/risk/fingerprint.cc

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/autofill/risk/fingerprint.cc
diff --git a/chrome/browser/autofill/risk/fingerprint.cc b/chrome/browser/autofill/risk/fingerprint.cc
index ef7b2ad5ce460fa9422714ed70bb96749b296d82..f979bc9a6e58c8f2ea72fa9c14cb43b77c50ee61 100644
--- a/chrome/browser/autofill/risk/fingerprint.cc
+++ b/chrome/browser/autofill/risk/fingerprint.cc
@@ -367,16 +367,6 @@ void GetFingerprint(
const WebKit::WebScreenInfo& screen_info,
const PrefServiceBase& prefs,
const base::Callback<void(scoped_ptr<Fingerprint>)>& callback) {
- // TODO(isherman): Add a DCHECK that the ToS have been accepted prior to
- // calling into this method. Also, ensure that the UI contains a clear
- // indication to the user as to what data will be collected. Until then, this
- // code should not be called.
-
- // TODO(isherman): In order to actually be able to pass in the WebScreenInfo
- // that's used here, we'll need to expose RenderWidgetHostImpl's
- // GetWebScreenInfo() as part of the public RenderWidgetHost interface.
- // We can then access it via the dialog's WebContents pointer.
-
// Begin loading all of the data that we need to load asynchronously.
// This class is responsible for freeing its own memory.
new FingerprintDataLoader(gaia_id, window_bounds, content_bounds,

Powered by Google App Engine
This is Rietveld 408576698