Create a content::ResourceContext.

content/browser/worker_host/worker_service.h

 // Copyright (c) 2011 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_BROWSER_WORKER_HOST_WORKER_SERVICE_H_
 #define CONTENT_BROWSER_WORKER_HOST_WORKER_SERVICE_H_
 #pragma once
 
 #include "base/basictypes.h"
 #include "base/memory/singleton.h"
+#include "base/threading/non_thread_safe.h"
 #include "content/browser/worker_host/worker_process_host.h"
 #include "googleurl/src/gurl.h"
 
+namespace content {
+class ResourceContext;
+}  // namespace content
 namespace net {
-class URLRequestContextGetter;
-}
-
+class URLRequestContext;
+}  // namespace net
 struct ViewHostMsg_CreateWorker_Params;
 
 // A singelton for managing HTML5 web workers.
-class WorkerService {
+class WorkerService : public base::NonThreadSafe {

[jam, 2011/04/13 16:43:06]

any reason to add this now?

[willchan no longer on Chromium, 2011/04/13 18:03:44]

Not a huge reason, but since I was reading this code for the first time, I had
no idea what the threading implications were. So everywhere I read, I had to
make sure that the calls were happening on the correct thread. I added this to
help other newbies to this code. Does it distract you? I don't necessarily need
it now, but it helped when I was reading the code the first time.

[jam, 2011/04/13 19:20:30]

When I see checking helpers like this that need to add hooks to check that it's
on the right thread, I ask myself if I'd like it if all of our code was done
like this (since eventually, if a minority likes to add it, it'll get added
everywhere :) ).  I don't think I'd want to see CHECK_ON_RIGHT_THREAD or
something similar on top of all our functions, hence my pushback.

[willchan no longer on Chromium, 2011/04/13 22:16:25]

I guess I don't mind having it in most public member functions. But you own this
code, so I'm happy to remove it. I'll take it out on my next patch.

  public:
   // Returns the WorkerService singleton.
   static WorkerService* GetInstance();
 
   // These methods correspond to worker related IPCs.
   void CreateWorker(const ViewHostMsg_CreateWorker_Params& params,
                     int route_id,
                     WorkerMessageFilter* filter,
-                    net::URLRequestContextGetter* request_context);
+                    net::URLRequestContextGetter* request_context_getter,
+                    const content::ResourceContext& resource_context);
   void LookupSharedWorker(const ViewHostMsg_CreateWorker_Params& params,
                           int route_id,
                           WorkerMessageFilter* filter,
                           bool incognito,
                           bool* exists,
                           bool* url_error);
   void CancelCreateDedicatedWorker(int route_id, WorkerMessageFilter* filter);
   void ForwardToWorker(const IPC::Message& message,
                        WorkerMessageFilter* filter);
   void DocumentDetached(unsigned long long document_id,
@@ skipping 74 matching lines @@
 
   // These are shared workers that have been looked up, but not created yet.
   // We need to keep a list of these to synchronously detect shared worker
   // URL mismatches when two pages launch shared workers simultaneously.
   WorkerProcessHost::Instances pending_shared_workers_;
 
   DISALLOW_COPY_AND_ASSIGN(WorkerService);
 };
 
 #endif  // CONTENT_BROWSER_WORKER_HOST_WORKER_SERVICE_H_