Download objects should keep track of their associated DownloadManager object
Created attachment 179307 [details] Patch
Committed r137636: <http://trac.webkit.org/changeset/137636>
Comment on attachment 179307 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=179307&action=review > Source/WebKit2/WebProcess/Downloads/Download.cpp:-49 > - : m_downloadID(downloadID) What's the rationale for removing this line? See https://bugs.webkit.org/show_bug.cgi?id=104978
> Source/WebKit2/WebProcess/Downloads/Download.cpp:-49 > - : m_downloadID(downloadID) The EFL API test failed due to the uninitialized downloadID: 9/53 Test #9: test_ewk2_download_job [ RUN ] EWK2UnitTestBase.ewk_download ASSERTION FAILED: m_downloads.contains(download->downloadID()) Source/WebKit2/WebProcess/Downloads/DownloadManager.cpp(70) : void WebKit::DownloadManager::downloadFinished(WebKit::Download*)