summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Remove bic data to block bic testsMichal Klocek2017-09-053-44737/+0
| | | | | | | | This is workaround till qtqa patch is in. Task-number: QTBUG-63005 Change-Id: I0e94304df489160f475e6b6542ca11220f52a63b Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
* Skip tests due to issues with current 58-based ChromiumAlexandru Croitor2017-09-044-1/+18
| | | | | | | | These will be restored once 60-based Chromium is used. Task-number: QTBUG-62973 Change-Id: Ia1ad8e1dd7914e41f92d7a790f7c0b3956c68f28 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Add auto tests for MultipleMouseClickHelperPeter Varga2017-08-313-0/+227
| | | | | | | | | | | Qt WebEngine implements a custom handler for multiple mouse clicks. These tests are intended to test whether the mouse click events are properly forwarded to the Chromium's RenderWidgetHost. Custom mouse click test functions have been also added for the custom mouse click handler. Change-Id: Ifdc7d09f5e0f0f76c37e137e5743482bf3fb8abb Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Make deletion of CallbackSpy safeMichal Klocek2017-08-282-7/+24
| | | | | | | | | The deletion of CallBackSpy before web engine page destruction could end up in segmentation fault. Make CallbackSpy safe to be deleted before web engine page. Change-Id: I71a184091c0251c61e3383010e3badfef5cb6124 Reviewed-by: Viktor Engelmann <viktor.engelmann@qt.io>
* Emit loadProgress only between loadStarted and loadFinishedViktor Engelmann2017-08-221-3/+0
| | | | | | | | | | | | | | | | | | | We now track the values we send to the loadProgress signal. We store the last value in a new int WebContentsDelegateQt::m_lastLoadProgress and only send values that are >= m_lastLoadProgress to ensure monotonicity. A value < 0 indicates that no loading is going on. Only on loadStarted it is set to 0 (and a loadProgress is emitted with progress 0) and on loadFinished, it is set to -1 (and a loadProgress is emitted with progress 100). This way, we ensure that you first get a loadStarted signal, at least two loadProgress signals (0 and 100) and a loadFinished signal AND all loadProgress signals come in a monotonous order. Task-number: QTBUG-57839 Task-number: QTBUG-61815 Change-Id: I219d3bffbd5691adb892a11b79647ba9e1ed248e Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* Merge remote-tracking branch 'origin/5.9' into devAllan Sandfeld Jensen2017-08-143-8/+22
|\ | | | | | | | | | | Includes sha1 bump of src/3rdparty to head of 58-based. Change-Id: I37743c3979c1b6fb21f71eab7048e82f7aa5d78b
| * 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>
* | Add HideScrollbars settingPeter Varga2017-08-021-31/+11
| | | | | | | | | | | | | | [ChangeLog][Settings] Added setting to hide scrollbars. Change-Id: I17695bb54a460621358345d303302b93eb342696 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Fix emitting selectionChanged signal for non-user text selection changePeter Varga2017-07-202-35/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes the case when the text selection is triggered by JavaScript. Text selection changes triggered by IME composition text replecement are ignored. Test has been added for mouse selection clipboard: non-user text selection should not update the clipboard. Pulls in Chromium changes: 3deea95 Update TextSelection for non-user initiated events Task-number: QTBUG-53134 Task-number: QTBUG-60381 Change-Id: Ib94f57a2aa61248fba75f595245fb388b9609b6c Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Remove tst_QWebEnginePage::scrollToAnchor auto testPeter Varga2017-07-191-37/+0
| | | | | | | | | | | | | | | | | | | | The tested QWebEnginePage::scrollToAnchor API can't be added to QtWebEngine for now. Currently, Chromium Content API doesn't support to control page scroll. The corresponding feature can be used by URL fragment identifier (#anchor) or JavaScript API (location.href). Change-Id: Ie9c87758d458858ba677d309a548cdced525557b Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Make error message more verbose in QML test test_loadProgressViktor Engelmann2017-07-191-1/+2
| | | | | | | | | | | | | | | | | | | | If the verify(loadProgressMin <= loadProgress) fails, the output was not verbose enough. The additional outputs will tell us more about what caused the test to fail. Task-number: QTBUG-61815 Change-Id: Ia9b45ef4f09a5b25a344e423a7b0c61d5d430542 Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* | Increase timespan for WebViewFindText::test_findTextInterruptedByLoadViktor Engelmann2017-07-192-9/+6
| | | | | | | | | | | | | | | | | | | | The timespans here are 5 seconds by default. These are not enough for this test and make it fail often. We therefore increase the timespans and un-blacklist the test. Task-number: QTBUG-61887 Change-Id: I6acc2dee78fb4d589fd87f7557e0d48725b36f08 Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* | Cleanup support for desktop captureJüri Valdmann2017-07-173-15/+307
| | | | | | | | | | | | | | | | | | - Use feature permissions system instead of hard-coded dialog. - Add QML test for getUserMedia() and extend existing widgets test. Task-number: QTBUG-60832 Change-Id: I533bed5021b3b0ee199b8abc6ddbd516cbd14ff6 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Blacklist flaky WebEngineViewLoadProgress::test_loadProgressViktor Engelmann2017-07-171-0/+3
| | | | | | | | | | | | | | | | | | | | We blacklist the qmltest WebEngineViewLoadProgress::test_loadProgress for now, because it fails often, causing trouble with integrations on the dev branch. Task-number: QTBUG-61815 Change-Id: Ifa74dcc0536075842af935613b5ad8ffcd56cef1 Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* | Merge "Merge remote-tracking branch 'origin/5.9' into dev" into refs/staging/devAllan Sandfeld Jensen2017-07-161-3/+3
|\ \
| * | Merge remote-tracking branch 'origin/5.9' into devKai Koehne2017-07-111-3/+3
| |\| | | | | | | | | | Change-Id: Idffd0c603f7925b8b9b037ce2d63cb13e964c20e
| | * 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>
* | | Update accessibility roles for Chromium 58Peter Varga2017-07-131-1/+24
| | | | | | | | | | | | | | | Change-Id: Id696dd7e56105a67b62b6d045e5af004de19ef79 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | | Blacklist flaky WebViewFindText::test_findTextInterruptedByLoad on LinuxPeter Varga2017-07-131-0/+3
| | | | | | | | | | | | | | | | | | Task-number: QTBUG-61887 Change-Id: I06e01774687935f8751ac4b601826499d6f08531 Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* | | Suppress extra loadStarted signal for data URL with specified base URLPeter Varga2017-07-131-6/+3
|/ / | | | | | | | | | | | | | | | | In these cases two NavigationHandles are created. One for the data URL and another one for the base URL. Suppress the loadStarted signal for data URL one since it is expected to be informed about the base URL. Change-Id: I1a3006cf552a26b8419bd76454e26c0e33c2e898 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Merge remote-tracking branch 'origin/5.9' into devAllan Sandfeld Jensen2017-07-0510-270/+323
|\| | | | | | | Change-Id: Ida702fd5fa05a32d6cd7fc737aa061da74b675a0
| * 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>
* | Update accessibility roles and add auto test for themPeter Varga2017-06-281-0/+146
| | | | | | | | | | Change-Id: I9fc39abd83b394a0241dba3340bc370c8b570481 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Add back input method auto tests and test virtual keyboard for QuickPeter Varga2017-06-225-16/+134
| | | | | | | | | | | | | | | | This patch also fixes the update of the ItemAcceptsInputMethod flag of QQuickWebEngineView regarding to the focused element in the view. Change-Id: Ic6e39401bdc7ca5fab9cc03e878aa731f23fc147 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Fix build of tst_qwebenginespellcheckSzabolcs David2017-06-201-1/+1
| | | | | | | | | | | | | | Both arguments of QCOMPARE have to be of the same type. Change-Id: I34d5131d7a34dd2987307889d28eaa0894d760a2 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
* | Auto test adaptations for Chromium 58Allan Sandfeld Jensen2017-06-148-16/+26
| | | | | | | | | | | | | | | | Adjustments to auto tests to pass with Chromium 58 Change-Id: I7f6905e5344c528c608734bc660cbf43c89d4a63 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
* | Merge remote-tracking branch 'origin/5.9' into devAllan Sandfeld Jensen2017-05-306-6/+114
|\| | | | | | | Change-Id: I962033871a1ef624807a4e1869fe869406aa73f7
| * 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>
* | Merge remote-tracking branch 'origin/5.9' into devLiang Qi2017-05-0216-598/+45322
|\| | | | | | | | | | | | | | | | | | | | | src/3rdparty used sha1 from 5.9, because both 5.9 and dev are using same branch now. Conflicts: src/3rdparty tests/auto/widgets/qwebenginepage/BLACKLIST Change-Id: I16a33e0f91779d1ee3a230070ff4248699c2e1bd
| * 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>
generated by cgit v1.2.3 (git 2.25.1) at 2024-06-19 15:42:22 +0000