[Background Sync] Use Mojo IPC to fire background sync events

content/renderer/background_sync/background_sync_client_impl.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 CONTENT_RENDERER_BACKGROUND_SYNC_BACKGROUND_SYNC_CLIENT_IMPL_H_
+#define CONTENT_RENDERER_BACKGROUND_SYNC_BACKGROUND_SYNC_CLIENT_IMPL_H_
+
+#include "base/memory/ref_counted.h"

[jkarlin, 2015/07/20 17:17:26]

needed?

[iclelland, 2015/07/20 18:05:21]

It provided two macros: NON_EXPORTED_BASE and DISALLOW_COPY_AND_ASSIGN. I've
replaced it with the direct includes.
Done.

+#include "content/common/background_sync_service.mojom.h"
+#include "content/common/content_export.h"
+#include "third_party/mojo/src/mojo/public/cpp/bindings/strong_binding.h"
+
+namespace content {
+
+class CONTENT_EXPORT BackgroundSyncClientImpl
+    : public NON_EXPORTED_BASE(BackgroundSyncServiceClient) {
+ public:
+  static void Create(
+      mojo::InterfaceRequest<BackgroundSyncServiceClient> request);
+
+  ~BackgroundSyncClientImpl() override;
+
+ private:
+  using SyncCallback = mojo::Callback<void(ServiceWorkerEventStatus)>;
+  explicit BackgroundSyncClientImpl(
+      mojo::InterfaceRequest<BackgroundSyncServiceClient> request);
+
+  // BackgroundSyncClient methods:

[jkarlin, 2015/07/20 17:17:25]

BackgruondSyncServiceClient

[iclelland, 2015/07/20 18:05:21]

I spent 5 minutes looking for the typo, before I saw the real difference :)
Done.

+  void Sync(content::SyncRegistrationPtr registration,
+            const SyncCallback& callback) override;
+
+  mojo::StrongBinding<BackgroundSyncServiceClient> binding_;
+
+  DISALLOW_COPY_AND_ASSIGN(BackgroundSyncClientImpl);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_RENDERER_BACKGROUND_SYNC_BACKGROUND_SYNC_CLIENT_IMPL_H_