diff options
author | Jüri Valdmann <juri.valdmann@qt.io> | 2018-02-26 12:48:03 +0100 |
---|---|---|
committer | Allan Sandfeld Jensen <allan.jensen@qt.io> | 2018-04-06 06:36:24 +0000 |
commit | 3bc5b9d4f9d481707c02e83e0cd20f60dec425f2 (patch) | |
tree | f29ba0036095fcf178133281be582d5fe650814b /tests/auto/widgets/qwebengineview/tst_qwebengineview.cpp | |
parent | fbfc9ee7f06c6f36b79145bc7e8ba05c5d3fa621 (diff) |
Avoid creating temporary blank WebContents
- Avoid creating a blank WebContents for new QWebEnginePages only to replace it
with a different blank WebContents moments later. This problem is already
solved in QQuickWebEngineView by lazy initialization, therefore the patch
moves this lazy initialization into WebContentsAdapter itself so that it can
be applied to both QQuickWebEngineView and QWebEnginePage.
- Try to delay WebContents creation until the first navigation so that we have
enough information available to pick the right initial SiteInstance when
calling WebContents::Create. This is done by triggering WebContents creation
from the first call to a WebContentsAdapter::load* method.
- Use the SiteInstance from WebContentsDelegate::OpenURLFromTab. This method
gets a SiteInstance pointer which we should give to WebContents::Create.
Task-number: QTBUG-65561
Task-number: QTBUG-67163
Change-Id: Id7b351998adefb810cf27c61a1447b61d7f4c606
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
Diffstat (limited to 'tests/auto/widgets/qwebengineview/tst_qwebengineview.cpp')
-rw-r--r-- | tests/auto/widgets/qwebengineview/tst_qwebengineview.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/auto/widgets/qwebengineview/tst_qwebengineview.cpp b/tests/auto/widgets/qwebengineview/tst_qwebengineview.cpp index d58bfe0b3..207836bef 100644 --- a/tests/auto/widgets/qwebengineview/tst_qwebengineview.cpp +++ b/tests/auto/widgets/qwebengineview/tst_qwebengineview.cpp @@ -873,6 +873,7 @@ void tst_QWebEngineView::stopSettingFocusWhenDisabled() QFETCH(bool, focusResult); QWebEngineView webView; + webView.settings()->setAttribute(QWebEngineSettings::FocusOnNavigationEnabled, true); webView.resize(640, 480); webView.show(); webView.setEnabled(viewEnabled); |