Do cookie checks in NetworkDelegate instead of the URLRequest::Delegate.

webkit/appcache/appcache_service.h

Index: webkit/appcache/appcache_service.h
diff --git a/webkit/appcache/appcache_service.h b/webkit/appcache/appcache_service.h
index 7a2b57e8576ed57b782beda0cbf91ed738e033fa..2557fffcd27721a29f20f3943fcf8bf8700744b2 100644
--- a/webkit/appcache/appcache_service.h
+++ b/webkit/appcache/appcache_service.h
@@ -108,6 +108,10 @@ class APPCACHE_EXPORT AppCacheService {
     request_context_ = context;
   }
 
+  // Perform additional initialization steps on an URLRequest used for cache
+  // updates. Should only be accessed on the UI thread.

[michaeln, 2012/03/01 17:47:01]

please define this method on AppCachePolicy instead of on this class, and
include in it's signature an AppCacheUpdateJob* parameter (otherwise there is no
context with which to determine the values you want associated with the
request).

void AppCachePolicy::AddUserDataForUpdateRequest(job, req);

[jochen (gone - plz use gerrit), 2012/03/01 18:41:01]

Ideally, I want to attach the process_id/routing_id at this point. Can I derive
these values from the job somehow?

+  virtual void InitializeUpdateRequest(net::URLRequest* request) {};
+
   // The appcache policy, may be null, in which case access is always allowed.
   // The service does NOT assume ownership of the policy, it is the callers
   // responsibility to ensure that the pointer remains valid while set.