Bug 63245 - [Qt] Fix tst_QWebPage::testStopScheduledPageRefresh() API test
Summary: [Qt] Fix tst_QWebPage::testStopScheduledPageRefresh() API test
Status: RESOLVED DUPLICATE of bug 68711
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Jocelyn Turcotte
Keywords: Qt, QtTriaged
Depends on:
Blocks: 38654
  Show dependency treegraph
Reported: 2011-06-23 04:29 PDT by Csaba Osztrogonác
Modified: 2012-11-27 08:35 PST (History)
3 users (show)

See Also:

Patch (5.17 KB, patch)
2012-11-09 06:20 PST, Jocelyn Turcotte
no flags Details | Formatted Diff | Diff
Patch (5.18 KB, patch)
2012-11-09 06:22 PST, Jocelyn Turcotte
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2011-06-23 04:29:53 PDT

FAIL!  : tst_QWebPage::testStopScheduledPageRefresh() '::waitForSignal(&page1, SIGNAL(loadFinished(bool)))' returned FALSE. ()
   Loc: [/ramdisk/qt-linux-release/build/Source/WebKit/qt/tests/qwebpage/tst_qwebpage.cpp(2808)]
Comment 1 Csaba Osztrogonác 2011-06-23 05:50:46 PDT
One more fail:

FAIL!  : tst_QWebPage::testStopScheduledPageRefresh() Compared values are not the same
   Actual (page1.mainFrame()->url()): about:blank
   Expected (QUrl(QLatin1String("qrc:///resources/index.html"))): qrc:/resources/index.html
   Loc: [/home/oszi/WebKit/Source/WebKit/qt/tests/qwebpage/tst_qwebpage.cpp(2812)]

Marked as expected fail until fix: http://trac.webkit.org/changeset/89560
Comment 2 Sergio Ahumada 2012-02-09 14:19:07 PST
These test is passing in the Qt Project CI

2/10/12 7:06:24 AM EST: XPASS  : tst_QWebPage::testStopScheduledPageRefresh() '::waitForSignal(&page1, SIGNAL(loadFinished(bool)))' returned FALSE. ()
2/10/12 7:06:24 AM EST:    Loc: [../../../src/3rdparty/webkit/Source/WebKit/qt/tests/qwebpage/tst_qwebpage.cpp(2843)]
2/10/12 7:06:26 AM EST: XPASS  : tst_QWebPage::testStopScheduledPageRefresh() COMPARE()
2/10/12 7:06:26 AM EST:    Loc: [../../../src/3rdparty/webkit/Source/WebKit/qt/tests/qwebpage/tst_qwebpage.cpp(2846)]
Comment 3 Jocelyn Turcotte 2012-11-09 06:20:45 PST
Created attachment 173291 [details]
Comment 4 Jocelyn Turcotte 2012-11-09 06:22:43 PST
Created attachment 173292 [details]
Comment 5 Jocelyn Turcotte 2012-11-27 08:35:59 PST
The patch wasn't the right way to fix it since it was previously correctly handled by SecurityPolicy::AllowLocalLoadsForLocalAndSubstituteData and regressed since http://trac.webkit.org/changeset/87756

*** This bug has been marked as a duplicate of bug 68711 ***