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>

1 files changed, 1 insertions, 0 deletions

diff --git a/src/core/web_contents_adapter_client.h b/src/core/web_contents_adapter_client.h
index 899678e56..248a963b1 100644
--- a/src/core/web_contents_adapter_client.h
+++ b/src/core/web_contents_adapter_client.h
@@ -395,6 +395,7 @@ public:
 
     virtual RenderWidgetHostViewQtDelegate* CreateRenderWidgetHostViewQtDelegate(RenderWidgetHostViewQtDelegateClient *client) = 0;
     virtual RenderWidgetHostViewQtDelegate* CreateRenderWidgetHostViewQtDelegateForPopup(RenderWidgetHostViewQtDelegateClient *client) = 0;
+    virtual void initializationFinished() = 0;
     virtual void titleChanged(const QString&) = 0;
     virtual void urlChanged(const QUrl&) = 0;
     virtual void iconChanged(const QUrl&) = 0;