RESOLVED FIXED 76297
Partially revert r104756 - multipart bugs 
https://bugs.webkit.org/show_bug.cgi?id=76297
Summary Partially revert r104756 - multipart bugs
Nate Chapin
Reported 2012-01-13 12:22:14 PST
The multipart code in SubresourceLoader has some subtleties I misunderstood when I tried to change them in http://trac.webkit.org/changeset/104756. That patch is causing a multipart test to assert in chromium win, and I suspect there are other issues I haven't found yet. I'm going to revert all the multipart changes (except the changes to CachedImage and when we call clear()) and give them more thought before trying again.
Attachments
patch (4.60 KB, patch)
2012-01-13 12:54 PST, Nate Chapin 		no flags
DetailsFormatted DiffDiff
Patch for landing (5.63 KB, patch)
2012-01-13 14:51 PST, Nate Chapin 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Nate Chapin
Comment 1 2012-01-13 12:54:45 PST
Created attachment 122485 [details] patch
Alexey Proskuryakov
Comment 2 2012-01-13 12:55:38 PST
Comment on attachment 122485 [details] patch rs=me
Early Warning System Bot
Comment 3 2012-01-13 13:04:40 PST
Comment on attachment 122485 [details] patch Attachment 122485 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/11152701
Nate Chapin
Comment 4 2012-01-13 14:51:18 PST
Created attachment 122505 [details] Patch for landing
WebKit Review Bot
Comment 5 2012-01-13 15:33:00 PST
Comment on attachment 122505 [details] Patch for landing Clearing flags on attachment: 122505 Committed r105000: <http://trac.webkit.org/changeset/105000>
WebKit Review Bot
Comment 6 2012-01-13 15:33:05 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.