This is a subtask of http://code.google.com/p/chromium/issues/detail?id=39368 Need to add an extra parameter to the WebSharedWorker::startWorkerContext() method to indicate which appcache (by long long appcacheId) the shared worker resource was loaded from, and provide that value when webcore/webkit calls out to chrome via this interface.
Created attachment 55157 [details] add param
Fyi: This interface is both... * used by webkit/webcore to call out to chrome (in the renderer process) * used by chrome to call into webkit/webcore (in the worker process) ... so altering a method in this interface makes for multi-sided patch sequencing.
Created attachment 55194 [details] take2 A new patch that avoids making a copy of the ResourceResponse.
Attachment 55194 [details] did not build on chromium: Build output: http://webkit-commit-queue.appspot.com/results/2194008
Great, forgot to define the method in the class definition. Moving edits from this client to that client is such an error prone time sync hole.
Created attachment 55254 [details] take3
Comment on attachment 55254 [details] take3 r=me
Comment on attachment 55254 [details] take3 Clearing flags on attachment: 55254 Committed r59022: <http://trac.webkit.org/changeset/59022>
All reviewed patches have been landed. Closing bug.