RESOLVED FIXED 102995
Remember the CachedResourceRequest for preloads 
https://bugs.webkit.org/show_bug.cgi?id=102995
Summary Remember the CachedResourceRequest for preloads
James Simonsen
Reported 2012-11-21 17:22:54 PST
Remember the CachedResourceRequest for preloads
Attachments
Patch (7.32 KB, patch)
2012-11-21 17:24 PST, James Simonsen 		no flags
DetailsFormatted DiffDiff
Patch (7.38 KB, patch)
2012-11-26 15:57 PST, James Simonsen 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
James Simonsen
Comment 1 2012-11-21 17:24:11 PST
Created attachment 175554 [details] Patch
Build Bot
Comment 2 2012-11-21 18:27:34 PST
Comment on attachment 175554 [details] Patch Attachment 175554 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/14963146 New failing tests: fast/encoding/preload-encoding.html
WebKit Review Bot
Comment 3 2012-11-23 07:22:39 PST
Comment on attachment 175554 [details] Patch Attachment 175554 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/14959701 New failing tests: fast/js/bom-in-file-retains-correct-offset.html fast/regex/unicodeCaseInsensitive.html fast/dom/attribute-legacy-colors.html dom/html/level1/core/hc_attrgetvalue1.html fast/encoding/preload-encoding.html dom/html/level1/core/hc_textparseintolistofelements.html editing/style/make-text-writing-direction-inline.html
James Simonsen
Comment 4 2012-11-26 15:57:48 PST
Created attachment 176100 [details] Patch
James Simonsen
Comment 5 2012-11-28 12:09:27 PST
This is needed for Resource Timing. The initiator should be the same regardless of whether or not it's preloaded.
Marja Hölttä
Comment 6 2012-11-29 00:32:28 PST
Ahh. This is also needed for our use cases. Thanks for fixing this!
Marja Hölttä
Comment 7 2012-11-29 00:34:00 PST
... meaning that we probably also want to notify the embedder in requestPreload; that's not done yet.
WebKit Review Bot
Comment 8 2012-11-30 11:18:35 PST
Comment on attachment 176100 [details] Patch Clearing flags on attachment: 176100 Committed r136255: <http://trac.webkit.org/changeset/136255>
WebKit Review Bot
Comment 9 2012-11-30 11:18:38 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.