Profile shouldn't own PersonalDataManager

chrome/browser/profiles/off_the_record_profile_impl.cc

Index: chrome/browser/profiles/off_the_record_profile_impl.cc
diff --git a/chrome/browser/profiles/off_the_record_profile_impl.cc b/chrome/browser/profiles/off_the_record_profile_impl.cc
index 49abd4df376c12cdb14d1dcf0e9c6c76557d9010..8e357dc70360f69a2e1ee247da25eda57275e9da 100644
--- a/chrome/browser/profiles/off_the_record_profile_impl.cc
+++ b/chrome/browser/profiles/off_the_record_profile_impl.cc
@@ -12,6 +12,7 @@
 #include "base/path_service.h"
 #include "base/string_util.h"
 #include "build/build_config.h"
+#include "chrome/browser/autofill/personal_data_manager_factory.h"

[Ilya Sherman, 2011/09/23 03:56:34]

nit: What relies on this #include?

[dhollowa, 2011/09/23 16:30:03]

Nothing.  Removed.

 #include "chrome/browser/background/background_contents_service_factory.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_plugin_service_filter.h"
@@ -338,10 +339,6 @@ bool OffTheRecordProfileImpl::HasCreatedDownloadManager() const {
   return download_manager_.get() != NULL;
 }
 
-PersonalDataManager* OffTheRecordProfileImpl::GetPersonalDataManager() {
-  return NULL;
-}
-
 fileapi::FileSystemContext* OffTheRecordProfileImpl::GetFileSystemContext() {
   CreateQuotaManagerAndClients();
   return file_system_context_.get();
@@ -670,10 +667,6 @@ class GuestSessionProfile : public OffTheRecordProfileImpl {
       : OffTheRecordProfileImpl(real_profile) {
   }
 
-  virtual PersonalDataManager* GetPersonalDataManager() {
-    return GetOriginalProfile()->GetPersonalDataManager();
-  }
-
   virtual void InitChromeOSPreferences() {
     chromeos_preferences_.reset(new chromeos::Preferences());
     chromeos_preferences_->Init(GetPrefs());