Make ShortcutsBackend a ProfileKeyedService.

chrome/browser/history/shortcuts_backend_factory.h

+// 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.
+
+#ifndef CHROME_BROWSER_HISTORY_SHORTCUTS_BACKEND_FACTORY_H_
+#define CHROME_BROWSER_HISTORY_SHORTCUTS_BACKEND_FACTORY_H_
+#pragma once
+
+#include "base/memory/ref_counted.h"
+#include "base/memory/singleton.h"
+#include "chrome/browser/profiles/refcounted_profile_keyed_service_factory.h"
+
+class Profile;
+
+namespace history {
+class ShortcutsBackend;
+}  // namespace history
+
+// Singleton that owns all instances of ShortcutsBackend and associates them
+// with Profiles.
+class ShortcutsBackendFactory : public RefcountedProfileKeyedServiceFactory {
+ public:
+  static scoped_refptr<history::ShortcutsBackend> GetForProfile(
+      Profile* profile);
+
+  static scoped_refptr<history::ShortcutsBackend> GetForProfileIfExists(
+      Profile* profile);
+
+  static ShortcutsBackendFactory* GetInstance();
+
+  // Creates and returns a backend for testing purposes.
+  static scoped_refptr<RefcountedProfileKeyedService>
+      GetForProfileForTesting(Profile* profile);

[rpetterson, 2012/06/29 20:57:06]

I think a better name would be BuildProfileForTesting since it mimics the
BuildServiceInstanceFor function rather than the GetForProfile function.

[mrossetti, 2012/07/02 18:31:39]

Done.

+
+  // Creates and returns a backend but without creating its persistant database
+  // for testing purposes.
+  static scoped_refptr<RefcountedProfileKeyedService>
+      GetForProfileNoDatabaseForTesting(Profile* profile);

[rpetterson, 2012/06/29 20:57:06]

Same as above, BuildProfileNoDatabaseForTesting.

[mrossetti, 2012/07/02 18:31:39]

Done.

+
+ private:
+  friend struct DefaultSingletonTraits<ShortcutsBackendFactory>;
+
+  ShortcutsBackendFactory();
+  virtual ~ShortcutsBackendFactory();
+
+  // ProfileKeyedServiceFactory:
+  virtual scoped_refptr<RefcountedProfileKeyedService> BuildServiceInstanceFor(
+      Profile* profile) const OVERRIDE;
+  virtual bool ServiceIsNULLWhileTesting() OVERRIDE;
+};
+
+#endif  // CHROME_BROWSER_HISTORY_SHORTCUTS_BACKEND_FACTORY_H_