Added DriveNotificationManagerFactory so DriveNotificationManager is a singleton and can be called …

chrome/browser/google_apis/drive_notification_manager_factory.h

+// Copyright 2013 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_GOOGLE_APIS_DRIVE_NOTIFICATION_MANAGER_FACTORY_H_
+#define CHROME_BROWSER_GOOGLE_APIS_DRIVE_NOTIFICATION_MANAGER_FACTORY_H_
+
+#include "base/memory/singleton.h"
+#include "chrome/browser/profiles/profile_keyed_service_factory.h"

[Lei Zhang, 2013/04/11 02:49:58]

Isn't bug 146989 trying to get you to not do this?

[calvinlo, 2013/04/11 04:05:33]

I talked to kinaba and he's fine with adding this as a dependency exception in
the DEPS file. I already updated the file in my other patch so I'll wait for
that to hopefully land before coming back to this patch.

+
+namespace google_apis {
+
+class DriveNotificationManager;
+
+class DriveNotificationManagerFactory : public ProfileKeyedServiceFactory {

[satorux1, 2013/04/11 05:27:54]

Class comment is missing.

[calvinlo, 2013/04/11 05:41:15]

Done.

+ public:
+  static DriveNotificationManager* GetForProfile(Profile* profile);
+
+  static DriveNotificationManagerFactory* GetInstance();
+
+ private:
+  friend struct DefaultSingletonTraits<DriveNotificationManagerFactory>;
+
+  DriveNotificationManagerFactory();
+  virtual ~DriveNotificationManagerFactory();
+
+  // ProfileKeyedServiceFactory implementation.
+  virtual ProfileKeyedService* BuildServiceInstanceFor(
+      Profile* profile) const OVERRIDE;
+};
+
+}  // namespace google_apis
+
+#endif  // CHROME_BROWSER_GOOGLE_APIS_DRIVE_NOTIFICATION_MANAGER_FACTORY_H_