Initial RemoteCommandsInvalidator

chrome/browser/chromeos/policy/remote_commands/affiliated_remote_commands_invalidator.h

+// Copyright 2015 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_REMOTE_COMMANDS_AFFILIATED_REMOTE_COMMANDS_INVALIDATOR_H_
+#define CHROME_BROWSER_CHROMEOS_POLICY_REMOTE_COMMANDS_AFFILIATED_REMOTE_COMMANDS_INVALIDATOR_H_
+
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+#include "chrome/browser/chromeos/policy/affiliated_invalidation_service_provider.h"

[bartfab (slow), 2015/05/15 15:55:08]

Nit: Forward-declare RemoteCommandsInvalidatorImpl instead and move the #include
to the .cc file.

[binjin, 2015/05/15 16:12:40]

Done.

+#include "chrome/browser/policy/cloud/remote_commands_invalidator_impl.h"
+
+namespace invalidation {
+class InvalidationService;

[bartfab (slow), 2015/05/15 15:55:08]

Nit: Not needed (only used in the declaration of a forward-declared method).

[binjin, 2015/05/15 16:12:40]

Done.

+}
+
+namespace policy {
+
+class CloudPolicyCore;
+
+class AffiliatedRemoteCommandsInvalidator

[bartfab (slow), 2015/05/15 15:55:08]

Nit: Document that this is a wrapper to be used for device commands and
device-local account commands.

[binjin, 2015/05/15 16:12:40]

Done.

+    : public AffiliatedInvalidationServiceProvider::Consumer {
+ public:
+  AffiliatedRemoteCommandsInvalidator(
+      CloudPolicyCore* core,
+      AffiliatedInvalidationServiceProvider* invalidation_service_provider);
+  ~AffiliatedRemoteCommandsInvalidator() override;
+
+  // AffiliatedInvalidationServiceProvider::Consumer:
+  void OnInvalidationServiceSet(
+      invalidation::InvalidationService* invalidation_service) override;
+
+ private:
+  CloudPolicyCore* const core_;
+  AffiliatedInvalidationServiceProvider* const invalidation_service_provider_;
+
+  scoped_ptr<RemoteCommandsInvalidatorImpl> invalidator_;
+
+  DISALLOW_COPY_AND_ASSIGN(AffiliatedRemoteCommandsInvalidator);
+};
+
+}  // namespace policy
+
+#endif  // CHROME_BROWSER_CHROMEOS_POLICY_REMOTE_COMMANDS_AFFILIATED_REMOTE_COMMANDS_INVALIDATOR_H_