Populate .current when navigator.serviceWorker is accessed

content/child/service_worker/service_worker_provider_context.h

Index: content/child/service_worker/service_worker_provider_context.h
diff --git a/content/child/service_worker/service_worker_provider_context.h b/content/child/service_worker/service_worker_provider_context.h
new file mode 100644
index 0000000000000000000000000000000000000000..a45c49ce2f2f92beb15fa41129df448a11332420
--- /dev/null
+++ b/content/child/service_worker/service_worker_provider_context.h
@@ -0,0 +1,70 @@
+// Copyright 2014 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_SERVICE_WORKER_SERVICE_WORKER_PROVIDER_CONTEXT_H_
+#define CONTENT_CHILD_SERVICE_WORKER_SERVICE_WORKER_PROVIDER_CONTEXT_H_
+
+#include <set>
+#include <vector>
+
+#include "base/memory/ref_counted.h"
+#include "base/sequenced_task_runner_helpers.h"
+#include "base/synchronization/lock.h"
+#include "content/common/service_worker/service_worker_types.h"
+
+namespace base {
+class MessageLoopProxy;
+}
+
+namespace IPC {
+class Message;
+}
+
+namespace content {
+
+class ServiceWorkerHandleReference;
+struct ServiceWorkerProviderContextDeleter;
+class ThreadSafeSender;
+
+// An instance of this class holds document-related information (e.g.
+// .current). Created and destructed on the main thread.
+// TODO(kinuko): To support navigator.serviceWorker in dedicated workers
+// this needs to be RefCountedThreadSafe and .current info needs to be
+// handled in a thread-safe manner (e.g. by a lock etc).
+class ServiceWorkerProviderContext
+    : public base::RefCounted<ServiceWorkerProviderContext> {
+ public:
+  explicit ServiceWorkerProviderContext(int provider_id);
+
+  // Gets the initial provider context information (e.g. to populate
+  // .current).
+  void GetInitializationInfo(
+      scoped_ptr<ServiceWorkerHandleReference>* current);

[michaeln, 2014/05/05 23:18:03]

would it make sense to have this method return the scoped_ptr<>

[kinuko, 2014/05/06 00:39:37]

If we started to have more fields (e.g. pending?) we'll probably want to return
them together, but for now yes we can.

+
+  // Called from ServiceWorkerDispatcher.
+  void OnServiceWorkerStateChanged(int thread_id,

[michaeln, 2014/05/05 23:18:03]

Does thread_id make sense here now?

If i understand the model now, there is one ServiceWorkerProviderContext for a
document and all of its dedicated workers. The browser process informs that
'context' of state transitions and then that context is responsible for
propagating the transition info further. If that's right, not sure thread_id is
needed here.

[kinuko, 2014/05/06 00:39:37]

It's not needed, my very early patch used to have code related to this but not
anymore.  I dropped it.

+                                   int handle_id,
+                                   blink::WebServiceWorkerState state);
+  void OnSetCurrentServiceWorker(int thread_id,
+                                 int provider_id,
+                                 const ServiceWorkerObjectInfo& info);
+
+  int provider_id() const { return provider_id_; }
+  int current_handle_id() const;
+
+ private:
+  friend class base::RefCounted<ServiceWorkerProviderContext>;
+  ~ServiceWorkerProviderContext();
+
+  const int provider_id_;
+  scoped_refptr<base::MessageLoopProxy> main_thread_loop_proxy_;
+  scoped_refptr<ThreadSafeSender> thread_safe_sender_;
+  scoped_ptr<ServiceWorkerHandleReference> current_;
+
+  DISALLOW_COPY_AND_ASSIGN(ServiceWorkerProviderContext);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_CHILD_SERVICE_WORKER_SERVICE_WORKER_PROVIDER_CONTEXT_H_