summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | Merge remote-tracking branch 'origin/5.13' into devAllan Sandfeld Jensen2019-05-274-0/+179
| |\| | | | | | | | | | | | | | | | | | Change-Id: I70917458ba43c8fbf27fee2382eee42618588fe4
| | * | | Merge remote-tracking branch 'origin/5.12' into 5.13Allan Sandfeld Jensen2019-05-244-0/+179
| | |\ \ \ | | | | |/ | | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/3rdparty src/core/net/network_delegate_qt.cpp tests/auto/widgets/qwebenginedownloaditem/tst_qwebenginedownloaditem.cpp Change-Id: Ib715b3047213583e0441b915d3cabb801d9d4ba8
| | | * | Add path validation for QWebEngineDownloadItem::setPath()Tamas Zakor2019-05-201-0/+123
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Do not set path if it ends with separator or if it matches with an already existing directory name. Task-number: QTBUG-75566 Change-Id: I4b78b28afe034c7589633c569a4945a36b32008e Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | | * | Referrer HTTP Header no longer ignored when set via RequestInterceptorLeander Beernaert2019-05-202-0/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make sure the HTTP referer is properly placed on a request when it's set via the QWebEngineUrlRequestInterceptor. Added test case to catch future incidents. Fixes: QTBUG-60203 Change-Id: Ida2f713a7352c3199fc9f8e15b5d8350d50afdda Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | | * | Update docs and test for runJavaScriptJüri Valdmann2019-05-091-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: QTBUG-69567 Change-Id: Icdf5a200f7be1eb7a98cce62848e3a641c49e804 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io> Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io>
* | | | | Implement page lifecycle APIJüri Valdmann2019-05-246-1/+725
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ChangeLog][QtWebEngine][WebEngineView] WebEngineView now supports lifecycle states that can be used for reducing CPU and memory consumption of invisible views. [ChangeLog][QtWebEngineWidgets][QWebEnginePage] QWebEnginePage now supports lifecycle states that can be used for reducing CPU and memory consumption of invisible pages. Fixes: QTBUG-74166 Fixes: QTBUG-55079 Change-Id: I7d70c85dc995bd17c9fe91385a8e2750dbc0a627 Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io> Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
* | | | Merge "Merge remote-tracking branch 'origin/5.13' into dev" into ↵Allan Sandfeld Jensen2019-05-172-0/+95
|\ \ \ \ | | | | | | | | | | | | | | | refs/staging/dev
| * | | | Merge remote-tracking branch 'origin/5.13' into devAllan Sandfeld Jensen2019-05-162-0/+95
| |\| | | | | | | | | | | | | | | | | | Change-Id: Idd02779e2910a792d50bb703ab92cd4da47f45fc
| | * | | Merge remote-tracking branch 'origin/5.12' into 5.13Allan Sandfeld Jensen2019-05-102-0/+95
| | |\| | | | | |/ | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/core/profile_adapter.cpp src/core/profile_adapter.h src/webengine/api/qquickwebengineprofile.cpp src/webenginewidgets/api/qwebengineprofile.cpp tests/auto/widgets/qwebengineprofile/tst_qwebengineprofile.cpp Change-Id: Ic561f22b361e32a0a63668c73bf6c0fc9c1ce16f
| | | * Q(Quick)WebEngineProfile: Update list of internal schemesJüri Valdmann2019-05-071-0/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Disallow installing handler for "about" and everything in kStandardURLSchemes of url/url_util.cc. Except for "gopher" which is used in tests. Suppress warning about custom schemes for "gopher" since it's not a custom scheme. Also lowercase the scheme in urlSchemeHandler() and removeUrlSchemeHandler(). Change-Id: I72b06d4fa6433882019405a0d600a593c8971bf1 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | | * Do not crash on QWebEnginePagePrivate::releaseProfile()Allan Sandfeld Jensen2019-05-031-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Deregistering the widget or deleting the view should be able to handle a now missing d_ptr in the old page. Change-Id: Ic843f7bde12776b6aad4fad865ccf14a25695154 Fixes: QTBUG-75547 Reviewed-by: Jüri Valdmann <juri.valdmann@qt.io>
| | | * Silence warnings in tst_qwebengineprofileAllan Sandfeld Jensen2019-04-301-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Register custom schemes Change-Id: I536a27da8f936e4ea6569e39a8ec6e30e23af03a Reviewed-by: Jüri Valdmann <juri.valdmann@qt.io>
| | | * Handle corner case of assigning an implicit page to a new viewAllan Sandfeld Jensen2019-04-301-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Pass the ownership on to the the new view, so it is still handled like an implicit page. Change-Id: I76ad3cb349a492e60e3ad2bdd4aebaabed07bd4f Reviewed-by: Jüri Valdmann <juri.valdmann@qt.io>
* | | | Add API to get the suggested download file nameTamas Zakor2019-05-163-0/+20
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | [ChangeLog][DownloadItem] Add functions and property to get the suggested download file name. Task-number: QTBUG-56978 Change-Id: I902870c6358271322da613a15ccc11639384e404 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | | Add navigation type for redirectsAllan Sandfeld Jensen2019-05-102-1/+2
| | | | | | | | | | | | | | | | | | Fixes: QTBUG-74490 Change-Id: Ia8de7dbec717021f367133cebb8c118e333388e4 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
* | | Merge remote-tracking branch 'origin/5.13' into devAllan Sandfeld Jensen2019-05-0919-44/+306
|\| | | | | | | | | | | Change-Id: I495adfcbb010ac29dc64e051c030a92d27859ca6
| * | Do not blacklist webChannelWithBadStringMichal Klocek2019-05-033-6/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit is about documenting some weird new behavior. Since Chormium 72 our webChannelWithBadString fails. This is fascinating issue and can be tracked down to commit in Chromium https://chromium-review.googlesource.com/c/1282993 which "fixes" well-forming of JSON.strigify. This function is used by webchannel.js to send data through webchannel transport. In c++ land we get now for invalid utf16 replacement in form of '\\u' + i.toString(16).padStart(4, '0'). This sadly does not trigger any longer REPLACE_INVALID_UTF8 in WebChannelTransport::NativeQtSendMessage. Moreover this goes even "worse" since QMetaObjectPublisher will create invalid utf16 form raw "\ud800" string -> '\ud800'. This is not an error per se, since test creates invalid utf16 character in javascript and it ends as invalid utf8 character in c++. Btw enable test on qemu. Change-Id: I36f2df417d7b9f3f2113792f08025821737d8d01 Reviewed-by: Jüri Valdmann <juri.valdmann@qt.io>
| * | Skip tst_QWebEngineProfile::qtbug_71895 if network load failsJüri Valdmann2019-05-031-2/+3
| | | | | | | | | | | | | | | Change-Id: I0daf14c4ec31dfb867d9d7f531b9fdc6f7244e1b Reviewed-by: Michal Klocek <michal.klocek@qt.io>
| * | Merge remote-tracking branch 'origin/5.12' into 5.13Allan Sandfeld Jensen2019-04-303-3/+98
| |\| | | | | | | | | | Change-Id: I051052a70a6e764202a90d77fc1a0f73688d3704
| | * Profile test: fix buffer for request job being deleted on main threadKirill Burtsev2019-04-231-2/+1
| | | | | | | | | | | | | | | Change-Id: I894946ce7c9dcf4f787c5c928e3737cd80a7dc52 Reviewed-by: Jüri Valdmann <juri.valdmann@qt.io>
| | * Fix QWebEngineView::setPage not deleting old pageJüri Valdmann2019-04-171-0/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also fix QWebEnginePage::setView not deleting old page Also fix wrong page being deleted if it's parented to the view. Fixes: QTBUG-75131 Fixes: QTBUG-75175 Change-Id: Ie4dfb15b3182de7aa3a94cddcac54ea40a86121b Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | * Fix resolving relative URLs with custom schemesJüri Valdmann2019-04-161-0/+49
| | | | | | | | | | | | | | | | | | | | | | | | Pull in 3rdparty patch and add test. Fixes: QTBUG-74864 Change-Id: I5440f58ff55297c2a51f896d43f479404ff6ca2f Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | * Profile test: fix waiting for load finished signalKirill Burtsev2019-04-111-1/+2
| | | | | | | | | | | | | | | Change-Id: I797b596b73d62e420592ef9e96601c35ea7ecad6 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
| * | Make offscreen test offlineMichal Klocek2019-04-174-3/+16
| | | | | | | | | | | | | | | | | | | | | This enables also the test on qemu. Change-Id: I70c40b3cedeadfb21e396d8a550a54fe0a319438 Reviewed-by: Michael Brüning <michael.bruning@qt.io>
| * | Fix running proxypac test on qemuMichal Klocek2019-04-172-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Do not override '--no-sandbox' on boot2qt. Starting resolver service on qemu is bonkers and ends up in unsupported calls during threading checks. Adding '--no-zygote' solves the issue, however introduces race condition and deadlocks on resolver service start, which would make test flaky. The deadlock issue is not reproducible without qemu. Therefore to workaround the issue run test in single process mode. Task-number: QTBUG-63346 Change-Id: I33a1079776edecddf3d7653db6c2ecc3a868b1a1 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Add unit test for verifying QWebEnginePage::contentsSize correctnessMichael Brüning2019-04-131-0/+29
| | | | | | | | | | | | | | | | | | | | | | | | An adaptation to Chromium 70 caused a regression in this. Change-Id: I8928694b9f99cd420afd858d3a1023228276bae0 Reviewed-by: Kirill Burtsev <kirill.burtsev@qt.io> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Fix qmltests::WebEngineViewSingleFileUpload auto testsTamas Zakor2019-04-104-21/+98
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add scheme checks for FilePickerController::accepted(). A file:// prefix to a file selected here causes chromium to terminate the render process due to an illegal ipc message. Extend auto tests with custom Dialog. Also fix directory upload test. Change-Id: I8d672e435129d62d268b48c3471b924c0161b44b Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Skip downloadToReadOnlyDir auto test on WindowsPeter Varga2019-04-102-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It should be rather skipped than blacklisted since it is not expected to work on Windows. See the corresponding warning about ACLs: https://doc.qt.io/qt-5/qfile.html#setPermissions Task-number: QTBUG-74698 Change-Id: Iad66c028583d6811b7a413dd9800e9095d8f6da7 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Unskip downloadUniqueFilenameWithTimestamp auto testPeter Varga2019-04-091-7/+4
| | | | | | | | | | | | | | | | | | Task-number: QTBUG-74764 Change-Id: I0e872449584b37c484e4d5ea46ba0bbc47124f51 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Merge branch '5.12' into 5.13Allan Sandfeld Jensen2019-04-093-1/+51
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/3rdparty src/core/renderer/user_resource_controller.cpp src/core/web_contents_adapter.cpp src/webengine/doc/src/qtwebengine-overview.qdoc Change-Id: I46be9d33b3b65d61dfa099ee72a3509afb9bd6a4
| | * Run non-MainWorld DocumentCreation scripts even if JS disabledJüri Valdmann2019-04-081-0/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes regression, introduced by the fix for QTBUG-66011, where setting JavascriptEnabled to false stops all scripts from running instead of only MainWorld scripts (as documented). Only the DocumentCreation injection point is affected. The original change which introduced the regression consisted of moving the DocumentCreation injection point from ContentRendererClient::RunScriptsAtDocumentStart to RenderFrameObserver::DidClearWindowObject. The problem of scripts not working on view-source URLs was fixed by this move, but it turns out that the call to DidClearWindowObject happens to be conditional on Document::CanExecuteScripts and this is, of course, false if JS is disabled. Hence the regression. This new patch moves the injection point again to a task launched from RenderFrameObserver::DidCommitProvisionalLoad. DidCommitProvisionalLoad and DidClearWindowObject are both indirectly called from DocumentLoader::InstallNewDocument, however the former is called before the Document is opened and is therefore too early for script execution. As such, the execution is delayed by posting a task which, in theory, should be scheduled very soon after the normal call to DidClearWindowObject. Fixes: QTBUG-74304 Change-Id: Iac8714bcc5651c287b73181811af26996d955af5 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | * Increase timeout and add some debug info to tst_dialogMichal Klocek2019-04-042-1/+3
| | | | | | | | | | | | | | | | | | Task-number: QTBUG-72113 Change-Id: I9b2dfb360a18995cae9e0e4a98443e796c3d8ad0 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | * Make tst_dialogs less flakyMichal Klocek2019-04-031-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | Wait to finish javascript execution before opening next dialog. Task-number: QTBUG-72113 Change-Id: I383e3fc3c405a94ddcbd74fe1cc7ab0d81c4e315 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | | Extend the QQuickWebEngineDownloadItem Qml type with url propertyTamas Zakor2019-04-292-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | Task-number: QTBUG-72996 Change-Id: I8af67d69a9a88ccbc95e37a252615dd3fd3ca0ca Reviewed-by: Kirill Burtsev <kirill.burtsev@qt.io> Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io> Reviewed-by: Jüri Valdmann <juri.valdmann@qt.io>
* | | Blacklist runJavaScriptFromSlot on Linux tooAllan Sandfeld Jensen2019-04-121-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | Now it fails on Linux as well. Task-number: QTBUG-75143 Change-Id: I4eeafb084d35278119aab0584fb5ffac196164f2 Reviewed-by: Liang Qi <liang.qi@qt.io>
* | | Merge remote-tracking branch 'origin/5.13' into devQt Forward Merge Bot2019-04-0515-45/+58
|\| | | | | | | | | | | Change-Id: I8f0fa53b1e202d1932008291ea786a4ed1a49a64
| * | Notification API cleanupKirill Burtsev2019-04-055-19/+24
| | | | | | | | | | | | | | | | | | Task-number: QTBUG-74543 Change-Id: Ice5a0dbfc3485c8b7e6fa900ef427a9aed871d42 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Fix tst_Dialogs::authenticationDialogRequestedMichal Klocek2019-04-044-13/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Chromium returns unexpected proxy reply if localhost is used. Since proxy resolver works now, the workaround to trigger dialog with "localhost proxy" can be skipped. Change-Id: I26d88c43779809e4f481101266e6b8784a6b196d Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Remove our last uses of Q_DECL_OVERRIDE and Q_NULLPTRAllan Sandfeld Jensen2019-04-036-12/+12
| | | | | | | | | | | | | | | Change-Id: I8806a3fb466006f14cf92f17510cdea8b50e8345 Reviewed-by: Jüri Valdmann <juri.valdmann@qt.io>
| * | Make the skip actually compileAllan Sandfeld Jensen2019-04-021-1/+1
| | | | | | | | | | | | | | | Change-Id: I348fd097db179557d04cde224f6a49f0d1c2f97d Reviewed-by: Jüri Valdmann <juri.valdmann@qt.io>
* | | Merge remote-tracking branch 'origin/5.13' into devAllan Sandfeld Jensen2019-03-2910-10/+26
|\| | | | | | | | | | | Change-Id: Iab8e05beb7ede38bacaa35322d1f79cfc85cd470
| * | Skip downloadUniqueFileNameWithTimeStamp auto test on WindowsPeter Varga2019-03-281-0/+3
| | | | | | | | | | | | | | | | | | Task-number: QTBUG-74764 Change-Id: I2638a6857e1d29ebacaafac3fff5daab57268b54 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Blacklist tst_QWebEnginePage::runJavaScriptFromSlot on macOSAllan Sandfeld Jensen2019-03-281-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | This is our most flaky test at the moment Task-number: QTBUG-74718 Change-Id: Ia7668347a74982abfb0297169f564db89944e87c Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Do not run offscreen test on windowsMichal Klocek2019-03-271-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | Offscreen plugin tries to open opengl handle and fails. Disable test for now. Change-Id: I6cecfef4e47a700ff720811f8e8dc4829a814821 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Normalize download path on WindowsAllan Sandfeld Jensen2019-03-261-9/+0
| | | | | | | | | | | | | | | | | | Task-number: QTBUG-74698 Change-Id: I3fd4b932b1d4b0ecc844818f87c1824eac1f56e6 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Blacklist download dir tests for windowsAllan Sandfeld Jensen2019-03-261-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | Needs some cleanup to do comparisons FS-agnostic, and is currently blocking. Task-number: QTBUG-74698 Change-Id: Idca2753992aa8d1d63456694129325e757e572bc Reviewed-by: Michal Klocek <michal.klocek@qt.io>
| * | Fix disabling http cache after 73-basedPeter Varga2019-03-231-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | BrowserDataRemoverImpl::Remove() indirectly calls TransportSecurityState::DeleteAllDynamicDataSince() which notifies by a callback about the finished deletion since: https://chromium-review.googlesource.com/c/chromium/src/+/1335939 During the deletion the ProfileIODataQt::requestStorageGeneration() should not be called because it deletes net::TransporSecurityPersister which background_runner is where the finished deletion callback is scheduled. Change-Id: I4782d701f706ed7c8e104a78ba84a27183166fa4 Reviewed-by: Jüri Valdmann <juri.valdmann@qt.io>
| * | Adaptations for Chromium 73Allan Sandfeld Jensen2019-03-233-5/+8
| | | | | | | | | | | | | | | Change-Id: I565d1e327852110a5abebed3388d7cd6986bef06 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
| * | Adaptations for Chromium 72Allan Sandfeld Jensen2019-03-232-3/+6
| | | | | | | | | | | | | | | Change-Id: Ic355257066c7c1433862cb41e6f2bfa831147e0d Reviewed-by: Jüri Valdmann <juri.valdmann@qt.io>
* | | Merge remote-tracking branch 'origin/5.13' into devAllan Sandfeld Jensen2019-03-2126-33/+755
|\| | | | | | | | | | | Change-Id: If771fabe97c4d75edc4627999f04f70ca1901060