Create a content::ResourceContext.

content/browser/worker_host/worker_service.h

Index: content/browser/worker_host/worker_service.h
diff --git a/content/browser/worker_host/worker_service.h b/content/browser/worker_host/worker_service.h
index 7869dd6d09a27e18f0c79b2ee232f320f1573e2c..9700f442b619636a64655e9d8aa65f683599d5b5 100644
--- a/content/browser/worker_host/worker_service.h
+++ b/content/browser/worker_host/worker_service.h
@@ -8,17 +8,20 @@
 
 #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();
@@ -27,7 +30,8 @@ class WorkerService {
   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,