Bug 63245

Summary:

[Qt] Fix tst_QWebPage::testStopScheduledPageRefresh() API test

Product:

WebKit

Reporter:

Csaba Osztrogonác <ossy>

Component:

WebKit Qt

Assignee:

Jocelyn Turcotte <jturcotte>

Status:

RESOLVED DUPLICATE

Severity:

Normal

CC:

jturcotte, ossy, sergio.ahumada

Priority:

Keywords:

Qt, QtTriaged

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Bug Depends on:

Bug Blocks:

38654

Attachments:

Description	Flags
Patch	none
Patch	none

Description Csaba Osztrogonác 2011-06-23 04:29:53 PDT

http://build.webkit.org/builders/Qt%20Linux%20Release/builds/34519

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]
Patch

Comment 4 Jocelyn Turcotte 2012-11-09 06:22:43 PST

Created attachment 173292 [details]
Patch

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 ***