Bug 46706

Summary: [Qt] tst_QWebPage::testStopScheduledPageRefresh() fails on MeeGo handset
Product: WebKit Reporter: Benjamin Poulain <benjamin>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Severity: Normal CC: ademar, commit-queue, webkit-ews
Priority: P2 Keywords: Qt
Version: 528+ (Nightly build)   
Hardware: Other   
OS: Linux   
Bug Depends on:    
Bug Blocks: 44675    
Description Flags
Disable bearer access management
Updated patch none

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>