summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Fix tst_QWebEngineHistory widget auto testsPeter Varga2019-05-081-14/+18
| | | | | | | | | | - Reset loadFinishedSpy counter after initialization - Do not expect loadFinished signal after "same document" navigation Change-Id: If78304d8fa67ec96b28811c148b4a23f664987fb Reviewed-by: Viktor Engelmann <viktor.engelmann@qt.io> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io> (cherry picked from commit 3f4daf02723b16e83dfb9bcee654b5c803de1ae8)
* Blacklist flaky qwebenginedownloads testMichal Klocek2018-05-311-0/+4
| | | | | | | | | This test suffers for out of order handling of network requests. To speed up release blacklist flaky tests. (cherry picked from commit 9dc8dff7a8f4d58f71d816375d49f8829f06aae5) Change-Id: Ief7abe726e074465f3c12fd88168c7b78a61f497 Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* Blacklist mouseMovementProperties test on WindowsAlexandru Croitor2018-05-311-0/+1
| | | | | | | (cherry picked from commit b6d0d11916a0b1f1edc913aafd8220d3e0253482) Task-number: QTBUG-66527 Change-Id: I3a7bcc3fdced1ebdf3b804eae7e3280de365464e Reviewed-by: Jani Heikkinen <jani.heikkinen@qt.io>
* Fix QWebEngineDownloadItem::type()Jüri Valdmann2018-01-191-4/+4
| | | | | | Task-number: QTBUG-62640 Change-Id: I2b16f24533b38c20a7071319723382ba240e35f3 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
* ProxyConfigServiceQt: Use default HostPortPair for SCHEME_DIRECTJüri Valdmann2018-01-151-0/+19
| | | | | | | | | Unlike QNetworkProxy, Chromium's net::ProxyServer expects the hostname and port to be at default values for special schemes (DIRECT and INVALID). Otherwise, a DCHECK is triggered at proxy_server.cc:73. Change-Id: I1ac6c425ea03fcbfe084d25c2fd05bf174c753d6 Reviewed-by: Viktor Engelmann <viktor.engelmann@qt.io>
* Stabilize tst_QWebEngineDownloads::downloadTwoLinks()Allan Sandfeld Jensen2018-01-151-4/+16
| | | | | | | | It was vulnerable to race conditions if two requests were send before the first was caught. Change-Id: Ia0b68594337028675678ad190f261202372c2ad6 Reviewed-by: Viktor Engelmann <viktor.engelmann@qt.io>
* Doc: QWebEngineUrlRequestJob: mention when to delete the deviceDavid Faure2017-12-201-5/+15
| | | | | | | | | | It's used from another thread, so it shouldn't be deleted immediately, and it's not deleted by the QtWebEngine code, so tell people to do it themselves at the right time. Change-Id: I54786be320f5fe82f144e7b1c2e6137260d9ceab Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Fix favicon update from JavaScriptPeter Varga2017-12-132-21/+100
| | | | | | Task-number: QTBUG-64967 Change-Id: Ida79d6cdb682d510e9a2e91e3e3ca263acf34a99 Reviewed-by: Viktor Engelmann <viktor.engelmann@qt.io>
* Blacklist failing test_findTextInterruptedByLoad QML testAlexandru Croitor2017-12-041-0/+3
| | | | | | | | | | | Continiously failed 5 times on Windows when integrating unrelated patches. Task-number: QTBUG-61887 Change-Id: Ibc3e1563355364302f7ee430c97209798e86e3e7 Reviewed-by: Michael Brüning <michael.bruning@qt.io> (cherry picked from commit f1003c50652c1f5ef9ae0e16036b175fe3e8f215) Reviewed-by: Liang Qi <liang.qi@qt.io>
* URLRequestQrcJobQt: remove ineffective URL formatting optionsJüri Valdmann2017-11-105-0/+42
| | | | | | | Remove strange options for call to QUrl::path and add test for "qrc" protocol. Change-Id: I6528d858b7661832852c333a7f932d4714f953f2 Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
* Notify Chromium about leaving viewPeter Varga2017-11-093-0/+150
| | | | | | | | Forward QEvent::Leave for Widget and QEvent::HoverLeave for Quick. Task-number: QTBUG-64265 Change-Id: Ide32768902956476d24b1d4115e305392b62feb3 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Build accessibility tests only when Qt is configured with accessibilityMichael Brüning2017-10-261-1/+4
| | | | | | | | Original patch by Daniel Mack <daniel@zonque.org> Task-number: QTBUG-64015 Change-Id: I7dd0a1aa9bff08c6d19e818acf0e1b8b4b701f5b Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
* Speculative stabilization of tst_QWebEngineView::emptyInputMethodEventPeter Varga2017-09-211-4/+4
| | | | | Change-Id: I5dd8c3251587c706f1fd3b90f77e4ac03c2fba97 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Refactor tst_qquickwebengineview auto testPeter Varga2017-09-202-100/+41
| | | | | | | Pass JavaScript result via callback instead of using console.log. Change-Id: Ie464f038d4fa778b1f64e95eca58e86e29184c1e Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Fix license headers for testsKai Koehne2017-09-208-206/+68
| | | | | | | | | Tests should all be GPL-EXCEPT. Task-number: QTBUG-60006 Change-Id: I2466374e863bd1c3cd791ade45caf1087be78cef Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io> Reviewed-by: Lars Knoll <lars.knoll@qt.io>
* Commit the done-so-far IME composition on touch eventPeter Varga2017-09-172-0/+170
| | | | | | | | | | | Fix is based on afc9e2d9674f7ab5800df4803cc68c71d1ae691a Moreover, new quick auto test has been added to check that the commit happens in case mouse and touch input events. Task-number: QTBUG-62942 Change-Id: Ie9d55e0bb5b3bbc34c099502e735b94f37c5d5f8 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Set referrer on download requestsJüri Valdmann2017-09-121-7/+7
| | | | | | | | | Note that the Referer header still won't be sent if the download is triggered via an anchor element with the 'download' attribute: crbug.com/455987 . Task-number: QTBUG-61354 Change-Id: I5af971af916b2190756e3e58f19736072a213922 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* Add test for saving a page over HTTPJüri Valdmann2017-09-121-0/+85
| | | | | Change-Id: I6d9261292e44484cded421402fc06ee2eb08bdea Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* Add test for two simultaneous downloadsJüri Valdmann2017-09-121-0/+92
| | | | | Change-Id: I8342aa806c3d1a8f4f1a5e822ef1be0e1755220c Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* Add test for downloading a link via user actionJüri Valdmann2017-09-129-1/+814
| | | | | Change-Id: Ide3294840ceb3d18da0c4da92d892ff467a9b739 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* Speculative stabilization of imeCompositionQueryEvent widget auto testPeter Varga2017-07-191-5/+7
| | | | | | | | Wait for receiver object to be set after show. Task-number: QTBUG-61816 Change-Id: I2eb62775d30b0f8f9ec3e76d60a924b56c811865 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Fix propagation of unhandled key press eventsJoerg Bornemann2017-07-141-1/+10
| | | | | | | | | | | | | | | Unhandled key presses for printable keys were not received by the parent widget anymore. This was a regression from 5.8.0. The propagation is suppressed if NativeWebKeyboardEvent::skip_in_browser is true. Commit 4501b9d8 accidentally removed the wrong skip_in_browser assignment. Task-number: QTBUG-61621 Change-Id: I7eafa1fac5fb9a7edc4af2bc1aac21c106264713 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
* [macOS] Use the OpenGL CoreProfile when the global shared context doesAlexandru Croitor2017-07-112-2/+5
| | | | | | | | | | | | | | | | | | | | Previously when a default QSurfaceFormat was set with an OpenGL Core profile, all the contexts created on the Qt side would obey the profile, but Chromium would still use the Compatibility profile for its contexts leading to warnings when trying to create shared contexts. The fix is to check which OpenGL profile is used in the Qt global shared context, and pass that information along to Chromium. Note that this works only on macOS for now, and the default non-Core profile is used on other platforms, even though Core was requested. Passing CoreProfile to Chromium on Windows and Linux currently leads to crashes. Task-number: QTBUG-60605 Change-Id: I27a77091923624d19ccc2019953a5b07f9282916 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu> Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* Call stopFinding, when navigating awayViktor Engelmann2017-07-061-3/+3
| | | | | | | | | | | When one searches for text on one page and then navigates away, results for the search-term are also highlighted on the new page. To stop this from happening, we now call adapter->stopFinding() when we navigate away from a page. Task-number: QTBUG-61506 Change-Id: I09e064d7788b12eae0a67c01e4f5872d94924617 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Fix cancelling IME compositionPeter Varga2017-06-221-2/+26
| | | | | | Task-number: QTBUG-61429 Change-Id: I936f1c924ae8158a5243fdf20d12e6c5508b684a Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* Override shortcuts only when an HTML input field has focusJoerg Bornemann2017-06-082-9/+15
| | | | | | | | | | | | | | | | Otherwise an application shortcut like Shift+Delete would no longer work when webengine has focus (e.g. "delete mail" in KMail) This removes unconditional calls to editorActionForKeyEvent for ShortcutOverride event handling. We can remove those, because the key sequences that are checked by editorActionForKeyEvent are a subset of the key sequences checked by isCommonTextEditShortcut. This amends commit 3902b27e. Change-Id: I12a98368381edef36f11457c8b864d843efb871a Reviewed-by: David Faure <david.faure@kdab.com> Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
* Stabilize tst_download.qml and add check for downloadInterruptReasonValentin Fokin2017-06-081-4/+13
| | | | | | | | | Change some compares to tryCompares to wait for downloadStateChanged signal before comparing downloadState. Change-Id: If8f7f15a776e60c937f0e006d2ac337f6c1f9ad9 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Merge loadProgressSignal qml test into loadProgressValentin Fokin2017-06-083-65/+17
| | | | | | | | | | tst_loadProgressSignal.qml is a duplicate of tst_loadProgress.qml. Move loadProgressChanged signal check to tst_loadProgress.qml and delete tst_loadProgressSignal.qml. Change-Id: I14821d4979def7b9f1985859384224b8f8111bd6 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Replace SignalBarrier by QSignalSpy in tst_QWebEngineHistoryViktor Engelmann2017-06-011-66/+65
| | | | | | | | | | | | tst_QWebEngineHistory has lots of race conditions similar to those we have seen for example in tst_QWebEngineView. We now use QSignalSpy and QTRY_COMPARE instead of SignalBarrier and ensureSignalEmitted(), because these are more robust. We did the same for tst_QWebEngineView. Task-number: QTBUG-58473 Change-Id: I1d12914670aa1c08b7edf7593df8c950ad6d4b0b Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Add back widget auto tests for inputMethodQuery and inputMethodHintsPeter Varga2017-05-312-118/+117
| | | | | | Task-number: QTBUG-55766 Change-Id: Ic90eb7b9a211b09281ccbb3eae727e8a1c081b9a Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Add QML auto test for findText interrupted by page loadPeter Varga2017-05-311-0/+58
| | | | | | Change-Id: I9eccddd420cd0eb8b5be3006a4091465d4b3fab0 Task-number: QTBUG-60472 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Store Target URL in WebContentsDelegateQt::WebContentsCreatedViktor Engelmann2017-05-301-6/+12
| | | | | | | | | | | | | | | | | | | | | | | When opening a new window, for example by using the JavaScript method window.open('...'), the requested url is not stored in the content::WebContents object we get in WebContentsDelegateQt::createWindow (at this point, it should at least be stored as pending request in the WebContents' NavigationController, but it is not). Because of this, the QQuickWebEngineNewViewRequest object in QQuickWebEngineViewPrivate::adoptNewWindow never contained the url. We have access to the target url in WebContentsDelegateQt::WebContentsCreated, so now we store it there in a new property m_initialTargetUrl, from where WebContentsDelegateQt::createWindow takes it and passes it to WebContentsAdapter::adoptNewWindow as a new parameter. [ChangeLog][WebEngine] Fix WebEngineNewViewRequest::requestedUrl being empty when opening window from JavaScript Task-number: QTBUG-57675 Change-Id: I7e2c7866899baade17ce2517e6be8b2b2709699e Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Merge remote-tracking branch 'origin/5.9.0' into 5.9Allan Sandfeld Jensen2017-05-291-2/+93
|\ | | | | | | Change-Id: Iba6114263488d6bf84b255b38182904dc5880386
| * Fix selectionChanged signal out of input fieldPeter Varga2017-05-171-2/+93
| | | | | | | | | | | | Task-number: QTBUG-60688 Change-Id: I6d0b78e6b8df54c40ae30d5f0909c631c440a9cd Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
* | Make error message more verbose in tst_QWebEnginePage::progressSignalViktor Engelmann2017-05-231-1/+7
| | | | | | | | | | | | | | | | | | | | If the Q_VERIFY(current >= previousValue) fails, the output was not verbose enough. The additional outputs will tell us more about what caused the test to fail. Task-number: QTBUG-57839 Change-Id: Ieeefe68ede111be7dba5eda9f2a770e975efc0ee Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* | Disable/blacklist failing tests for boot2qtSami Nurmenniemi2017-05-083-0/+11
|/ | | | | | | | | Currently boot2qt is tested with QEMU and some tests fail. Disabling them for now to make it possible to enable qt5 tests for Coin. Task-number: QTBUG-60266 Change-Id: I91a61bfb99b2629cb549b76d60c24f1d5f8c3ad8 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Fix typo in tst_newViewRequest.qmlPeter Varga2017-05-031-3/+3
| | | | | | | Change-Id: I3e22f09b0847cb7c5edc50ad12c73b62c4525072 Reviewed-by: Viktor Engelmann <viktor.engelmann@qt.io> Reviewed-by: Valentin Fokin <fokinv@inf.u-szeged.hu> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Stabilize keyboardEvents auto testsPeter Varga2017-04-282-2/+2
| | | | | | | | The typeAheadTimeout is set to 1sec in Blink. Wait a bit more when selecting the next option in a HTML select element in auto tests. Change-Id: Ia3d9fde05b4d6cde9186aaa701ca48354ca74b4d Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Override shortcuts in HTML input fieldsJoerg Bornemann2017-04-251-0/+58
| | | | | | | | | | | | | | | | | | | When users defined a single-letter short cut it was not possible to type this letter in HTML input fields. Fix this by accepting ShortcutOverride events whenever the web page is editing text. Use QInputControl::isCommonTextEditShortcut for Qt 5.9 and later. For the case where QtWebEngine is built against an older Qt a duplicated code path is used. Also, ensure users do not override web action short cuts. Task-number: QTBUG-59053 Change-Id: Ic26cf2a040a72b118273c6645c00b2913b995b0b Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
* Blacklist tst_QWebEngineView::imeCompositionPeter Varga2017-04-241-0/+3
| | | | | | Task-number: QTBUG-60381 Change-Id: Ib0e7ac3e5d7c3a9cea453fee96b2c5984567aa99 Reviewed-by: Liang Qi <liang.qi@qt.io>
* Rename macos blacklisting to osxTony Sarajärvi2017-04-241-1/+1
| | | | | Change-Id: If3581b261bfb1961320e145f94d27417ab8f8ee8 Reviewed-by: Liang Qi <liang.qi@qt.io>
* Disable setting background color in QInputMethodEventPeter Varga2017-04-211-0/+6
| | | | | | | | | | | | | IME managers may set background and foreground color of the composition string. Since there is no Chromium API for setting the foreground color, changing the background color to black may produce "black-on-black" text. Chromium locks the background color to transparent thus we do the same in this fix. Task-number: QTBUG-58362 Task-number: QTBUG-60031 Change-Id: I939c8aaaa847de191890d6924ba3c7652465f8a0 Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
* Handle QInputMethodQueryEvent forwarded by RWHVQDWPeter Varga2017-04-121-0/+80
| | | | | | Task-number: QTBUG-58362 Change-Id: I6c80c8063ccad97aa80aff8ee44aa10a899c5ff5 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Fix text selection and input method queryPeter Varga2017-04-122-565/+385
| | | | | | | | | | | | | | | Instruct the render process to change the text selection if it was requested via an input method event. Raise the selectionChanged() signal when all the corresponding input method properties are set. Moreover, add back the remaining input method widget auto tests. The updated tests are moved to the QWebEngineView tests since the corresponding APIs (inputMethodQuery() and input event handling) are now available via the QWebEngineView's focus proxy (aka RWHV). Task-number: QTBUG-55766 Change-Id: Ia0022d5f38b31dd59b084ff42e4abc2780ae90ec Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Extend blacklisting of tst_QWebEnginePage to macOS 10.11Tony Sarajärvi2017-04-111-0/+1
| | | | | | Task-number: QTBUG-59808 Change-Id: I25b73f1a89fb9e19d04c5a4ccbadbe44acea039d Reviewed-by: Liang Qi <liang.qi@qt.io>
* Merge "Merge remote-tracking branch 'origin/5.8' into 5.9" into refs/staging/5.9Allan Sandfeld Jensen2017-04-094-0/+44744
|\
| * Merge remote-tracking branch 'origin/5.8' into 5.9Allan Sandfeld Jensen2017-04-054-0/+44744
| |\ | | | | | | | | | | | | | | | | | | Conflicts: src/core/gyp_run.pro Change-Id: I3e2d72769947e7abb0585e922e7517ecb0087331
| | * Binary compatibility files and global.cfg for QtWebEngine5.8Milla Pohjanheimo2017-04-054-0/+44744
| | | | | | | | | | | | | | | | | | | | | Add means to test binary compatibility for QtWebEngine too. Change-Id: I1bc39da3da1595b8ac39be6936ed47080a52c78b Reviewed-by: Sergio Ahumada <sahumada@texla.cl>
* | | Blacklist LinkHovered tests on Linuxes after fix did not workTony Sarajärvi2017-04-071-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6252f0231f85e5f4e6b887a6814d4968acae13bd tried fixing these autotests. As it evidently didn't work, we bring the blacklisting back. Task-number: QTBUG-55870 Change-Id: I12dc54a81cc25f743f1b37e321a6524ba6b5915c Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
* | | Yet another speculative fix of linkHovered QML testsPeter Varga2017-04-056-18/+19
|/ / | | | | | | | | | | | | | | | | | | | | | | | | When the test moves the mouse cursor out of the link area it positions the cursor above an iframe. We should not expect linkHovered signal above an iframe until its load is succeeded. Since we don't have API to test iframes' loading state, remove the iframe from the test. Moreover, increase signal timeout values to 12sec as it is used in other tests too since the 55-based chromium update. Task-number: QTBUG-55870 Change-Id: Id47494ac41a9d34f8a5bf4804fc5c157868cf923 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>