Should be able to pause and resume downloads
<rdar://problem/8691135>
Created attachment 75062 [details] Patch
Comment on attachment 75062 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=75062&action=review Might be better to name this resumptionData rather than resumeData so it doesn’t sound like a verb phrase. But I guess the NSURLDownload method is already named resumeData? > WebKit2/UIProcess/Downloads/DownloadProxy.cpp:153 > + if (!resumeData.isEmpty()) > + m_resumeData = WebData::create(resumeData.data(), resumeData.size()); Would be nice to share this code instead of writing it twice.
(In reply to comment #3) > (From update of attachment 75062 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=75062&action=review > > Might be better to name this resumptionData rather than resumeData so it doesn’t sound like a verb phrase. But I guess the NSURLDownload method is already named resumeData? Yeah. > > > WebKit2/UIProcess/Downloads/DownloadProxy.cpp:153 > > + if (!resumeData.isEmpty()) > > + m_resumeData = WebData::create(resumeData.data(), resumeData.size()); > > Would be nice to share this code instead of writing it twice. Fixed. Thanks for reviewing!
Committed r72841: <http://trac.webkit.org/changeset/72841>