Bug 38605 - [chromium] WebKit API additions to support appcache in workers.
Summary: [chromium] WebKit API additions to support appcache in workers.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit API (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Michael Nordman
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-05 13:38 PDT by Michael Nordman
Modified: 2010-05-08 05:39 PDT (History)
3 users (show)

See Also:


Attachments
add param (8.87 KB, patch)
2010-05-05 14:48 PDT, Michael Nordman
no flags Details | Formatted Diff | Diff
take2 (8.77 KB, patch)
2010-05-05 19:34 PDT, Michael Nordman
no flags Details | Formatted Diff | Diff
take3 (8.98 KB, patch)
2010-05-06 10:14 PDT, Michael Nordman
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Nordman 2010-05-05 13:38:15 PDT
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.
Comment 1 Michael Nordman 2010-05-05 14:48:37 PDT
Created attachment 55157 [details]
add param
Comment 2 Michael Nordman 2010-05-05 15:26:13 PDT
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.
Comment 3 Michael Nordman 2010-05-05 19:34:24 PDT
Created attachment 55194 [details]
take2

A new patch that avoids making a copy of the ResourceResponse.
Comment 4 WebKit Review Bot 2010-05-05 21:02:18 PDT
Attachment 55194 [details] did not build on chromium:
Build output: http://webkit-commit-queue.appspot.com/results/2194008
Comment 5 Michael Nordman 2010-05-05 21:14:21 PDT
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.
Comment 6 Michael Nordman 2010-05-06 10:14:46 PDT
Created attachment 55254 [details]
take3
Comment 7 Dmitry Titov 2010-05-06 11:11:06 PDT
Comment on attachment 55254 [details]
take3

r=me
Comment 8 WebKit Commit Bot 2010-05-08 05:39:36 PDT
Comment on attachment 55254 [details]
take3

Clearing flags on attachment: 55254

Committed r59022: <http://trac.webkit.org/changeset/59022>
Comment 9 WebKit Commit Bot 2010-05-08 05:39:41 PDT
All reviewed patches have been landed.  Closing bug.