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

See Also:

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 an attachment (proposed patch, testcase, etc.)

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>