summaryrefslogtreecommitdiffstats
path: root/tests/auto/quick
Commit message (Collapse)AuthorAgeFilesLines
* Fix loadStarted signal after an in-page navigationv5.11.0-beta2Peter Varga2018-03-081-0/+26
| | | | | | | | | | | In-page navigations does not trigger loadFinished thus m_lastLoadProgress is not reset to -1. Do not ignore loadStarted signal if m_lastLoadProgress is set to 100 because it can mean a new navigation after an in-page one. Task-number: QTBUG-66869 Change-Id: I2b556fe7e3e097c338bdd7ca47e7384722e7eca9 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Rename protocol() to scheme() in QWERPHPermissionRequestJüri Valdmann2018-03-081-1/+1
| | | | | | | | | Scheme refers to a part of a URL. Protocol refers to a formal standard for communication over a network. Therefore scheme is more accurate here, since registerProtocolHandler only defines simple aliases. Change-Id: I71a1511aa92e41c3e3aca4c0403f54af6d91c563 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* Move quota and registerProtocolHandler permission classes to coreJüri Valdmann2018-03-081-12/+14
| | | | | | | | | | | | | | | | | | | | | | | - Move QWebEngineQuotaPermissionRequest and QWebEngineRegisterProtocolHandlerPermissionRequest to QtWebEngineCore. - Delete QQuickWebEngineQuotaPermissionRequest and QQuickWebEngineRegisterProtocolHandlerPermissionRequest. Miscellaneous cleanup: - Mark QWebEngineQuotaPermissionRequest constructors as internal for QDoc. - Remove superfluous Q_DECLARE_METATYPEs (implied by Q_GADGET). - Remove Q_UNREACHABLE from default constructor. For some reason QML seems to default construct an unused temporary object before throwing it away and copy constructing the actual object. This triggers Q_UNREACHABLE. Change-Id: Icf9f4e34996e4c64aec65b734bcb3bbd22b4dc51 Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* Stabilize WebEngineViewLoadHtml::test_loadProgressAfterLoadHtmlPeter Varga2018-03-081-1/+9
| | | | | | | | | | | The previous test will set the webEngineView.loadProgress to 100 due to a successful load. Therefore, the test can't expect 0 loadProgress at the beginning. Count the loadProgress changes instead of checking the initial value. Change-Id: I8f4727d5b8c7ebf1b7c640e44a9eb3ca5a7fef44 Reviewed-by: Jüri Valdmann <juri.valdmann@qt.io> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Expose actual URL for data URLs instead of the virtual URL in the APIPeter Varga2018-02-211-0/+20
| | | | | | | | | | | Chromium considers the actual URL as "scary" therefore prefers to pass a simpler URL via the WebContents::GetVisibleURL() content API function. For data URLs, use the actual URL instead to keep their anchor information. Task-number: QTBUG-64972 Change-Id: I74db3e5dd22a728656a58e50a4e3fba93b82dae2 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Merge remote-tracking branch 'origin/5.10' into 5.11Allan Sandfeld Jensen2018-02-151-2/+2
|\ | | | | | | Change-Id: I2169baef0a8adb5523d2cfe58896199e967ac1b7
| * Disable failing developer build tests for QemuSami Nurmenniemi2018-02-051-2/+2
| | | | | | | | | | | | Task-number: QTBUG-63152 Change-Id: I7016195c43ba308153d4748a6f9837f1772e762d Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Fix undefined reference to qCompare errorJüri Valdmann2018-02-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | Linking against qtbase 5.9 fails with error: undefined reference to 'bool QTest::qCompare<QString, char [13]>([...])' due to QCOMPARE of QString with C string literal. Change-Id: If4f4312d21d0b1a8ce2577d5f9d3a9ac0731b3a2 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
* | Add support for registerProtocolHandlerJüri Valdmann2018-02-131-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extend initialization of URLRequestContextGetterQt to create a content::ProtocolHandlerRegistry for each content::BrowserContext and add the registry's URL request interceptor to the front of the interceptor chain. Implement methods in WebContentsDelegateQt to add/remove protocol handlers to/from the ProtocolHandlerRegistry. Add permission request signal and classes for core, quick and widgets. Add widgets autotest. Add signal handlers to quicknanobrowser and simplebrowser. Task-number: QTBUG-62783 Change-Id: I808e7eb9a1cb4d7216686deed4895de14fe46310 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Merge "Merge remote-tracking branch 'origin/5.10' into dev" into ↵Allan Sandfeld Jensen2018-02-062-1/+17
|\ \ | | | | | | | | | refs/staging/dev
| * | Merge remote-tracking branch 'origin/5.10' into devAllan Sandfeld Jensen2018-02-022-1/+17
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/core/download_manager_delegate_qt.cpp src/core/download_manager_delegate_qt.h src/core/render_widget_host_view_qt.cpp src/core/web_contents_adapter.cpp src/webengine/api/qquickwebengineview.cpp tests/auto/widgets/qwebenginedownloads/tst_qwebenginedownloads.cpp Change-Id: I2308414ce257ae5bb0fc9f6493aa111a267ff39b
| | * Remove full blacklist of osx 10.11Allan Sandfeld Jensen2018-01-231-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | It was added for a single test failure that was caused by a screen locker on the CI, a problem that has since been solved. This CI configuration is the only one that's running qmltests on osx. Change-Id: Ia6f11649ce2f63000488aa18080c5d149d04eac0 Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
| | * Deselect text on each fourth clickSzabolcs David2018-01-221-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Triple click selects the entire paragraph and it should be deselected on a quadra click. Task-number: QTBUG-65649 Change-Id: I87c9405a202d8b6eacd7c19dbbcb051756c41220 Reviewed-by: Viktor Engelmann <viktor.engelmann@qt.io> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | | Merge remote-tracking branch 'origin/5.11' into devAllan Sandfeld Jensen2018-02-022-0/+69
|\ \ \ | |/ / |/| | | | | Change-Id: I454bb7216b013cc9aa9ee660893fe37034fd50e1
| * | Expose JavascriptCanPaste attributeKai Koehne2018-02-021-0/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Expose an attribute to enable the execCommand("paste") functionality, which is by default disabled due to security concerns. [ChangeLog][General] Added JavaScriptCanPaste attribute that enables JavaScript 'execCommand("paste")'. Task-number: QTBUG-64056 Change-Id: I1a414021e020473dd15946fff91fb103d871e961 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Blacklist unstable test_viewSourceAllan Sandfeld Jensen2018-02-021-0/+3
| | | | | | | | | | | | | | | | | | Change-Id: Ib7562c716faa4221211f2bf53e5f3401b1ee492d Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu> Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | | Support regex format include rules in user scriptsPaul Clark2018-01-262-2/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For user scripts with `@include` and `@exclude` directives in their headers, if the value starts and ends with `/` then assume it is a regular expression to be matched against a URL. This feature has been in the original greasemonkey module since 2011 and currently in webengine regex include rules are being blindly passed to the glob matcher. An alternative to this approach of leaving the bracketing slashes on the values as a kind of in-band signalling (and making the glob escaping conditional) would be to add new members to the UserScriptData struct for holding regex patterns. This commit also applies cleanly to v5.8 and every release after it. [ChangeLog][][UserScript] User script metadata parsing now supports regular expressions in @include and @exclude rules. Task-number: QTBUG-65484 Change-Id: Ie2e483419f6784f995131ffb7d2d5e91c2c55b1d Reviewed-by: Jüri Valdmann <juri.valdmann@qt.io> Reviewed-by: Szabolcs David <davidsz@inf.u-szeged.hu> Reviewed-by: Florian Bruhin <qt-project.org@the-compiler.org> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | | Test adaptations for Chromium 63Allan Sandfeld Jensen2018-01-253-14/+15
| | | | | | | | | | | | | | | Change-Id: I19bf693fb3916a78f4a199e6e095ca58b8fdfa27 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
* | | Adaptations for Chromium 62Allan Sandfeld Jensen2018-01-252-4/+4
|/ / | | | | | | | | Change-Id: I49cd3c419d4dd1180144c3c07bdd9a628ab73caa Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
* | Merge remote-tracking branch 'origin/5.10' into devAllan Sandfeld Jensen2018-01-178-40/+417
|\| | | | | | | | | | | | | | | | | | | Conflicts: .qmake.conf src/webengine/api/qquickwebengineview_p_p.h src/webenginewidgets/api/qwebenginepage_p.h tests/auto/quick/qmltests/data/TestWebEngineView.qml Change-Id: Id2acc92e8d0364bdaaf5a63ea2d2cb9cd533ade3
| * Merge remote-tracking branch 'origin/5.9' into 5.10Allan Sandfeld Jensen2018-01-081-21/+70
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: .qmake.conf src/core/api/qwebengineurlrequestjob.cpp src/core/browser_context_adapter.cpp src/core/renderer/user_resource_controller.cpp src/core/web_engine_context.cpp src/webenginewidgets/api/qwebenginepage.cpp Change-Id: I5278e5e22e1776d42975fc94d70ff8ca4f81fb9a
| | * Fix favicon update from JavaScriptPeter Varga2017-12-131-21/+70
| | | | | | | | | | | | | | | | | | 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>
| * | Add QML autotests for ContextMenuValentin Fokin2017-12-197-20/+348
| | | | | | | | | | | | | | | | | | Change-Id: If18bb163744b9064c0e4d97a9476851b6dee39ad Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
* | | tst_publicapi: Update expectedAPI listJüri Valdmann2018-01-121-4/+148
| | | | | | | | | | | | | | | Change-Id: I756a92288eb243942507ae11382ba55735e593f3 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | | tst_publicapi: Sort expectedAPI list alphabeticallyJüri Valdmann2018-01-121-389/+391
| | | | | | | | | | | | | | | Change-Id: I77739953b10599db4e5f6640b90ea9280ea0e5c3 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | | Merge remote-tracking branch 'origin/5.10' into devAllan Sandfeld Jensen2017-11-297-14/+115
|\| | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/core/web_event_factory.cpp src/core/web_event_factory.h src/webengine/render_widget_host_view_qt_delegate_quick.h Change-Id: Ic43787e2689c81b501ed395a990190eb67d83a2b
| * | Fix QtQuickControls1 mock-delegates paths in qmltests.proValentin Fokin2017-11-271-5/+5
| | | | | | | | | | | | | | | Change-Id: I4743c69202443c3ddb128ae89efc93cd524e775b Reviewed-by: Michal Klocek <michal.klocek@qt.io>
| * | Speculative stabilization of mouseLeave auto testsPeter Varga2017-11-231-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Occasionally the mouse pointer might be moved above the testDiv during the html load and this changes its text. This seems to happen on CI from time to time. Text of testDiv is expected to be empty in the beginning to make sure an explicit mouse move event changes its text. Change-Id: Ib8488b1e2dd7cc80b78cc40adecbf04c12dbec1b Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Blacklist failing test_findTextInterruptedByLoad QML testAlexandru Croitor2017-11-181-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | Continiously failed 5 times on Windows when integrating unrelated patches. Task-number: QTBUG-61887 Change-Id: I8055673e04c3bdfddfcfe05c3905ef5a86e8d1ec Reviewed-by: Michael Brüning <michael.bruning@qt.io>
| * | Merge remote-tracking branch 'origin/5.9' into 5.10Liang Qi2017-11-152-0/+97
| |\| | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/3rdparty tests/auto/quick/qmltests/qmltests.pro tests/auto/widgets/qwebengineview/tst_qwebengineview.cpp Change-Id: I2a1e3a83e29a1d6709f3e98fda3f7b4ab7ab297e
| | * Notify Chromium about leaving viewPeter Varga2017-11-092-0/+97
| | | | | | | | | | | | | | | | | | | | | | | | 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>
| * | Stop preserving aborted navigation entriesJüri Valdmann2017-11-141-5/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Consider the scenario 1. user enters url "http://localhost:8000/" A new navigation entry is created and committed in the NavigationController. 2. user enters url "http://localhost:8000/download.bin" A new navigation entry is created and a download is triggered, but the pending navigation entry in the NavigationController is neither committed nor discarded (since our WebContentsDelegate's ShouldPreserveAbortedURLs() returns true). 3. user enters url "http://localhost:8000/download.bin" At this point the NavigationController will have "http://localhost:8000/" as the committed navigation entry and "http://localhost:8000/download.bin" as the pending entry. NavigateToPendingEntry will see that the user is trying to navigate again to the same URL as the last pending entry and will therefore identify this new navigation as a reload. However Blink interprets 'reload' to mean reloading the last committed entry, i.e. "http://localhost:8000/", and so we end up trying to download "http://localhost:8000/" instead of "http://localhost:8000/download.bin" as the user might have expected. The patch removes the ShouldPreserveAbortedURLs override and relies on the default implementation which always returns false. As a result the pending navigation entry in step 2 above is discarded once the download has been triggered and the unexpected behavior in step 3 is no longer triggered. Removing the override resurrects QTBUG-48995 where, for example, calling QWebEnginePage::setUrl triggers first a urlChanged signal for the *old* URL. The patch adds url and title properties to WebContentsDelegateQt so that property change signals are triggered only if the properties have actually changed. A consequence of this fix is that the first urlChanged signal is delivered directly from the setUrl/load method and not asynchronously once the loading starts (this is also how Chrome's URL bar is updated). Task-number: QTBUG-63388 Change-Id: Icfa300b165e5e56f1fbc8978a00a237c263df183 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
| * | Add 'webengine' prefix to configure features, tests, libraries, etcAlexandru Croitor2017-11-124-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | This is done to make sure there are no conflicts with features in other modules, because they all share a global namespace. Change-Id: I95b3b7fadd8ffc2979ee3aad2234ee543d57c7d8 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | | Merge remote-tracking branch 'origin/5.10' into devLiang Qi2017-11-031-1/+0
|\| | | | | | | | | | | | | | | | | | | | Conflicts: src/3rdparty Change-Id: Ie6b1922db2269e0e0561022162228a7c8609c9ba
| * | Remove pointless inclusions of qtwebengine-config.priAlexandru Croitor2017-11-021-1/+0
| | | | | | | | | | | | | | | | | | | | | Only projects which are built before the actual module need to do that. Change-Id: Ia8b071a627bd39c5df4600877b6873f308cbf690 Reviewed-by: Michael Brüning <michael.bruning@qt.io>
* | | Merge "Merge remote-tracking branch 'origin/5.10' into dev" into ↵Allan Sandfeld Jensen2017-10-281-8/+10
|\ \ \ | | | | | | | | | | | | refs/staging/dev
| * | | Merge remote-tracking branch 'origin/5.10' into devAllan Sandfeld Jensen2017-10-271-8/+10
| |\| | | | | | | | | | | | | | Change-Id: Ibc1c881a67bf088ba6fb044e2553c74c7e9e24ca
| | * | Stabilize tst_javaScriptDialogs similateUserGestureAllan Sandfeld Jensen2017-10-251-8/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was only reading window.mousePressReceived once, which can sometimes be before the click is processed. Change-Id: Ia0a7067a202afa563696708e6df5aa9445335eb8 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
* | | | Un-blacklist DesktopWebEngineViewLinkHovered QML auto testsPeter Varga2017-10-261-6/+0
|/ / / | | | | | | | | | | | | | | | Task-number: QTBUG-55870 Change-Id: Iffde36f9970d2a9d82e7a427e29314a6685a1d67 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | | Merge branch '5.10' into devAllan Sandfeld Jensen2017-10-134-61/+152
|\| | | | | | | | | | | Change-Id: I3a411e4019a5ec2f7d1a967b62a00ad7cf92f6f4
| * | Merge remote-tracking branch 'origin/5.9' into 5.10Allan Sandfeld Jensen2017-09-294-61/+152
| |\| | | | | | | | | | Change-Id: I7094e85a7770303a2ae30baccbc484c04f33600e
| | * 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-202-39/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | | Merge remote-tracking branch 'origin/5.10' into devLiang Qi2017-09-225-6/+12
|\| | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/webengine/api/qquickwebenginesettings_p.h src/webenginewidgets/doc/src/qwebenginesettings_lgpl.qdoc Change-Id: I9cac7059557084a575af02782af886f55a8d404f
| * | Remove WEBENGINE_CONFIG from configureMichal Klocek2017-09-201-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Do not store flags in qmake.cache with WEBENGINE_CONFIG. Use directly qtConfig values insted. This makes configuration more consistent, simplifies handling and avoids passing values from qtConfig to WEBENIGNE_CONFIG, which then were passed to gn. [ChangeLog] Removing WEBENGINE_CONFIG from qtwebengine configure Change-Id: I1a773fb4bff6d67ad75c237d044998051d92ab51 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
| * | Add testsupport to new configure systemMichal Klocek2017-09-194-5/+11
| | | | | | | | | | | | | | | Change-Id: I123ce22ea3a3d8b7b80c67fa322cb817d924f2e0 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | | Merge branch '5.10' into devAllan Sandfeld Jensen2017-09-187-4/+46
|\| | | | | | | | | | | Change-Id: If9b7ec5e67a49aa017b49b75d7ccf6d45820f72e
| * | Adaptations for Chromium 60Alexandru Croitor2017-09-115-9/+36
| | | | | | | | | | | | | | | | | | Change-Id: I536258e22c2ec143f2fd3f1cbda229e0611b6af4 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>