summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSzabolcs David <davidsz@inf.u-szeged.hu>2015-04-07 08:39:01 -0700
committerSzabolcs David <davidsz@inf.u-szeged.hu>2015-04-09 11:34:31 +0000
commiteebf1d8bfde9cf8261487e9cef88ffbe7451cb61 (patch)
treed2c441574e8d429a1ae472ee4ea721fc319b3e77
parent16b93c2d7dad73c33a745dce2a2eb7eac0e9826e (diff)
Remove tst_setUrlWithFragment test case
This test is flaky, we don't have proper signals for in-page navigations and this case tests the behavior of the browser engine, not the QtWebEngine API. Change-Id: Ia94219df3501126bf1ab00a400a57d589b651589 Reviewed-by: Andras Becsi <andras.becsi@theqtcompany.com>
-rw-r--r--tests/auto/widgets/qwebengineframe/tst_qwebengineframe.cpp41
1 files changed, 0 insertions, 41 deletions
diff --git a/tests/auto/widgets/qwebengineframe/tst_qwebengineframe.cpp b/tests/auto/widgets/qwebengineframe/tst_qwebengineframe.cpp
index 9901bcd7..00e1b112 100644
--- a/tests/auto/widgets/qwebengineframe/tst_qwebengineframe.cpp
+++ b/tests/auto/widgets/qwebengineframe/tst_qwebengineframe.cpp
@@ -87,8 +87,6 @@ private Q_SLOTS:
void setContent();
void setCacheLoadControlAttribute();
void setUrlWithPendingLoads();
- void setUrlWithFragment_data();
- void setUrlWithFragment();
void setUrlToEmpty();
void setUrlToInvalid();
void setUrlHistory();
@@ -1227,45 +1225,6 @@ void tst_QWebEngineFrame::setUrlWithPendingLoads()
page.setUrl(QUrl("about:blank"));
}
-void tst_QWebEngineFrame::setUrlWithFragment_data()
-{
- QTest::addColumn<QUrl>("previousUrl");
- QTest::newRow("empty") << QUrl();
- QTest::newRow("same URL no fragment") << QUrl("qrc:/test1.html");
- // See comments in setUrlSameUrl about using setUrl() with the same url().
- QTest::newRow("same URL with same fragment") << QUrl("qrc:/test1.html#");
- QTest::newRow("same URL with different fragment") << QUrl("qrc:/test1.html#anotherFragment");
- QTest::newRow("another URL") << QUrl("qrc:/test2.html");
-}
-
-// Based on bug report https://bugs.webkit.org/show_bug.cgi?id=32723
-void tst_QWebEngineFrame::setUrlWithFragment()
-{
- QSKIP("FIXME: https://trello.com/c/3L7F8VZJ/217-take-care-about-the-in-page-navigations-in-the-tests");
- QFETCH(QUrl, previousUrl);
-
- QWebEnginePage page;
-
- if (!previousUrl.isEmpty()) {
- page.load(previousUrl);
- ::waitForSignal(&page, SIGNAL(loadFinished(bool)));
- QCOMPARE(page.url(), previousUrl);
- }
-
- QSignalSpy spy(&page, SIGNAL(urlChanged(QUrl)));
- const QUrl url("qrc:/test1.html#");
- QVERIFY(!url.fragment().isNull());
-
- page.setUrl(url);
- ::waitForSignal(&page, SIGNAL(urlChanged(QUrl)));
-
- QCOMPARE(spy.count(), 1);
- QVERIFY(!toPlainTextSync(&page).isEmpty());
- // Slight change: This information now comes from Chromium and the behavior of requestedUrl changed in this case.
- // QCOMPARE(page.requestedUrl(), url);
- QCOMPARE(page.url(), url);
-}
-
void tst_QWebEngineFrame::setUrlToEmpty()
{
int expectedLoadFinishedCount = 0;