Fix an assertion in web_content_delegates_qt.cpp

If we get a replacement content, we can see a DidFinishLoad event for a frame.
This error page should be ignored based on the frame.

Change-Id: I3e1cd1773e8c5fc608605197c957011ddf258123
Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>

1 files changed, 11 insertions, 0 deletions

diff --git a/tests/auto/quick/qmltests/data/tst_loadFail.qml b/tests/auto/quick/qmltests/data/tst_loadFail.qml
index e2282129f..7b0a1849e 100644
--- a/tests/auto/quick/qmltests/data/tst_loadFail.qml
+++ b/tests/auto/quick/qmltests/data/tst_loadFail.qml
@@ -50,6 +50,12 @@ TestWebEngineView {
 
     property variant testUrl
 
+    SignalSpy {
+        id: spyIconChanged
+        target: webEngineView
+        signalName: "iconChanged"
+    }
+
     TestCase {
         id: test
         name: "WebEngineViewLoadFail"
@@ -57,6 +63,11 @@ TestWebEngineView {
             testUrl = Qt.resolvedUrl("file_that_does_not_exist.html")
             webEngineView.url = testUrl
             verify(webEngineView.waitForLoadFailed())
+            spyIconChanged.clear()
+
+            // If this testcase finishes too early, we can not handle the received replacement content.
+            // So we should wait to ignore this error page.
+            spyIconChanged.wait()
         }
     }