Moved WebDataService to ProfileKeyedService

chrome/browser/webdata/web_data_service_factory.cc

Index: chrome/browser/webdata/web_data_service_factory.cc
diff --git a/chrome/browser/webdata/web_data_service_factory.cc b/chrome/browser/webdata/web_data_service_factory.cc
new file mode 100644
index 0000000000000000000000000000000000000000..be142031dc862e49eddb6eaf4ffd224f30d605bd
--- /dev/null
+++ b/chrome/browser/webdata/web_data_service_factory.cc
@@ -0,0 +1,64 @@
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/webdata/web_data_service_factory.h"
+
+#include "base/file_path.h"
+#include "chrome/browser/profiles/profile_dependency_manager.h"
+#include "chrome/browser/webdata/web_data_service.h"
+#include "chrome/common/chrome_constants.h"
+
+WebDataServiceFactory::WebDataServiceFactory()
+    : RefcountedProfileKeyedServiceFactory(
+          "WebDataService",
+          ProfileDependencyManager::GetInstance()) {
+  // WebDataServiceFactory has no dependecies.
+}
+
+WebDataServiceFactory::~WebDataServiceFactory() {}
+
+// static
+scoped_refptr<WebDataService> WebDataServiceFactory::GetForProfile(
+    Profile* profile, Profile::ServiceAccessType access_type) {
+  if (access_type == Profile::IMPLICIT_ACCESS && profile->IsOffTheRecord()) {
+    NOTREACHED() << "This profile is OffTheRecord";

[Ilya Sherman, 2012/03/27 01:01:17]

nit: Please remove the logged comment, as it is clearly deducible from the code,
but if included as a string adds to the binary size.

[GeorgeY, 2012/03/30 19:20:12]

Done.

+    return NULL;
+  }
+  return static_cast<WebDataService*>(
+      GetInstance()->GetServiceForProfile(profile, true).get());
+}
+
+// static
+scoped_refptr<WebDataService> WebDataServiceFactory::GetForProfileIfExists(
+    Profile* profile, Profile::ServiceAccessType access_type) {
+  if (access_type == Profile::IMPLICIT_ACCESS && profile->IsOffTheRecord()) {
+    NOTREACHED() << "This profile is OffTheRecord";

[Ilya Sherman, 2012/03/27 01:01:17]

nit: Ditto.

[GeorgeY, 2012/03/30 19:20:12]

Done.

+    return NULL;
+  }
+  return static_cast<WebDataService*>(
+      GetInstance()->GetServiceForProfile(profile, false).get());
+}
+
+// static
+WebDataServiceFactory* WebDataServiceFactory::GetInstance() {
+  return Singleton<WebDataServiceFactory>::get();
+}
+
+bool WebDataServiceFactory::ServiceRedirectedInIncognito() {
+  return false;
+}
+
+scoped_refptr<RefcountedProfileKeyedService>
+WebDataServiceFactory::BuildServiceInstanceFor(Profile* profile) const {
+  DCHECK(profile);
+
+  FilePath path = profile->GetPath();
+  path = path.Append(chrome::kWebDataFilename);
+
+  scoped_refptr<WebDataService> wds(new WebDataService());
+  if (!wds->Init(profile->GetPath())) {
+    NOTREACHED() << "failed to initialize WebService";

[Ilya Sherman, 2012/03/27 01:01:17]

nit: Ditto.

[GeorgeY, 2012/03/30 19:20:12]

Done.

+  }
+  return wds.get();
+}