service worker scaffolding

content/browser/service_worker/service_worker_context_wrapper.cc

+// Copyright 2013 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.
+
+#include "content/browser/service_worker/service_worker_context_wrapper.h"
+
+#include "base/files/file_path.h"
+#include "content/browser/service_worker/service_worker_context_core.h"
+#include "content/public/browser/browser_thread.h"
+#include "webkit/browser/quota/quota_manager.h"
+
+namespace content {
+
+ServiceWorkerContextWrapper::ServiceWorkerContextWrapper() {
+}
+
+ServiceWorkerContextWrapper::~ServiceWorkerContextWrapper() {
+}
+
+void ServiceWorkerContextWrapper::Init(
+    const base::FilePath& user_data_directory,
+    quota::QuotaManagerProxy* quota_manager_proxy) {
+  if (!BrowserThread::CurrentlyOn(BrowserThread::IO)) {

[alecflett, 2013/11/11 18:59:03]

Since, as it exists today, it will always be called on the IO thread, it seems
like it would be a bug to make it created on another thread?

I'm starting to wonder what "other" thread is ever going to be calling Init()?
Can we DCHECK that this called on the right thread for now, and then if code
evolves to create the this on another thread, we add the if() ?

[michaeln, 2013/11/11 20:56:38]

But the wrapper is created by StoragePartitionImpl::Create which happens on the
UI thread. Also fyi, ServiceWorkerDispatcherHost instances are created on the UI
thread too.

+    BrowserThread::PostTask(
+        BrowserThread::IO, FROM_HERE,
+        base::Bind(&ServiceWorkerContextWrapper::Init, this,
+                   user_data_directory, quota_manager_proxy));
+    return;
+  }
+  context_core_.reset(
+      new ServiceWorkerContextCore(
+          user_data_directory, quota_manager_proxy));
+}
+
+
+void ServiceWorkerContextWrapper::Shutdown() {
+  if (!BrowserThread::CurrentlyOn(BrowserThread::IO)) {
+    BrowserThread::PostTask(
+        BrowserThread::IO, FROM_HERE,
+        base::Bind(&ServiceWorkerContextWrapper::Shutdown, this));
+    return;
+  }
+  context_core_.reset();
+}
+
+ServiceWorkerContextCore* ServiceWorkerContextWrapper::context() {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
+  return context_core_.get();
+}
+
+}  // namespace content