summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch '5.6' into 5.7Allan Sandfeld Jensen2016-11-241-0/+2
|\ | | | | | | | | | | | | | | | | Conflicts: src/core/api/qwebengineurlrequestinfo.cpp src/core/api/qwebengineurlrequestinfo.h src/core/core_gyp_generator.pro Change-Id: I5c78f0c86f6dcd61697148f0729d3d3a2cb2c76f
| * Blacklist doNotSendMouseKeyboardEventsWhenDisabled on WindowsAllan Sandfeld Jensen2016-11-231-0/+2
| | | | | | | | | | | | | | | | It fails occasionally and blocks integration Task-number: QTBUG-57117 Change-Id: I7266ccf4fe7e0905dc31e5e7c085be6f90d0aa03 Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
* | Speculative stabilization of tst_QWebEnginePage::testJSPromptAllan Sandfeld Jensen2016-10-051-0/+3
| | | | | | | | | | | | | | | | Make sure we start in a well-defined state, and is not in process of setting up the blank page. Change-Id: I787db2ef75323bb72a5a6c6fd2e66b268f89b16d Reviewed-by: Michael Brüning <michael.bruning@qt.io>
* | Merge remote-tracking branch 'origin/5.6' into 5.7Liang Qi2016-10-041-3/+2
|\| | | | | | | | | | | | | Conflicts: src/3rdparty Change-Id: I49d99b5ac2cfba243eda1ea705c2a7d71b5542f6
| * Fix flaky tst_QQuickWebEngineView::loadProgress testAlexandru Croitor2016-09-301-3/+2
| | | | | | | | | | Change-Id: I102bda62bfaa16a8c9ff4c4929e6b5a055de3e25 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Fix flaky tst_QQuickWebEngineView::printToPdf test by using QTRY_VERIFYMichael Brüning2016-10-021-4/+2
| | | | | | | | | | | | Task-number: QTBUG-55857 Change-Id: Ieb7fe836e7b03bea28dd97bcad343708a5be2cb5 Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
* | Fix test_scrollPositionAfterReload quick auto testPeter Varga2016-09-131-1/+5
| | | | | | | | | | | | | | | | | | Wait for scroll position change before reload. Task-number: QTBUG-55855 Change-Id: Ia42af1a4a76b2c507f4a88b39a469e3e37184ef7 Reviewed-by: Adam Kallai <kadam@inf.u-szeged.hu> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Blacklist flaky getUserMediaRequest test for all platformsMichael Bruning2016-08-121-1/+1
| | | | | | | | | | | | | | | | It seems to fail on RHEL 7.2 and Windows on integration machines. Task-number: QTBUG-51652 Change-Id: I44fe19e625f3a639e91325ddef29b910ddd3525c Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
* | Merge remote-tracking branch 'origin/5.6' into 5.7Allan Sandfeld Jensen2016-08-026-56/+150
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/3rdparty src/core/resources/resources.gyp src/webengine/doc/src/qtwebengine-overview.qdoc src/webenginewidgets/api/qwebenginepage.cpp src/webenginewidgets/api/qwebenginescriptcollection.cpp src/webenginewidgets/api/qwebenginescriptcollection_p.h tests/auto/widgets/qwebenginepage/BLACKLIST And readded newly in 5.6 enabled tests to the BLACKLIST. Change-Id: I4ab1fc54ebfaaf940df81b0d8d6bdd15cae8b7c4
| * Prevent a crash after having downloaded a fileAndy Shaw2016-07-261-0/+13
| | | | | | | | | | | | | | | | | | | | | | Since the profile owns the QWebEngineDownloadItem then it is likely that it will still be around when the QWebEngineProfile is deleted. As the QWebEngineDownloadItem is a child of the QWebEngineProfile then it will try to delete it when deleting the QWebEngineProfile which means it cannot trigger a function call into QWebEngineProfilePrivate. Change-Id: I51077a7857fb49a6708224a9e9942d17de6f6778 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * Blacklist showHideShow test on LinuxAllan Sandfeld Jensen2016-07-151-1/+1
| | | | | | | | | | | | | | | | It was already skipped on OS X and Windows, but it is also unrealiable on Linux. Change-Id: Ie443c97595979a3cc9d34991a5b0b3f9c69f136c Reviewed-by: Michael Brüning <michael.bruning@qt.io>
| * Propagate the view's screen coordinates on global position changeJoerg Bornemann2016-07-122-0/+116
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Suppose having a QWebEngineView as one of many child widgets in a layout. Resize some child widget such that the position of the QWebEngineView changes (without changing the position of the top-level window). Chromium must be informed of the changed global position, otherwise popups will be opened at the old position. Also see commit 7f941a34, which originally introduced the coordinate propagation for top-level window moves. Task-number: QTBUG-51244 Change-Id: Ieb372e8d7554700d5e8d1e2148ab778094ea3878 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * Remove tst_QWebEnginePage::localURLSchemesJoerg Bornemann2016-07-121-29/+0
| | | | | | | | | | | | | | | | This was a rudimentary test case for QWebSecurityOrigin. This API was replaced in QtWebEngine. Change-Id: If41a7f48906a6b49d5a83c72c69dbfd125804537 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
| * Enable qquickwebengineviewgraphics test in CIJoerg Bornemann2016-07-084-10/+8
| | | | | | | | | | | | | | | | | | This test requires the QtQuick test support API. Exclude it from build if the API isn't available. Blacklist one problematic test case. Remove the - now superfluous - ENABLE_QML_TESTSUPPORT_API checks. Change-Id: I0b36de182628969e1185d685fa098d6140ae8d9c Reviewed-by: Michal Klocek <michal.klocek@qt.io>
| * Clear internal selected text when searchingAlexandru Croitor2016-07-011-18/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | Previously if a selection was made on a web page, and afterwards a find operation is executed, the selection in the web page would be cleared, but the call to selectedText() would still return the old selection. Make sure selectedText() is always cleared, when starting a find operation, as well as when stopping one. Change-Id: If78f0fa1dd836a52184015e749ef5a84b9f784cd Task-number: QTBUG-54071 Reviewed-by: Michael Brüning <michael.bruning@qt.io>
* | Fix flaky qml user script testAlexandru Croitor2016-07-051-3/+3
| | | | | | | | | | Change-Id: I21dfb9146dad44329c490c1c6b32864c70384db3 Reviewed-by: Michael Brüning <michael.bruning@qt.io>
* | Merge remote-tracking branch 'origin/5.6' into 5.7Allan Sandfeld Jensen2016-06-271-7/+24
|\| | | | | | | Change-Id: Ic6686df8f82f710a3441501b7eeaffe69fbcbdf7
| * Fix inputEventForwardingDisabledWhenActiveFocusOnPressDisabled test.Alexandru Croitor2016-06-241-7/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | Test was flaky due to a couple of reasons, mainly: 1) Size of event recording widget did not match web view size, which led to some events not being caught. 2) There were some bogus QHoverEvents being sent by QQuickWindow whenever a QQuickItem is repositioned (can happen when using a tiling window manager on a Linux machine). 3) Usual timing issues. Change-Id: Idd9d4fa6bd65095f98125196025e0ad560025348 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Merge remote-tracking branch 'origin/5.6' into 5.7Allan Sandfeld Jensen2016-06-172-12/+42
|\| | | | | | | Change-Id: If4d8a25f5a2192a658d031252435935cbc675725
| * Stabilize FilePicker QML autotestsAllan Sandfeld Jensen2016-06-161-12/+6
| | | | | | | | | | | | | | | | Switch from timed wait to using tryCompare. Task-number: QTBUG-53777 Change-Id: I845fb8ee7b84641bd23314794512960634423d54 Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
| * Fix translation of multiple pressed mouse buttonsAllan Sandfeld Jensen2016-06-141-0/+36
| | | | | | | | | | | | | | | | | | | | | | | | We were reporting any mouse button event where the left mouse button was held as a left mouse button event. We should instead separate the button changed and the ones held. Also adds test of the translation. Change-Id: I1139d6615d54a036dfe843fdb1d1c90b23b467b5 Reviewed-by: Alexandru Croitor <alexandru.croitor@theqtcompany.com> Reviewed-by: Michael Brüning <michael.bruning@qt.io>
* | Add QML test for scrollPositionAdam Kallai2016-06-164-1/+94
| | | | | | | | | | Change-Id: Ia7541befb43d2b6fdd09b13ca51fc7d09b955964 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
* | Fix the IPC webChannelTransport not being available on reloadJocelyn Turcotte2016-06-131-2/+23
| | | | | | | | | | | | | | | | | | | | | | | | The gin JavaScript binging gets destroyed on page reload after all references to it disappear from the previous document. Do like other gin wrapper and reinstall the transport binding on DidCreateDocumentElement, following how it's done by the MojoBindingsController. Task-number: QTBUG-53411 Change-Id: Ibcd9ef9dbedc5762d4f2210fd81f68e5b9127680 Reviewed-by: Michael Brüning <michael.bruning@qt.io>
* | [Reapply] Blacklist getUserMediaRequest on WindowsAllan Sandfeld Jensen2016-06-131-0/+3
| | | | | | | | | | | | | | | | | | This change was lost in the last merge from 5.6, but is still needed. Task-number: QTBUG-51652 Change-Id: Ic87ebb19ddda7d06ea12bd0826ddf10d404a75d8 Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
* | Merge remote-tracking branch 'origin/5.6' into 5.7Allan Sandfeld Jensen2016-06-112-16/+8
|\| | | | | | | Change-Id: Ib010ede9756fb02992a3276ae7ec90ef1ab56a00
| * Fix tst_QWebEnginePage::setHtmlWithImageResource testAllan Sandfeld Jensen2016-06-102-11/+8
| | | | | | | | | | | | | | | | | | | | | | We have changed qrc resources so that they can be accessed from any security origin. To have a test of local resources we would need a file URL. Task-number: QTBUG-53153 Change-Id: I22b730f07785eacd365e3ec591f5dcbd09b6fc84 Reviewed-by: Alexandru Croitor <alexandru.croitor@theqtcompany.com> Reviewed-by: Michael Brüning <michael.bruning@qt.io>
| * Skip geolocation testAllan Sandfeld Jensen2016-06-101-2/+1
| | | | | | | | | | | | | | This test is unreliable in 5.6. We have a working version in 5.7. Change-Id: I1e4e2abe896e479d8960a3a3b990d80d41b844f1 Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
* | Fix and enable tst_QWebEnginePage::scrollPositionAllan Sandfeld Jensen2016-06-091-14/+13
| | | | | | | | | | | | | | | | With scroll position readers available in Qt 5.7 we can now perform this test. Change-Id: I3aad92f55cd075fa211a9c4abe56fb0c2198af7d Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
* | Remove tst_QWebEnginePage::renderHints()Allan Sandfeld Jensen2016-06-011-137/+0
| | | | | | | | | | | | | | | | Since we do not use QPainter inside the webview, this property is not going to make sense again. Change-Id: I6d2948699deb574ca7f8e850efe3bddbe61713af Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
* | Merge "Merge remote-tracking branch 'origin/5.6' into 5.7" into refs/staging/5.7Allan Sandfeld Jensen2016-05-314-46/+291
|\ \
| * | Merge remote-tracking branch 'origin/5.6' into 5.7Allan Sandfeld Jensen2016-05-314-46/+291
| |\| | | | | | | | | | Change-Id: I561c00b3a844ab493a5bf0148a5923662842cf5d
| | * Remove tests for the javaScriptWindowObjectCleared signalJoerg Bornemann2016-05-301-45/+0
| | | | | | | | | | | | | | | | | | | | | | | | We have no equivalent of QWebFrame::javaScriptWindowObjectCleared and won't need it. Change-Id: I998e0412256887414c125df9ab994216dee22dc9 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | * Fix event forwarding when activeFocusOnPress is false.Alexandru Croitor2016-05-262-1/+109
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a QQuickWebEngineView does not have focus, and activeFocusOnPress is set to false, a user can still partially interact with the view. For instance hovering the mouse over a link would change the cursor, a link can be clicked to go to a different page. Clicking on a text input field would focus the text field, but entering characters will not be possible, because the view does not have QtQuick keyboard focus, and clicking does not give the focus (because activeFocusOnPress is set to false) and this leads to confusing behavior. Thus the fix is to make sure no mouse / keyboard events are forwarded to Chromium if the view has no focus, and activeFocusOnPress is set to false, in order to maintain a more user-friendly behavior. Manually forcing the focus via some user-provided method that calls forceActiveFocus() would allow further proper interaction. Change-Id: I72c3ff69438972b9a93ee2d415fa1d4b44b86cd9 Reviewed-by: Michael Brüning <michael.bruning@theqtcompany.com>
| | * Disabled WebEngine views should not receive focus.Alexandru Croitor2016-05-262-0/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently if a QWebEngineView or a QQuickWebEngineView is disabled using setEnabled(false), after loading a web page, the views are automatically focused, and a user might see a blinking caret in an html input for example, even though the user can't interact with it. Fix consists in not calling the Focus() method whenever a view is disabled. Change-Id: I1014fb5898a5ddf01a4e9b14c3eaf5d4006e5131 Task-number: QTBUG-53159 Reviewed-by: Michal Klocek <michal.klocek@theqtcompany.com>
| | * Stop disabled QWebEngineView widget from processing input events.Alexandru Croitor2016-05-261-0/+120
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a QWebEngineView is disabled, input events are still forwarded to Chromium, thus allowing the user to interact with a web page. Fix consists in stopping the forwarding of input events in the generic event handler, just like QWidget::event() does. Change-Id: Ie822d1f3d640840569a282223d76749686cf3419 Reviewed-by: Michal Klocek <michal.klocek@theqtcompany.com>
* | | Fix widget geolocation testsAllan Sandfeld Jensen2016-05-316-139/+7
| | | | | | | | | | | | | | | | | | | | | | | | Avoid having a test location plugin, and have the tests allow failure if no position service is available. Change-Id: Ib2d4d6c5a269077b70b6ea31e67a8e3eab62f98a Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
* | | Update public API listAdam Kallai2016-05-311-2/+1
| | | | | | | | | | | | | | | Change-Id: I348d135949aaca4695352b1d7a27187d891ddd56 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | | Fix QML geolocation testAllan Sandfeld Jensen2016-05-313-3/+4
|/ / | | | | | | | | | | | | | | | | The test had a race condition and never waited for success or failure. In addition it didn't handle the case where there is no location service available. Change-Id: Icac02d0cc6396f31fed83b8b77b691ba872ac2f0 Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
* | Unskip test_errorPageEnabled in tst_favicon.qml testAdam Kallai2016-05-251-8/+4
| | | | | | | | | | | | | | | | | | - Unskip this test becuse the releted issue is already fixed. - It is worth to restore the initial page before each test functions, not only when there was icon url. Change-Id: I2a71e026ab3ffabb8de191b7296a67ac44479934 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Update test_promptSzabolcs David2016-05-221-2/+1
| | | | | | | | | | | | | | | | | | | | If the user clicks "cancel" the prompt() method returns null. This is the expected behavior in browsers, see the specification: https://dev.w3.org/html5/spec-preview/user-prompts.html Change-Id: I2089b7fe551d454904bc1b5b07a28e5e49ba8dfd Task-number: QTBUG-51749 Reviewed-by: Michael Brüning <michael.bruning@theqtcompany.com>
* | Merge remote-tracking branch 'origin/5.6' into 5.7Liang Qi2016-05-204-0/+65
|\| | | | | | | | | | | | | | | | | | | Conflicts: src/3rdparty src/core/browser_context_adapter.cpp src/core/web_contents_adapter.cpp tests/auto/widgets/qwebenginepage/BLACKLIST Change-Id: I45ee0a33f6316f585555d58fede8072fe514aecf
| * Merge "Merge remote-tracking branch 'origin/5.6.1' into 5.6" into ↵Liang Qi2016-05-191-0/+3
| |\ | | | | | | | | | refs/staging/5.6
| | * Blacklist flaky QWebEnginePage::setHtmlWithImageResource test.Alexandru Croitor2016-05-101-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | It used to be an expected fail, then it seemed to work in 5.6, and for some reason it fails again in 5.6.1. Change-Id: I95161e001c3b8af3ea48fc284e448a2d8853e108 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Fix pasting images to web pagesJoerg Bornemann2016-05-193-0/+62
| |/ | | | | | | | | | | | | | | | | WebKit expects pasted images to have the MIME type image/png (see code and comment in DataObjectItem::getAsFile()). Task-number: QTBUG-53409 Change-Id: I2b0c1244d309687ad190db26c5b00718ed0c4258 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Better handle failure on read in custom url handlersAllan Sandfeld Jensen2016-05-131-1/+53
| | | | | | | | | | | | | | | | | | Chromium 49 expects url request failures to be reported as a response to either start or on read. So we should report failures during read and make sure we can handle QIODevice calling fail on the job. Change-Id: Id8f619d3cb6f187c99da580c6923130a95ca91ee Reviewed-by: Michael Brüning <michael.bruning@theqtcompany.com>
* | Add spellcheck autotestMichal Klocek2016-05-067-0/+262
| | | | | | | | | | | | | | Skip test on osx for now. Change-Id: I8f748d00da54fd5b7217b08a18e7a58d31cb898c Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>
* | Combine candidate icons for a page into a single iconPeter Varga2016-05-0410-18/+104
| | | | | | | | | | | | | | | | | | | | | | | | For the Widget API the QIcon returned by QWebEnginePage::icon() function contains all the candidate icons for the current page. For the Quick API the QQuickWebEngineFaviconProvider provides the best quality icon for the requested size from the candidates. Task-number: QTBUG-51179 Change-Id: I42b8427f957e2f2fc745dd0111bedcc71b577216 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
* | Cleanup license leftoversMichal Klocek2016-05-037-92/+138
| | | | | | | | | | Change-Id: Ie0ecb1808c6b6a689ef29d7fa4831ca0a2e92721 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
* | Revert "Fix unexpected passes in tst_QWebEnginePage."Alexandru Croitor2016-05-031-0/+2
| | | | | | | | | | | | | | | | The test passes in 5.6, but not in 5.7. This reverts commit fe84829aeb3ba19b74935198dbcbd6f5c67915b3. Change-Id: I204c2d80b80e7c33d573963fd4366e4613ceb789 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
* | Merge "Merge remote-tracking branch 'origin/5.6' into 5.7" into refs/staging/5.7Allan Sandfeld Jensen2016-05-031-2/+0
|\ \