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

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

Issue 10919066: Use BrowserContext as key in API. Switch Autofill to use BC in place of Profile. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 3 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/autofill_manager.cc
diff --git a/chrome/browser/autofill/autofill_manager.cc b/chrome/browser/autofill/autofill_manager.cc
index 4201b813add0260c6f13e19e14bef68a858f4cb2..9dc3b99f6837eb1c778b27f4a98566fa2275ac67 100644
--- a/chrome/browser/autofill/autofill_manager.cc
+++ b/chrome/browser/autofill/autofill_manager.cc
@@ -46,6 +46,7 @@
#include "chrome/common/chrome_switches.h"
#include "chrome/common/pref_names.h"
#include "chrome/common/url_constants.h"
+#include "content/public/browser/browser_context.h"
#include "content/public/browser/browser_thread.h"
#include "content/public/browser/notification_service.h"
#include "content/public/browser/notification_source.h"
@@ -179,7 +180,7 @@ AutofillManager::AutofillManager(autofill::AutofillManagerDelegate* delegate,
manager_delegate_(delegate),
tab_contents_(tab_contents),
personal_data_(NULL),
- download_manager_(tab_contents->profile(), this),
+ download_manager_(delegate->GetBrowserContext(), this),
disable_download_manager_requests_(false),
metric_logger_(new AutofillMetrics),
has_logged_autofill_enabled_(false),
@@ -192,7 +193,7 @@ AutofillManager::AutofillManager(autofill::AutofillManagerDelegate* delegate,
external_delegate_(NULL) {
// |personal_data_| is NULL when using test-enabled WebContents.
personal_data_ = PersonalDataManagerFactory::GetForProfile(
- tab_contents->profile()->GetOriginalProfile());
+ tab_contents_->profile());
erikwright (departed) 2012/09/04 19:42:14 Presumably you have determined that this change fr
Jói 2012/09/04 20:10:25 No, this was a mistake. Thanks for noticing.
RegisterWithSyncService();
registrar_.Init(manager_delegate_->GetPrefs());
registrar_.Add(prefs::kPasswordGenerationEnabled, this);

Powered by Google App Engine
This is Rietveld 408576698