summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Reimplement QML test cases of tst_loadUrl auto testv5.8.0-beta1Peter Varga2016-10-263-118/+209
| | | | | | | | | | | | | | | | | Get rid of some potential flaky issues: - Fix typo in redirect.html - Do not check URL before wait the result of the load. As a corner case the load might be finished earlier than the comparison. - Get rid of unnecessary signal handlers for unrelated test cases. The superfluous operations might have enough overhead to cause a comparison to be timed out. - Separate data URL test case and blacklist it due a to known issue (QTBUG-56661) Task-number: QTBUG-56180 Task-number: QTBUG-56661 Change-Id: I76cec4e761be646c2f60588a34bae806cf558b3a Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* Add quick and widget auto tests for keyboard event handlingPeter Varga2016-10-256-0/+465
| | | | | | Task-number: QTBUG-56242 Change-Id: Ib6689d6f183532502382c86f92fdbdf27dd6c37c Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
* simplify setup of exclusive buildsOswald Buddenhagen2016-10-212-6/+2
| | | | | | | | | the simulator_and_device CONFIG flag is now gone. this also implies that we can set the debug_and_release and build_all flags based on the same condition. Change-Id: Ib16d05213e738a961c9e0b232829985b8ad64a50 Reviewed-by: Jake Petroules <jake.petroules@qt.io>
* Improve configure outputAllan Sandfeld Jensen2016-10-191-27/+58
| | | | | | | | Format the output slightly more, and report the list of optional bundled and system libraries used. Change-Id: I1fd06a5a83fb01e93dfd29236774a6e320530a6e Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
* Revert "Revert "Lower minimum OS X requirements back to 10.9""Liang Qi2016-10-192-7/+7
| | | | | | | | | fdf3a40 in qtwebview fixed the macOS 10.9 build issue. This reverts commit 41ba9b70525139fb52d00ba81eb8a53293ad71db. Change-Id: I178ba785500bf7d15f1b0b8319fe520d07f0388c Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
* Fix find after navigationPeter Varga2016-10-184-3/+20
| | | | | | | | | Update Chromium for the actual fix. Add quick auto test for the issue and unskip WebViewFindText::testfindTextManyMatches. Task-number: QTBUG-56373 Change-Id: Id70291e5231f572c85b0a7c1ec1b6facd161115a Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* Fix spellchecking for macOSAlexandru Croitor2016-10-1711-30/+170
| | | | | | | | | | | | The change fixes spellchecking to work on macOS. A new WebEngine configure option is available to allow spellchecking on macOS to use either Hunspell like the other platforms, or the native spellchecker that comes with the OS. The default is to use Hunspell. Task-number: QTBUG-53135 Change-Id: I3e45b2e0d728b1bf2659c35f3d0a042b0ecd6239 Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io> Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* Speculative fix of title check in quick auto testsPeter Varga2016-10-173-20/+14
| | | | | | | | | Title might be changed after a successful page load too. The order is not guaranteed. Implement wait for the expected title change. Task-number: QTBUG-56180 Change-Id: If8c529e6246fe9e8c93e41be84c8934c8ed8d729 Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* Revert "Lower minimum OS X requirements back to 10.9"Jani Heikkinen2016-10-172-7/+7
| | | | | | | | | | | This reverts commit 6e9779ac2c5b0d941bb415fb00e0f193d685457c. It seems there is an issue with 10.9 & Qt 5.8, see QTBUG-56563. Reverting this to be able to integrate qt5.git in '5.8' Change-Id: Ide35be544e81d31c3bcbd525bb43d3a864ce116c Reviewed-by: Frederik Gladhorn <frederik.gladhorn@qt.io> Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
* Change linking information for qwebengine_convert_dictMichal Klocek2016-10-173-3/+4
| | | | | | | | | | | | | | Use convert_dict_linking.pri instead of QtWebEngineCore_linking.pri for qwebengine_convert_dict linking step. New linking information contains only required dependencies, so linker should have much less work to resolve all the symbols. The target change to executable is also required to generate convert_dict_linking.pri Task-number: QTBUG-53135 Change-Id: I1432207bd1853e762d5d8291ce61027592fd0161 Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
* Doc: Fix grouping Qt WebEngine modulesLeena Miettinen2016-10-178-5/+55
| | | | | | | Add a page for exposed Qt WebEngine module C++ classes. Change-Id: I4fd15e9d5c5c1b8c51a836ad5a036083adaae328 Reviewed-by: Topi Reiniö <topi.reinio@theqtcompany.com>
* Refactor tst_keyboardModifierMapping.qml quick auto testPeter Varga2016-10-151-29/+13
| | | | | | | Simplify code and remove unnecessary wait calls. Change-Id: If3a783a2a4f97e84de993b9394f5cfa0df948ec2 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Speculative fix of tst_unhandledKeyEventPropagation.qml quick auto testPeter Varga2016-10-152-8/+3
| | | | | | | | This fix implements an attempt to remove wait from the test. Task-number: QTBUG-56180 Change-Id: I2bfeb14ebc9451a1237178aa3c97bd1b27e9485c Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Speculative stabilization of test_focusOnNavigation quick auto testPeter Varga2016-10-152-13/+19
| | | | | | | Reimplement setting focus on a HTML element in JavaScript. Change-Id: I1db53bcb526fd52df1fa23527089f8186dfd0ea3 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Blacklist flaky QML NewViewRequest::test_jsWindowOpenPeter Varga2016-10-141-0/+3
| | | | | | Task-number: QTBUG-56540 Change-Id: Ic48883f94d72acf9a965fe8687abb20205a4703b Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Blacklist flaky QML WebViewFindText::test_findTextManyMatchesPeter Varga2016-10-141-0/+3
| | | | | | Task-number: QTBUG-56373 Change-Id: I599c02a12a0dfb35bb8b3d1d3abbede1c6498f9e Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Lower minimum OS X requirements back to 10.9Alexandru Croitor2016-10-122-7/+7
| | | | | | | | This reverts commit c8851dd1a77e730bc6a3c17b7c75b1a4c6b41f53, and allows building on OS X 10.9 with OS X 10.10 SDK. Change-Id: Id59b08424165272fd0d35418eef45a03fce731fb Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* QML plugin: Introduce message helper function for uncreatable typesFriedemann Kleint2016-10-111-10/+16
| | | | | | | | Reduce the number of messages to be translated to reduce the translation workload. Change-Id: Ifa76b02ed6474522eea3b9f74434ead95a6b0d0c Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Ignore generated QML/JS code cache filesPeter Varga2016-10-101-0/+4
| | | | | Change-Id: I4d781ba23e4c46a8dd515edcf01e253c208170b3 Reviewed-by: Simon Hausmann <simon.hausmann@qt.io>
* demobrowser: Fix compilation warningPeter Varga2016-10-071-0/+2
| | | | | Change-Id: I283365c72d351e62bc749cbc60e06472e52255ec Reviewed-by: Michael Brüning <michael.bruning@qt.io>
* Add Quick auto test for QQuickWebEngineNewViewRequestPeter Varga2016-10-072-0/+137
| | | | | | | Task-number: QTBUG-55765 Change-Id: I5576876191fe1c819aa85f2e312391126c904201 Reviewed-by: Christophe Chapuis <chris.chapuis@gmail.com> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Doc: Do not advise to overwrite QWidget::contextMenuEvent()Kai Koehne2016-10-061-10/+7
| | | | | | | | | | | | There's also QWidget::setContextMenuPolicy and QWidget::customContextMenuRequested, which is arguably a better approach. Anyhow, this isn't specific to QWebEngineView, so let's leave it at that. Finally, there's no need to call triggerPageAction if you add an action to a menu. Change-Id: I0c84038746a78316ed6b90a98b68fa5eb95fbad5 Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io> Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* Merge remote-tracking branch 'origin/5.7' into 5.8Liang Qi2016-10-0614-25/+43
|\ | | | | | | | | | | | | | | Conflicts: src/3rdparty src/webengine/api/qquickwebenginesettings.cpp Change-Id: Ie4abadb076c51d070cbb05f29b9ebcd9705933ad
| * 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-045-5/+19
| |\ | | | | | | | | | | | | | | | | | | 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>
| | * Doc: Fix initialize() documentationMichal Klocek2016-09-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | We do not share context between "processes", but between "threads" Change-Id: I20f558f913c1f19fc469ade6faab45762f42d528 Reviewed-by: Kai Koehne <kai.koehne@qt.io>
| | * Update ChromiumAllan Sandfeld Jensen2016-09-271-0/+0
| | | | | | | | | | | | | | | | | | | | | Pulls in changes to build debug+release correctly and build with Xcode 8 Change-Id: I38d6bd50048b332939416b0fae3f20978995d789 Reviewed-by: Michael Brüning <michael.bruning@qt.io>
| | * Propagate the view's screen coordinates on global position change in QuickPeter Varga2016-09-272-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | Based on widget solution: 1e83a2d1b61b13323163dfe8cac64dad397cb202 Task-number: QTBUG-55650 Change-Id: I6df45e7e018fa201c50fe81e8679c36f97ddeb1e Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | * Include neon support for any -fpu=.*neon.*Donald Carr2016-09-271-1/+1
| | | | | | | | | | | | | | | | | | Change-Id: I9c571c823b6fa2411e003799f7ff8627ef6e1d5e Reviewed-by: Kai Koehne <kai.koehne@qt.io> 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>
| * | Update ChromiumAllan Sandfeld Jensen2016-10-011-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | Pulls in changes to build debug+release correctly, build with Xcode 8, and added support for running on macOS 10.12 Sierra. Change-Id: I17668229c1b90af9dca26fb48f6cf945ef0ed114 Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
| * | Merge "Merge remote-tracking branch 'origin/5.6' into 5.7" into refs/staging/5.7Allan Sandfeld Jensen2016-09-255-12/+14
| |\ \
| | * | Merge remote-tracking branch 'origin/5.6' into 5.7Liang Qi2016-09-245-12/+14
| | |\| | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/webengine/api/qquickwebenginesettings.cpp Change-Id: Ib7e2a7558c685410c49c455218d3b9c30f32358f
| | | * Merge remote-tracking branch 'origin/5.6.2' into 5.6Liang Qi2016-09-233-9/+9
| | | |\ | | | | | | | | | | | | | | | Change-Id: I3e152ee134ed38628d1bf9571df58f469b29e74b
| | | | * Fix crash when using openIn on newly created viewv5.6.2Christophe Chapuis2016-09-231-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a link triggers a NewViewRequest, and the latter creates a new window with a new WebEngineView inside, the "adapter" pointer can still be NULL when the adoptWebContents is called by openIn. Therefore is it necessary to test the adapter pointer before using it. (cherry picked from commit bfc2683ab4ca3f56a2effd1b2f7e68850acb72ba) Task-number: QTBUG-55765 Change-Id: Iaa7cb4e8c7780a2e3f1a8c85b7b5da0ec541b2f3 Reviewed-by: Jani Heikkinen <jani.heikkinen@qt.io>
| | | | * Doc: Fix types of \qmlpropertyKai Koehne2016-09-193-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ie355f72a0bf575e66ab465fb2fcada0430ed8c7c Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io>
| | | * | Use canonical include paths for public headersKai Koehne2016-09-221-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Do not rely on "include/QtWebEngineWidgets" or "include/QtCore" being in the default include path. Task-number: QTBUG-56107 Change-Id: I9535f723814efdef7bf8290a891978aabfa899bb Reviewed-by: Michael Brüning <michael.bruning@qt.io>
| | | * | Fix crash when using openIn on newly created viewChristophe Chapuis2016-09-201-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a link triggers a NewViewRequest, and the latter creates a new window with a new WebEngineView inside, the "adapter" pointer can still be NULL when the adoptWebContents is called by openIn. Therefore is it necessary to test the adapter pointer before using it. Task-number: QTBUG-55765 Change-Id: Ia1a299dd65c229705462c9444b81d7b26567fc09 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | | * | Fix dangling pointer problemViktor Engelmann2016-09-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Turn the raw pointer ResourceDispatcherHostLoginDelegateQt::m_authInfo into a scoped_refptr, to prevent chromium from freeing the memory, which caused this pointer to dangle and SEGFAULT upon later usage. Task-number: QTBUG-55828 Change-Id: Ib57e89ca042a4494e2ab77ea10328495e6fc1431 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu> Reviewed-by: Michael Brüning <michael.bruning@qt.io>
| * | | | Disable local database when local storage is disabledAllan Sandfeld Jensen2016-09-251-0/+1
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Ties the feature switch for Indexed DB to the same as local storage. Change-Id: I9ea8992cc00097a8f7bd86236f8cd43008566d2b Reviewed-by: Michael Brüning <michael.bruning@qt.io>
| * | | Use canonical include paths for public headersKai Koehne2016-09-232-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Do not rely on "include/QtWebEngineWidgets" or "include/QtCore" being in the default include path. Task-number: QTBUG-56107 Change-Id: I1a1eb7baf6d3166239eadc5b0b9863704e87888c Reviewed-by: Michael Brüning <michael.bruning@qt.io>
| * | | Update ChromiumAllan Sandfeld Jensen2016-09-231-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Pulls in the second batch of cherry-picked security patches from the 53 release. Change-Id: Ibd5a596648cb8caa47b9df45e019d198eb494d9d Reviewed-by: Michael Brüning <michael.bruning@qt.io>
* | | | Generate domKey for printable charactersPeter Varga2016-10-051-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes the key event when spacebar is pressed (for example above a checkbox). Task-number: QTBUG-56283 Change-Id: I76bcc9a66abcef3a9e5ac10cfa53d60fce4c9179 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | | | Doc: Sanitize WebEngineView::newViewRequested code snippetKai Koehne2016-10-051-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use ; in JavaScript snippets. Also, use = function(x) notation for slots that get an argument. Change-Id: Iba6ffe94f882cae2b31022f4f80482ea6c922fde Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu> Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | | | Update plugins.qmltypesKai Koehne2016-10-042-26/+232
| | | | | | | | | | | | | | | | | | | | | | | | Task-number: QTBUG-56283 Change-Id: If92230e245d0257b11e59fb1d5fbb86f8af88f55 Reviewed-by: Thomas Hartmann <Thomas.Hartmann@theqtcompany.com>
* | | | Fix keyboard event handlingPeter Varga2016-10-041-0/+553
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Set domKey and domCode property of NativeWebKeyboardEvent. Blink uses this new interface for keyboard event handling: https://codereview.chromium.org/2045603002 Task-number: QTBUG-56242 Change-Id: I33f6f17087629f5021874cacd3e75295bbd97870 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | | | MarkDownEditor: Hide context menuKai Koehne2016-10-042-2/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Most actions do not make sense. A custom context menu might be good for text actions, but would require reimplementing QWebEngineView. Change-Id: Idf94939c16469e1dcee96d9ecfd0eb5309589fdd Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io>
* | | | Doc: Do not call Greasemonkey attributes 'extensions'Kai Koehne2016-10-044-6/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The term stems from Chromium extensions, which are however overloaded. Sadly there's no good page we can link to inside Chrome/Chromium for the attributes we support ... Change-Id: Ife7fcdba7bdfdbd4b7248b247c4baa188a44867a Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io>
* | | | Clean up qt quick controls 1 dialogsMichal Klocek2016-10-0414-55/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Based on code review for qqc2 integration, apply cleanup changes to qqc1. Change-Id: I9752ce944385a529a1d999a5419638868e18d799 Reviewed-by: Kai Koehne <kai.koehne@qt.io>