Summary: | [WinCairo] Support file downloads | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Yury Semikhatsky <yurys> | ||||||
Component: | WebKit2 | Assignee: | Yury Semikhatsky <yurys> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | annulen, Basuke.Suzuki, conojab991, don.olmstead, ews-watchlist, galpeter, gyuyoung.kim, Hironori.Fujii, kkinnunen, ryuan.choi, sergio, takashi.komori, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | PC | ||||||||
OS: | Windows 10 | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 234887 | ||||||||
Attachments: |
|
Description
Yury Semikhatsky
2022-05-10 17:39:11 PDT
Created attachment 459142 [details]
Patch
Comment on attachment 459142 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=459142&action=review > Source/WebKit/ChangeLog:8 > + Hooked up Curl calls to Download callbacks. Added basix support for basix → basic > Source/WebKit/NetworkProcess/curl/NetworkDataTaskCurl.cpp:194 > + FileSystem::writeToFile(m_downloadDestinationFile, segment.data(), segment.size()); Why do you ignore the return value of writeToFile? Comment on attachment 459142 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=459142&action=review >> Source/WebKit/ChangeLog:8 >> + Hooked up Curl calls to Download callbacks. Added basix support for > > basix → basic Done. >> Source/WebKit/NetworkProcess/curl/NetworkDataTaskCurl.cpp:194 >> + FileSystem::writeToFile(m_downloadDestinationFile, segment.data(), segment.size()); > > Why do you ignore the return value of writeToFile? Fixed. Added error handling. Created attachment 459177 [details]
Patch
Comment on attachment 459177 [details]
Patch
LGTM
Committed r294077 (250472@main): <https://commits.webkit.org/250472@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 459177 [details]. *** Bug 234887 has been marked as a duplicate of this bug. *** |