Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(377)

Unified Diff: content/child/background_sync/background_sync_provider_thread_proxy.h

Issue 1104283002: [Background Sync] Add renderer code to interface between JS API and back-end service (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@bgsync-mojo
Patch Set: Add comment re: BackgroundSyncProvider lifetime Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/child/background_sync/background_sync_provider_thread_proxy.h
diff --git a/content/child/background_sync/background_sync_provider_thread_proxy.h b/content/child/background_sync/background_sync_provider_thread_proxy.h
new file mode 100644
index 0000000000000000000000000000000000000000..6378401c4eb8ce6c77fe3446ab51b323076f0f8a
--- /dev/null
+++ b/content/child/background_sync/background_sync_provider_thread_proxy.h
@@ -0,0 +1,73 @@
+// 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_CHILD_BACKGROUND_SYNC_BACKGROUND_SYNC_PROVIDER_THREAD_PROXY_H_
+#define CONTENT_CHILD_BACKGROUND_SYNC_BACKGROUND_SYNC_PROVIDER_THREAD_PROXY_H_
+
+#include "base/macros.h"
+#include "base/memory/ref_counted.h"
+#include "content/child/worker_task_runner.h"
+#include "third_party/WebKit/public/platform/modules/background_sync/WebSyncProvider.h"
+
+namespace base {
+class SingleThreadTaskRunner;
+}
+
+namespace content {
+
+class BackgroundSyncProvider;
+
+// BackgroundSyncProviderThreadProxy is a a proxy to the BackgroundSyncProvider
jkarlin 2015/05/07 11:51:53 "is a a proxy" -> "is a proxy"
iclelland 2015/05/09 06:41:17 Done.
+// for callers running on a different thread than the main thread. There is one
+// instance of that class per thread.
+class BackgroundSyncProviderThreadProxy :
+ public blink::WebSyncProvider,
+ public WorkerTaskRunner::Observer {
+ public:
+ static BackgroundSyncProviderThreadProxy* GetThreadInstance(
+ base::SingleThreadTaskRunner* main_thread_task_runner,
+ BackgroundSyncProvider* permissions_dispatcher);
+
+ // blink::WebSyncProvider implementation
+ void registerBackgroundSync(
+ const blink::WebSyncRegistration* options,
+ blink::WebServiceWorkerRegistration* service_worker_registration,
+ blink::WebSyncRegistrationCallbacks* callbacks);
+ void unregisterBackgroundSync(
+ blink::WebSyncRegistration::Periodicity periodicity,
+ int64_t id, const blink::WebString& tag,
+ blink::WebServiceWorkerRegistration* service_worker_registration,
+ blink::WebSyncUnregistrationCallbacks* callbacks);
+ void getRegistration(
+ blink::WebSyncRegistration::Periodicity,
+ const blink::WebString& tag,
+ blink::WebServiceWorkerRegistration* service_worker_registration,
+ blink::WebSyncRegistrationCallbacks* callbacks);
+ void getRegistrations(
+ blink::WebSyncRegistration::Periodicity periodicity,
+ blink::WebServiceWorkerRegistration* service_worker_registration,
+ blink::WebSyncGetRegistrationsCallbacks* callbacks);
+
+ // WorkerTaskRunner::Observer implementation.
+ void OnWorkerRunLoopStopped() override;
+
+ private:
+ BackgroundSyncProviderThreadProxy(
+ base::SingleThreadTaskRunner* main_thread_task_runner,
+ BackgroundSyncProvider* sync_provider);
+
+ virtual ~BackgroundSyncProviderThreadProxy();
+
+ scoped_refptr<base::SingleThreadTaskRunner> main_thread_task_runner_;
+ // This belongs to the renderer main thread, (created by BlinkPlatformImpl)
+ // and so should outlive the BackgroundSyncProviderThreadProxy, which is
+ // created for a worker thread.
+ BackgroundSyncProvider* sync_provider_;
+
+ DISALLOW_COPY_AND_ASSIGN(BackgroundSyncProviderThreadProxy);
+};
+
+} // namespace content
+
+#endif // CONTENT_CHILD_BACKGROUND_SYNC_BACKGROUND_SYNC_PROVIDER_THREAD_PROXY_H_

Powered by Google App Engine
This is Rietveld 408576698