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

Unified Diff: content/common/service_worker/service_worker_provider_host_info.h

Issue 2653493009: Add two interfaces for ServiceWorkerProviderContext/ProviderHost (Closed)
Patch Set: Rebased/Fixed unittests when BrowserSideNavigation is enabled Created 3 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/common/service_worker/service_worker_provider_host_info.h
diff --git a/content/common/service_worker/service_worker_provider_host_info.h b/content/common/service_worker/service_worker_provider_host_info.h
index 106fb620ab3339552b1ea0a010d951d8a26106f6..8f6ca9b0665e8a235ebc8c7fabd4a7676261bc43 100644
--- a/content/common/service_worker/service_worker_provider_host_info.h
+++ b/content/common/service_worker/service_worker_provider_host_info.h
@@ -5,13 +5,19 @@
#ifndef CONTENT_COMMON_SERVICE_WORKER_SERVICE_WORKER_PROVIDER_HOST_INFO_H_
#define CONTENT_COMMON_SERVICE_WORKER_SERVICE_WORKER_PROVIDER_HOST_INFO_H_
+#include "content/common/service_worker/service_worker_provider_interfaces.mojom.h"
#include "content/common/service_worker/service_worker_types.h"
namespace content {
+// Container object to create a ServiceWorkerProviderHost.
struct CONTENT_EXPORT ServiceWorkerProviderHostInfo {
ServiceWorkerProviderHostInfo();
ServiceWorkerProviderHostInfo(ServiceWorkerProviderHostInfo&& other);
+ ServiceWorkerProviderHostInfo(
+ ServiceWorkerProviderHostInfo&& other,
+ mojom::ServiceWorkerProviderHostAssociatedRequest host_request,
+ mojom::ServiceWorkerProviderAssociatedPtrInfo client_ptr_info);
ServiceWorkerProviderHostInfo(int provider_id,
int route_id,
ServiceWorkerProviderType type,
@@ -44,6 +50,19 @@ struct CONTENT_EXPORT ServiceWorkerProviderHostInfo {
// is_parent_frame_secure| is true.
bool is_parent_frame_secure;
+ // Mojo endpoint to send a message from the renderer to the browser. This
+ // will be associated with ServiceWorkerDisptacherHost. |host_request| should
+ // be valid when ServiceWorkerProviderHostInfo is passed to any mojo methods.
falken 2017/05/18 04:13:10 Mojo
shimazu 2017/05/19 08:31:07 Done.
+ // After used to create the ServiceWorkerProviderHost, this will be invalid.
+ mojom::ServiceWorkerProviderHostAssociatedRequest host_request;
+
+ // Mojo endpoint to send a message from the browser to the renderer. This
+ // will be associated with ServiceWorkerDisptacherHost. |client_ptr_info|
+ // should be valid when ServiceWorkerProviderHostInfo is passed to any mojo
falken 2017/05/18 04:13:10 Mojo
shimazu 2017/05/19 08:31:07 Done.
+ // methods.
+ // After used to create the ServiceWorkerProviderHost, this will be invalid.
+ mojom::ServiceWorkerProviderAssociatedPtrInfo client_ptr_info;
+
private:
DISALLOW_COPY_AND_ASSIGN(ServiceWorkerProviderHostInfo);
};

Powered by Google App Engine
This is Rietveld 408576698