Bug 46706

Summary:

[Qt] tst_QWebPage::testStopScheduledPageRefresh() fails on MeeGo handset

Product:

WebKit

Reporter:

Benjamin Poulain <benjamin>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ademar, commit-queue, webkit-ews

Priority:

Keywords:

Version:

528+ (Nightly build)

Hardware:

Other

OS:

Linux

Bug Depends on:

Bug Blocks:

44675

Attachments:

Description	Flags
Disable bearer access management	none
Updated patch	none

Benjamin Poulain

Reported 2010-09-28 02:11:50 PDT

The test tst_QWebPage::testStopScheduledPageRefresh() fails on the test bot of MeeGo handset. Here is the output from the bot: FAIL! : tst_QWebPage::testStopScheduledPageRefresh() Compared values are not the same Actual (page1.mainFrame()->url().toString()): about:blank Expected (QString("http://qt.nokia.com/favicon.ico")): http://qt.nokia.com/favicon.ico

Attachments
Disable bearer access management (1.75 KB, patch) 2010-09-28 02:35 PDT, Benjamin Poulain	no flags	Details Formatted Diff Diff
Updated patch (2.72 KB, patch) 2010-09-28 04:40 PDT, Benjamin Poulain	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Benjamin Poulain

Comment 1 2010-09-28 02:35:13 PDT

Created attachment 69033 [details] Disable bearer access management I am not able to reproduce the exact same error as the bot, but the test fails as well on my device. In my case, the bearer access management is messing with the network access and prevent some tests from working. I suspect the bot might have similar issues with its own bearer access management, this patch might solve the issue on the bot.

Early Warning System Bot

Comment 2 2010-09-28 02:45:20 PDT

Attachment 69033 [details] did not build on qt: Build output: http://queues.webkit.org/results/4139002

Benjamin Poulain

Comment 3 2010-09-28 04:40:21 PDT

Created attachment 69039 [details] Updated patch While updating the patch, I noticed it could fail with Qt trunk on MeeGo handset. If the network is not setup at all on the device, the access to http://qt.nokia.com/favicon.ico just don't do anything, even without bearer access management. I modified the test to use local URLs. With a recent version of Qt, this should work on MeeGo without network issues.

WebKit Commit Bot

Comment 4 2010-09-28 05:18:51 PDT

Comment on attachment 69039 [details] Updated patch Clearing flags on attachment: 69039 Committed r68507: <http://trac.webkit.org/changeset/68507>

WebKit Commit Bot

Comment 5 2010-09-28 05:18:56 PDT

All reviewed patches have been landed. Closing bug.

Ademar Reis

Comment 6 2010-09-28 07:57:47 PDT

Revision r68507 cherry-picked into qtwebkit-2.1 with commit 37291a9 <http://gitorious.org/webkit/qtwebkit/commit/37291a9>

Note You need to log in before you can comment on or make changes to this bug.