Bug 173709 - [WPE] Downloads never have a web view associated in WPE
Summary: [WPE] Downloads never have a web view associated in WPE
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WPE WebKit (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-06-22 04:04 PDT by Carlos Garcia Campos
Modified: 2017-06-22 04:55 PDT (History)
2 users (show)

See Also:


Attachments
Patch (11.74 KB, patch)
2017-06-22 04:08 PDT, Carlos Garcia Campos
zan: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Garcia Campos 2017-06-22 04:04:00 PDT
PageClientImpl::handleDownloadRequest is not implemented in WPE. This causes some failures in the download unit tests:

TEST: bin/TestWebKitAPI/WPE/TestDownloads... (pid=20583)
  /wpe/Downloads/local-file:                                           OK
  /wpe/Downloads/overwrite-destination-allowed:                        OK
  /wpe/Downloads/overwrite-destination-disallowed:                     OK
  /wpe/Downloads/local-file-error:                                     OK
  /wpe/Downloads/remote-file:                                          OK
  /wpe/Downloads/remote-file-error:                                    OK
  /wpe/Downloads/policy-decision-download:                             **
ERROR:/home/cgarcia/src/git/gnome/WebKit-WPE/Tools/TestWebKitAPI/Tests/WebKitGLib/TestDownloads.cpp:621:void testPolicyResponseDownload(PolicyResponseDownloadTest*, gconstpointer): assertion failed: (test->m_webView == webkit_download_get_web_view(test->m_download.get()))
FAIL
GTester: last random seed: R02Sa8d9d000d6db4269850e6c1f4a34d47c
(pid=20692)
  /wpe/Downloads/policy-decision-download-cancel:                      **
ERROR:/home/cgarcia/src/git/gnome/WebKit-WPE/Tools/TestWebKitAPI/Tests/WebKitGLib/TestDownloads.cpp:640:void testPolicyResponseDownloadCancel(PolicyResponseDownloadTest*, gconstpointer): assertion failed: (test->m_webView == webkit_download_get_web_view(test->m_download.get()))
FAIL
GTester: last random seed: R02S8a053b0c2c679fe83edfdde89d0606b7
(pid=20727)
  /wpe/Downloads/mime-type:                                            OK
  /wpe/WebKitWebView/download-uri:                                     **
ERROR:/home/cgarcia/src/git/gnome/WebKit-WPE/Tools/TestWebKitAPI/Tests/WebKitGLib/TestDownloads.cpp:568:void testWebViewDownloadURI(WebViewDownloadTest*, gconstpointer): assertion failed: (test->m_webView == webkit_download_get_web_view(download.get()))
FAIL
GTester: last random seed: R02Sa218af78bd296a6c027f85e8642edab9
(pid=20771)
FAIL: bin/TestWebKitAPI/WPE/TestDownloads
Comment 1 Carlos Garcia Campos 2017-06-22 04:08:21 PDT
Created attachment 313609 [details]
Patch
Comment 2 Carlos Garcia Campos 2017-06-22 04:55:17 PDT
Committed r218691: <http://trac.webkit.org/changeset/218691>