summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge remote-tracking branch 'origin/5.8' into devMichael Brüning2016-12-0758-530/+931
|\ | | | | | | | | | | | | Conflicts: src/3rdparty Change-Id: Ie7b7c469aa24716816a23b8fe7a8df9f477a9f67
| * Report size to UrlRequestJob for qrcAllan Sandfeld Jensen2016-12-061-0/+1
| | | | | | | | | | | | | | | | We already added this custom url jobs, but it was missing for qrc. Task-number: QTBUG-57447 Change-Id: I5f832b083bdadc86a4bd0124f92a970d2df8be27 Reviewed-by: Milian Wolff <milian.wolff@kdab.com>
| * Ensure the deletion of testing pagesSzabolcs David2016-12-062-142/+108
| | | | | | | | | | | | | | | | | | If a test fails it may trigger an assert in debug mode because the delete wasn't performed. Move QWebEngine* objects to the stack or use QScopedPointers to avoid these assertion fails. Change-Id: Ic44c3fddc7cd9cf57a0e2d1c57e0a6fd99033e02 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
| * Merge remote-tracking branch 'origin/5.8.0' into 5.8Michael Brüning2016-12-056-24/+135
| |\ | | | | | | | | | Change-Id: Ifa26409c722bd4eb373b9ced55f5a58132702ad5
| | * Add changes file for 5.8.0Allan Sandfeld Jensen2016-12-021-0/+95
| | | | | | | | | | | | | | | | | | | | | | | | Listing important changes and bug-fixes. Change-Id: Ief57951009b59dc0c0b861315b2dfa35543dd960 Reviewed-by: Michael Brüning <michael.bruning@qt.io> Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io>
| | * Fix build when PDF support is disabledPeter Varga2016-12-012-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | Wrap unused parameters. Change-Id: Iac550783a55b3fc585db6f7b8241aa2ae89fdf2e Reviewed-by: Michael Brüning <michael.bruning@qt.io> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | * Fix build with -no-opengl configured QtMichael Brüning2016-12-011-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | Commit e2541ebdfadea0fe43baac748cfa9e07f3b57215 introduced an unguarded reference to QOpenGLContext. Guard it with the appropriate flags. Task-number: QTBUG-57374 Change-Id: Ic9f0cc3c99aabd52813d8b828f1a51ca6871d4b4 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | * Add environment variable to set chromium flagsAllan Sandfeld Jensen2016-12-011-21/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is not always possible to pass random flags to any QtWebEngine application, so we should allow another way of setting Chromium flags. Among other things this makes it easy to set flags when running auto tests. Change-Id: I7ffb3bf8c9ceafc7d6b1bd2ba135643a687e480f Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
| | * Doc: update platform notesMichal Klocek2016-11-291-2/+2
| | | | | | | | | | | | | | | Change-Id: I2482a570f7c22b5cff5ce217dbece1746441961e Reviewed-by: Kai Koehne <kai.koehne@qt.io>
| * | Wait for loadVisuallyCommitted signal with SignalSpyPeter Varga2016-11-293-33/+31
| | | | | | | | | | | | | | | Change-Id: Ic200807246c5feb8e1f33215cbc6194b03765359 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Port autotests from own waitForSignal() to QSignalSpy::wait()David Faure2016-11-299-116/+116
| | | | | | | | | | | | | | | | | | | | | | | | I added QSignalSpy::wait() in Qt 5.0 exactly for this purpose. Change-Id: I895a92f5f7e4e8554e00f6668e6973cc2c903adf Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | Fix keyboard focus of RWHVQDW after popupPeter Varga2016-11-293-1/+67
| |/ | | | | | | | | | | Task-number: QTBUG-56652 Change-Id: I1a6655587a9104dd817332e2eb5f886c057d8f64 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * Fix DesktopWebEngineViewLinkHovered QML testsPeter Varga2016-11-252-6/+22
| | | | | | | | | | | | | | | | | | | | The linkHovered signal can be emitted only if the page is rendered. Waiting for rendering and for the UpdateTargetURL event may exceed the time limit so add wait for the rendering too. Task-number: QTBUG-55870 Change-Id: I3aa8e323a3bf8ff3f4a85c3de9efd3ab0f459f62 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * Rework and fix loadVisuallyCommitted signalPeter Varga2016-11-258-26/+31
| | | | | | | | | | | | | | | | | | | | | | - Remove RenderViewObserverHostQt_DidFirstVisuallyNonEmptyLayout message and use WebContentsObserver::DidFirstVisuallyNonEmptyPaint instead. - The order of the DidFirstVisuallyNonEmptyPaint and OnSwapCompositorFrame events is not guaranteed. Check for both events to do not miss to send any loadVisuallyCommitted signal. Change-Id: Ic733b3e9a6fae64b1d827e8e07514f180273cf8c Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * Fix multiple copy handling while printingMichael Brüning2016-11-251-1/+5
| | | | | | | | | | | | | | | | | | | | Both the printer code and the webengine printing code were producing the number of requested copies. So if n copies were requested, n*n copies of the document were printed. Task-number: QTBUG-56722 Change-Id: I34bce2d61041e1234139139dcd70b7b3d1c7cd7b Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * Move the QPrinter and QtWidgets related code out of the PDFium wrapperMichael Brüning2016-11-2520-175/+239
| | | | | | | | | | | | | | | | | | | | | | This moves the actual printing using QPrinter to the WebEngineWidgets part of the API. The printsupport module depends on the widgets module and therefore QtWebEngineCore also had a dependency to widgets. This is removed by this change. Change-Id: If6e5745709a59de18f2123b930cbe6e64390c867 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
| * Merge remote-tracking branch 'origin/5.7' into 5.8Allan Sandfeld Jensen2016-11-2518-23/+193
| |\ | | | | | | | | | | | | | | | | | | Conflicts: src/3rdparty Change-Id: Ib9c9eca457c1c42dab948e6cb56d44b57d5da32a
| | * Increase timeout in loadSignalsOrder testsAllan Sandfeld Jensen2016-11-241-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use a standard QTRY_VERIFY. Apparently launching the web-process might sometimes take more than 500ms on the CI under load. Task-number: QTBUG-57119 Change-Id: Icb7706b5aed3cf72f026af6d1d2f9430c4942a2a Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu> Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
| | * Merge branch '5.6' into 5.7Allan Sandfeld Jensen2016-11-2411-13/+111
| | |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/core/api/qwebengineurlrequestinfo.cpp src/core/api/qwebengineurlrequestinfo.h src/core/core_gyp_generator.pro Change-Id: I5c78f0c86f6dcd61697148f0729d3d3a2cb2c76f
| | | * ResourceType ABI/API fixupAllan Sandfeld Jensen2016-11-234-10/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ChangeLog][Important Changes] The enum value ResourceTypeUnknown has changed value since there was a mismatch between 5.6 and 5.7+ definitions. In general any unknown ResourceType value should handled as unknown for forward compatibility, since more types are and can be added in later Qt versions. Change-Id: I0a9f8a2129d4549deeae01e199f432fbbf1bbb9e Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io> Reviewed-by: Kai Koehne <kai.koehne@qt.io>
| | | * 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>
| | | * Fix IPC message loggingPeter Varga2016-11-222-0/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Register Qt IPC messages for logging. The usage of content::RegisterIPCLogger function adds the content_common.content_ipc_logging.o to the link dependency. Thus it will register the Chromium Content IPC messages too. Task-number: QTBUG-57224 Change-Id: I2c45691feb22a34f6074940cc35b8a4ba7804370 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | | * Check if the .git directory exists before invoking gitAndy Shaw2016-11-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If .git does not exist then this is not from a git build and therefore there is no reason to invoke git (which might not even exist on the system anyway). This prevents an error appearing due to trying to invoke git on Windows when it does not exist. Change-Id: I8c0b5b237cfdaffdbb33efdd16cf20cd1560f1a1 Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
| | | * Fix select tag interaction when the web view is inside a modal dialogAlexandru Croitor2016-11-103-2/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously when a QWebEngineView was inside a modal QDialog, trying to click on a select tag option did not properly select the option. It either focused the new option without closing the popup, or didn't focus it at all. Fix consists in making sure the newly created popup QWindow and RenderWidgetHostViewQtDelegateWidget are marked as children of the QWebEngineView, so that they are considered part of the current modal session by the OS, thus allowing user interaction with them. Because the ownership of the delegate widget should still be retained by its respective RenderWidgetHostViewQt instance, the QObject parent of the delegate is unset before the parent is destroyed. Also to make it work on macOS, the window attribute has to be set to Qt::Tool instead of Qt::ToolTip. Change-Id: I56d6f446254a624428a0c661ac3c49eb409c931e Task-number: QTBUG-54836 Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Michael Brüning <michael.bruning@qt.io>
| | * | Merge remote-tracking branch 'origin/5.7.1' into 5.7Allan Sandfeld Jensen2016-11-215-9/+78
| | |\ \ | | | | | | | | | | | | | | | Change-Id: Ifcf7423fa8d892ddad595300e33a09c7f63f5e6e
| | | * | Update ChromiumAllan Sandfeld Jensen2016-11-092-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Pulls in the security fix from Chrome 54.0.2840.87 Change-Id: I70064927cba01b7978742951ba0636b780d9eb68 Reviewed-by: Michael Brüning <michael.bruning@qt.io> Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
| | | * | Update changes file for 5.7.1Michael Brüning2016-10-271-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ib683ecda4c4bb2d58f36ad92f703d58f00e9d445 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
| | | * | Do not call QOpenGLContext::openGLModuleType() from MainDllMichal Klocek2016-10-261-7/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In case of dynamic OpenGL on Windows this ends on calling QWindowsIntegration::staticOpenGLContext() from MainDll which in case of angle will end up badly. Add warning message when webengine is loaded from plugin and context is not initialized. Task-number: QTBUG-52201 Task-number: QTBUG-55501 Task-number: QTBUG-56020 Change-Id: I03570cad5f686c4a63910c71136bf3eb9499f223 Reviewed-by: Kai Koehne <kai.koehne@qt.io>
| | | * | Update ChromiumAllan Sandfeld Jensen2016-10-172-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Pulls in security fixes from the Chromium 54.0.2840.59 release, Changes included: - Fix renderer crash on null family strings - [Backport] Blink-in-JS should not run micro tasks - [Backport] Disallow reentrance of FrameView::updateLifecyclePhasesInternal() - [Backport] Check CORS policy on redirect in TextTrackLoader - [Backport] Keep top controls visible if SHOW is called right after HIDE. - [Backport] Merge to 2840 "[DevTools] Avoid current_ and pending_ being the same host in RenderFrameDevToolsAgentHost." - [Backport] Enable do not allow default action for untrusted events. - [Backport] Compare font-feature-settings as part of Font::operator==(). - Stop the flood of accessibility messages - [Backport] Fix for hitting an assert when refreshing a page with an image - [Backport] Report the decoded size to ImageObserver, instead of deltas Change-Id: I142cc070ba7fb215e4a5b9c162852b583dab9784 Reviewed-by: Kai Koehne <kai.koehne@qt.io>
| | | * | Add changes for 5.7.1Allan Sandfeld Jensen2016-10-171-0/+51
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Iec54a4a3a117ce5f6bca8ff22218eb8fea0008e8 Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io>
| | | * | Fix nullpointer dereferencing errorViktor Engelmann2016-10-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Check whether iconUrls is NULL before calling contains(iconUrl) on it, which, under certain circumstances, caused a SEGFAULT. Task-number: QTBUG-56330 Change-Id: Ia167d68a4c4d62af4740a8cbab2686bfbc975455 Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
| | | * | Update ChromiumAllan Sandfeld Jensen2016-10-121-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I6803569bd7bee4bf8eb94f61771ea2a318eb752b Reviewed-by: Kai Koehne <kai.koehne@qt.io>
| | | * | Fix crash in WebEngineContext when using WaylandAllan Sandfeld Jensen2016-10-121-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For some reason the OpenGL context wayland QPA sets has no nativeHandle, so we end up crashing in strcmp. Assume a context without nativeHandle is Wayland or other GLES2 platform and also force GLES2 when using Ozone. Change-Id: Ia3fc524f3ffbb278d86f9153ec96c7258ef86656 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
| | * | | Pass size to URLRequestJobAllan Sandfeld Jensen2016-11-161-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If QIODevice has a size we can pass that on as expected size. This fixes media playback where Chromium does not always support streaming outside of HTTP or HTTPS. Task-number: QTBUG-57139 Change-Id: Ie8bf96b1f7f6af80fe707936055620d154c3ef2d Reviewed-by: Michal Klocek <michal.klocek@qt.io> Reviewed-by: Milian Wolff <milian.wolff@kdab.com>
* | | | | Split ninja target into runninja and buildninja in gyp_runMichal Klocek2016-12-022-16/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Small refactor to split the targets in makefile. Bootstrapping gn will need also ninja build. Change-Id: Id177a0189b79cf353b6bd27c4e087f4e960f56c2 Reviewed-by: Michael Brüning <michael.bruning@qt.io>
* | | | | Add AllowGeolocationOnInsecureOrigins settingAllan Sandfeld Jensen2016-12-018-2/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This provides a way to restore earlier QtWebEngine behavior where geolocation was possible from all origins. [ChangeLog][Settings] Added setting to again allow insecure origins to request geolocation. Change-Id: I043fb7a36a56fa5acc1740d52a50b9d7ff49a2c5 Reviewed-by: Michael Brüning <michael.bruning@qt.io>
* | | | | Add support for system based ninjaMichal Klocek2016-11-304-13/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds system-ninja feature, which is enabled if ninja is found in system path. Change-Id: Icc58bc252d67a27f80c139349be28f9fd239108a Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@qt.io> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | | | | Merge remote-tracking branch 'origin/5.8' into devAllan Sandfeld Jensen2016-11-24156-584/+4997
|\| | | | | | | | | | | | | | | | | | | Change-Id: Ib2a1eb4dfae2fb5a98066d3d0f8f09f4e1d61d3f
| * | | | Add pkg-config test for system harfbuzz version on linuxMichael Brüning2016-11-232-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Older harfbuzz versions like the one bundled with Ubuntu 16.04 cause crashes of Qt WebEngine applications. Use the bundled copy although Qt was configured with -system-harfbuzz in those scenarios. Task-number: QTBUG-56895 Change-Id: I8fad6754447ba32c3630e6772ca2ae6d4fd710c1 Reviewed-by: Kai Koehne <kai.koehne@qt.io> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | | | Force tst_QWebEnginePage to use English languageSzabolcs David2016-11-231-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some tests were failing with non-english settings. Change-Id: Iebc4dc06217688955cdc7291aca73a2b0c628698 Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
| * | | | Blacklist flaky test_linkHoveredDoesntEmitRepeated QML testPeter Varga2016-11-221-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Task-number: QTBUG-55870 Change-Id: I0d8dc7253e49662d1f73e7ef09d2529c530595b1 Reviewed-by: Milla Pohjanheimo <milla.pohjanheimo@qt.io> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | | | Add support for compiling WebEngine with Clang sanitizer optionsAlexandru Croitor2016-11-222-0/+106
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change pipes the already supported Qt sanitize options to Chromium, so the Chromium build system adds the correct compiler sanitizer flags. The change also checks whether the requirements to build a sanitizer enabled Chromium are met (currently works properly on macOS with an Apple clang >= 7.3.1, and on Linux with clang + libc++). Change-Id: Ibab2d5735935ddc92226575fc0e9f5912b807329 Task-number: QTBUG-52656 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | | | Remove mouse click workaround from test_navigationRequestedPeter Varga2016-11-171-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The issue has been fixed in the quick test framework. Task-number: QTBUG-56223 Change-Id: I101ca4cdd3f741623111d35d13dfcdb35ca85ea3 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | | | Build with Qt 5.6Allan Sandfeld Jensen2016-11-173-1/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make minimal changes so QtWebEngineCore 5.8 still builds with Qt LTR. Change-Id: Iae3ae606b3457a6ca311ae174b3d2a331e9b20bc Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
| * | | | Update plugin.qmltypes to 1.4Marco Benelli2016-11-161-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I988eead2a65a3aa657864631d1d87b9d9a4f0ab3 Reviewed-by: Kai Koehne <kai.koehne@qt.io>
| * | | | Wait for focus before key press in NewViewRequest::test_jsWindowOpenPeter Varga2016-11-165-45/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch is intended to fix flaky failure and factors focus related test functions out to TestWebEngineView. Task-number: QTBUG-56540 Change-Id: Id7e17c08a4a9e863389ae991c9cadc00379377db Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | | | Add gn build files to take_snapshotMichal Klocek2016-11-161-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I9da4f98c203dcaa41653dc1d00fcd34a1dd66b85 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | | | Add faviconbrowser quick manual testPeter Varga2016-11-1511-11/+774
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I3b6353b5c58d62c968c704b20224ff8ee600a524 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | | | Rename class of tst_QWebEngineAccessibility widget auto testPeter Varga2016-11-151-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I042e1be6f0c29b8dc8e5e409e5ecff71a58d0c64 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | | | Pass correct clang prefix directory to GYPJan Kundrát2016-11-151-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously the prefix was hardcoded to "/usr". Now the current environment clang path is considered instead. Change-Id: I23173bb31591dfb6df7a81a37a58502dd6b12886 Task-number: QTBUG-52656 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>