Move the notification part out of ConsumerEnrollmentHandler so that it can be reused for unenrollme…

chrome/browser/chromeos/policy/consumer_management_notifier_factory.h

+// Copyright 2014 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_CHROMEOS_POLICY_CONSUMER_MANAGEMENT_NOTIFIER_FACTORY_H_
+#define CHROME_BROWSER_CHROMEOS_POLICY_CONSUMER_MANAGEMENT_NOTIFIER_FACTORY_H_
+
+#include "base/macros.h"
+#include "base/memory/singleton.h"
+#include "components/keyed_service/content/browser_context_keyed_service_factory.h"
+
+namespace content {
+class BrowserContext;
+}
+
+namespace policy {
+
+class ConsumerManagementNotifier;
+
+class ConsumerManagementNotifierFactory
+    : public BrowserContextKeyedServiceFactory {
+ public:
+  static ConsumerManagementNotifier* GetForBrowserContext(
+      content::BrowserContext* context);
+
+  static ConsumerManagementNotifierFactory* GetInstance();
+
+ private:
+  friend struct DefaultSingletonTraits<ConsumerManagementNotifierFactory>;
+
+  ConsumerManagementNotifierFactory();
+  virtual ~ConsumerManagementNotifierFactory();

[bartfab (slow), 2014/11/27 14:09:09]

Nit: Use |override| instead of |virtual|.

[davidyu, 2014/11/28 02:45:26]

Done.

+
+  // BrowserContextKeyedServiceFactory:
+  KeyedService* BuildServiceInstanceFor(
+      content::BrowserContext* context) const override;
+  virtual bool ServiceIsCreatedWithBrowserContext() const override;

[bartfab (slow), 2014/11/27 14:09:09]

Nit: No |virtual| with |override|.

[davidyu, 2014/11/28 02:45:26]

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(ConsumerManagementNotifierFactory);
+};
+
+}  // namespace policy
+
+#endif  // CHROME_BROWSER_CHROMEOS_POLICY_CONSUMER_MANAGEMENT_NOTIFIER_FACTORY_H_