| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
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>
|
|\
| |
| |
| |
| |
| |
| | |
Conflicts:
src/3rdparty
Change-Id: I49d99b5ac2cfba243eda1ea705c2a7d71b5542f6
|
| |
| |
| |
| |
| | |
Change-Id: I102bda62bfaa16a8c9ff4c4929e6b5a055de3e25
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
|
| |
| |
| |
| |
| |
| | |
Task-number: QTBUG-55857
Change-Id: Ieb7fe836e7b03bea28dd97bcad343708a5be2cb5
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
|\|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| | |
Change-Id: I21dfb9146dad44329c490c1c6b32864c70384db3
Reviewed-by: Michael Brüning <michael.bruning@qt.io>
|
|\|
| |
| |
| | |
Change-Id: Ic6686df8f82f710a3441501b7eeaffe69fbcbdf7
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
|\|
| |
| |
| | |
Change-Id: If4d8a25f5a2192a658d031252435935cbc675725
|
| |
| |
| |
| |
| |
| |
| |
| | |
Switch from timed wait to using tryCompare.
Task-number: QTBUG-53777
Change-Id: I845fb8ee7b84641bd23314794512960634423d54
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| | |
Change-Id: Ia7541befb43d2b6fdd09b13ca51fc7d09b955964
Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
|\|
| |
| |
| | |
Change-Id: Ib010ede9756fb02992a3276ae7ec90ef1ab56a00
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
|\ \ |
|
| |\|
| | |
| | |
| | | |
Change-Id: I561c00b3a844ab493a5bf0148a5923662842cf5d
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
We have no equivalent of QWebFrame::javaScriptWindowObjectCleared and
won't need it.
Change-Id: I998e0412256887414c125df9ab994216dee22dc9
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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>
|
| | |
| | |
| | |
| | |
| | | |
Change-Id: I348d135949aaca4695352b1d7a27187d891ddd56
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| | |
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 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>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
|\|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Conflicts:
src/3rdparty
src/core/browser_context_adapter.cpp
src/core/web_contents_adapter.cpp
tests/auto/widgets/qwebenginepage/BLACKLIST
Change-Id: I45ee0a33f6316f585555d58fede8072fe514aecf
|
| |\
| | |
| | |
| | | |
refs/staging/5.6
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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>
|
| |/
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| | |
Skip test on osx for now.
Change-Id: I8f748d00da54fd5b7217b08a18e7a58d31cb898c
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| | |
Change-Id: Ie0ecb1808c6b6a689ef29d7fa4831ca0a2e92721
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
|\ \ |
|
| |\|
| | |
| | |
| | | |
Change-Id: I922e637ffbf03aa15711fa5d0e10ecd4f25448ab
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Requesting a local file resource without a baseUrl set, should not be
allowed. This hasn't worked before in Webkit, so the checks were marked
with QEXPECT_FAIL. Remove these, as the issue has been fixed, and now
causes unexpected passes.
Change-Id: I7d38692a9961651d48abe28218d7ee022a85f101
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|