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

Unified Diff: content/browser/service_worker/service_worker_test_utils.h

Issue 2779763004: Create ServiceWorkerProviderHost before starting worker (Closed)
Patch Set: Pass the param of BindWithProviderInfo by value instead of pointer Created 3 years, 6 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/browser/service_worker/service_worker_test_utils.h
diff --git a/content/browser/service_worker/service_worker_test_utils.h b/content/browser/service_worker/service_worker_test_utils.h
index 2c9a08681db536de76f22171a7c6ca1760d60cee..dc7217c4e9167e7ae570a67d54f9b305067b2f52 100644
--- a/content/browser/service_worker/service_worker_test_utils.h
+++ b/content/browser/service_worker/service_worker_test_utils.h
@@ -11,7 +11,7 @@
#include "base/callback.h"
#include "base/command_line.h"
#include "base/memory/weak_ptr.h"
-#include "content/common/service_worker/service_worker_provider_interfaces.mojom.h"
+#include "content/common/service_worker/service_worker_provider.mojom.h"
#include "content/public/browser/browser_thread.h"
#include "content/public/common/content_switches.h"
#include "testing/gtest/include/gtest/gtest.h"
@@ -21,6 +21,7 @@ namespace content {
class ServiceWorkerContextCore;
class ServiceWorkerDispatcherHost;
class ServiceWorkerProviderHost;
+class ServiceWorkerVersion;
struct ServiceWorkerProviderHostInfo;
template <typename Arg>
@@ -58,6 +59,8 @@ class ServiceWorkerRemoteProviderEndpoint {
~ServiceWorkerRemoteProviderEndpoint();
void BindWithProviderHostInfo(ServiceWorkerProviderHostInfo* info);
+ void BindWithProviderInfo(
+ mojom::ServiceWorkerProviderInfoForStartWorkerPtr info);
mojom::ServiceWorkerProviderHostAssociatedPtr* host_ptr() {
return &host_ptr_;
@@ -88,8 +91,8 @@ std::unique_ptr<ServiceWorkerProviderHost> CreateProviderHostForWindow(
std::unique_ptr<ServiceWorkerProviderHost>
CreateProviderHostForServiceWorkerContext(
int process_id,
- int provider_id,
bool is_parent_frame_secure,
+ ServiceWorkerVersion* hosted_version,
base::WeakPtr<ServiceWorkerContextCore> context,
ServiceWorkerRemoteProviderEndpoint* output_endpoint);

Powered by Google App Engine
This is Rietveld 408576698