Bug 46706 - [Qt] tst_QWebPage::testStopScheduledPageRefresh() fails on MeeGo handset
Summary: [Qt] tst_QWebPage::testStopScheduledPageRefresh() fails on MeeGo handset
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other Linux
: P2 Normal
Assignee: Nobody
Keywords: Qt
Depends on:
Blocks: 44675
  Show dependency treegraph
Reported: 2010-09-28 02:11 PDT by Benjamin Poulain
Modified: 2010-09-28 07:57 PDT (History)
3 users (show)

See Also:

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

Note You need to log in before you can comment on or make changes to this bug.
Description Benjamin Poulain 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
Comment 1 Benjamin Poulain 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.
Comment 2 Early Warning System Bot 2010-09-28 02:45:20 PDT
Attachment 69033 [details] did not build on qt:
Build output: http://queues.webkit.org/results/4139002
Comment 3 Benjamin Poulain 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.
Comment 4 WebKit Commit Bot 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>
Comment 5 WebKit Commit Bot 2010-09-28 05:18:56 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Ademar Reis 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>