46706 2010-09-28 02:11:50 -0700 [Qt] tst_QWebPage::testStopScheduledPageRefresh() fails on MeeGo handset 2010-09-28 07:57:47 -0700 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) Other Linux RESOLVED FIXED Qt P2 Normal --- 44675 1 benjamin webkit-unassigned ademar commit-queue webkit-ews oldest_to_newest 286130 0 benjamin 2010-09-28 02:11:50 -0700 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 286134 1 69033 benjamin 2010-09-28 02:35:13 -0700 Created attachment 69033 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. 286135 2 webkit-ews 2010-09-28 02:45:20 -0700 Attachment 69033 did not build on qt: Build output: http://queues.webkit.org/results/4139002 286180 3 69039 benjamin 2010-09-28 04:40:21 -0700 Created attachment 69039 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. 286187 4 69039 commit-queue 2010-09-28 05:18:51 -0700 Comment on attachment 69039 Updated patch Clearing flags on attachment: 69039 Committed r68507: <http://trac.webkit.org/changeset/68507> 286188 5 commit-queue 2010-09-28 05:18:56 -0700 All reviewed patches have been landed. Closing bug. 286229 6 ademar 2010-09-28 07:57:47 -0700 Revision r68507 cherry-picked into qtwebkit-2.1 with commit 37291a9 <http://gitorious.org/webkit/qtwebkit/commit/37291a9> 69033 2010-09-28 02:35:13 -0700 2010-09-28 04:40:21 -0700 Disable bearer access management 46706.patch text/plain 1796 benjamin ZGlmZiAtLWdpdCBhL1dlYktpdC9xdC9DaGFuZ2VMb2cgYi9XZWJLaXQvcXQvQ2hhbmdlTG9nCmlu ZGV4IGVkYzhlMjQuLmFlMmI3NmQgMTAwNjQ0Ci0tLSBhL1dlYktpdC9xdC9DaGFuZ2VMb2cKKysr IGIvV2ViS2l0L3F0L0NoYW5nZUxvZwpAQCAtMSwzICsxLDIwIEBACisyMDEwLTA5LTI4ICBCZW5q YW1pbiBQb3VsYWluICA8YmVuamFtaW4ucG91bGFpbkBub2tpYS5jb20+CisKKyAgICAgICAgUmV2 aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgW1F0XSB0c3RfUVdlYlBhZ2U6OnRl c3RTdG9wU2NoZWR1bGVkUGFnZVJlZnJlc2goKSBmYWlscyBvbiBNZWVHbyBoYW5kc2V0CisgICAg ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD00NjcwNgorCisgICAg ICAgIFNvbWUgdGVzdHMgYXJlIHRyeWluZyB0byBhY2Nlc3MgVVJMcyBidXQgZG8gbm90IHJlcXVp cmUgdGhlCisgICAgICAgIHJlcXVlc3RzIHRvIHN1Y2NlZWQuIFRoZSBiZWFyZXIgYWNjZXNzIG1h bmFnZW1lbnQga2lja3MgaW4gZm9yIHRob3NlCisgICAgICAgIGFjY2Vzc2VzIHdoaWNoIHByZXZl bnQgYW55IHJlcXVlc3QgZnJvbSBmaW5pc2hpbmcuCisKKyAgICAgICAgVGhpcyBwYXRjaCBzZXQg YSBudWxsIGNvbmZpZ3VyYXRpb24gdG8gZW5zdXJlIHJlcXVlc3QgYSBwcm9jZXNzZWQgYW5kCisg ICAgICAgIGNhbiBmYWlsLgorCisgICAgICAgICogdGVzdHMvcXdlYnBhZ2UvdHN0X3F3ZWJwYWdl LmNwcDoKKyAgICAgICAgKFRlc3ROZXR3b3JrTWFuYWdlcjo6VGVzdE5ldHdvcmtNYW5hZ2VyKToK KwogMjAxMC0wOS0yMyAgQWRlbWFyIGRlIFNvdXphIFJlaXMgSnIgIDxhZGVtYXIucmVpc0BvcGVu Ym9zc2Eub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IEtlbm5ldGggUm9oZGUgQ2hyaXN0aWFu c2VuLgpkaWZmIC0tZ2l0IGEvV2ViS2l0L3F0L3Rlc3RzL3F3ZWJwYWdlL3RzdF9xd2VicGFnZS5j cHAgYi9XZWJLaXQvcXQvdGVzdHMvcXdlYnBhZ2UvdHN0X3F3ZWJwYWdlLmNwcAppbmRleCAxMGFm OTRjLi5kNzQ1ZTZiIDEwMDY0NAotLS0gYS9XZWJLaXQvcXQvdGVzdHMvcXdlYnBhZ2UvdHN0X3F3 ZWJwYWdlLmNwcAorKysgYi9XZWJLaXQvcXQvdGVzdHMvcXdlYnBhZ2UvdHN0X3F3ZWJwYWdlLmNw cApAQCAtMzMsNiArMzMsNyBAQAogI2luY2x1ZGUgPHFncmFwaGljc3NjZW5lLmg+CiAjaW5jbHVk ZSA8cWdyYXBoaWNzdmlldy5oPgogI2luY2x1ZGUgPHFncmFwaGljc3dlYnZpZXcuaD4KKyNpbmNs dWRlIDxxbmV0d29ya2NvbmZpZ3VyYXRpb24uaD4KICNpbmNsdWRlIDxxbmV0d29ya3JlcXVlc3Qu aD4KICNpbmNsdWRlIDxxd2ViZGF0YWJhc2UuaD4KICNpbmNsdWRlIDxxd2ViZWxlbWVudC5oPgpA QCAtMzc5LDcgKzM4MCw5IEBAIHZvaWQgdHN0X1FXZWJQYWdlOjphY2NlcHROYXZpZ2F0aW9uUmVx dWVzdFdpdGhOZXdXaW5kb3coKQogY2xhc3MgVGVzdE5ldHdvcmtNYW5hZ2VyIDogcHVibGljIFFO ZXR3b3JrQWNjZXNzTWFuYWdlcgogewogcHVibGljOgotICAgIFRlc3ROZXR3b3JrTWFuYWdlcihR T2JqZWN0KiBwYXJlbnQpIDogUU5ldHdvcmtBY2Nlc3NNYW5hZ2VyKHBhcmVudCkge30KKyAgICBU ZXN0TmV0d29ya01hbmFnZXIoUU9iamVjdCogcGFyZW50KSA6IFFOZXR3b3JrQWNjZXNzTWFuYWdl cihwYXJlbnQpIHsKKyAgICAgICAgc2V0Q29uZmlndXJhdGlvbihRTmV0d29ya0NvbmZpZ3VyYXRp b24oKSk7CisgICAgfQogCiAgICAgUUxpc3Q8UVVybD4gcmVxdWVzdGVkVXJsczsKICAgICBRTGlz dDxRTmV0d29ya1JlcXVlc3Q+IHJlcXVlc3RzOwo= 69039 2010-09-28 04:40:21 -0700 2010-09-28 05:18:51 -0700 Updated patch 46706.patch text/plain 2781 benjamin ZGlmZiAtLWdpdCBhL1dlYktpdC9xdC9DaGFuZ2VMb2cgYi9XZWJLaXQvcXQvQ2hhbmdlTG9nCmlu ZGV4IGVkYzhlMjQuLjI5NzRiYmUgMTAwNjQ0Ci0tLSBhL1dlYktpdC9xdC9DaGFuZ2VMb2cKKysr IGIvV2ViS2l0L3F0L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDEwLTA5LTI4ICBCZW5q YW1pbiBQb3VsYWluICA8YmVuamFtaW4ucG91bGFpbkBub2tpYS5jb20+CisKKyAgICAgICAgUmV2 aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgW1F0XSB0c3RfUVdlYlBhZ2U6OnRl c3RTdG9wU2NoZWR1bGVkUGFnZVJlZnJlc2goKSBmYWlscyBvbiBNZWVHbyBoYW5kc2V0CisgICAg ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD00NjcwNgorCisgICAg ICAgIFRoZSB0ZXN0IHJlcXVpcmUgdGhlIG5ldHdvcmsgdG8gc3VjY2VlZCBvciBmYWlsIGluIG9y ZGVyIHRvIHdvcmsuCisgICAgICAgIE9uIE1lZUdvLCB0aGUgbmV0d29yayBjYW4gc2ltcGx5IHRp bWVvdXQgdGhlIGJlY2F1c2Ugb2YgdGhlIG5ldHdvcmQgYWNjZXNzCisgICAgICAgIG1hbmFnZW1l bnQuIFRoaXMgcGF0Y2ggbWFrZXMgdGhlIHRlc3QgdXNlIGxvY2FsIFVSTHMgdG8gYXZvaWQgdGhl CisgICAgICAgIHByb2JsZW0uCisKKyAgICAgICAgKiB0ZXN0cy9xd2VicGFnZS90c3RfcXdlYnBh Z2UuY3BwOgorICAgICAgICAoVGVzdE5ldHdvcmtNYW5hZ2VyOjpUZXN0TmV0d29ya01hbmFnZXIp OgorCiAyMDEwLTA5LTIzICBBZGVtYXIgZGUgU291emEgUmVpcyBKciAgPGFkZW1hci5yZWlzQG9w ZW5ib3NzYS5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgS2VubmV0aCBSb2hkZSBDaHJpc3Rp YW5zZW4uCmRpZmYgLS1naXQgYS9XZWJLaXQvcXQvdGVzdHMvcXdlYnBhZ2UvdHN0X3F3ZWJwYWdl LmNwcCBiL1dlYktpdC9xdC90ZXN0cy9xd2VicGFnZS90c3RfcXdlYnBhZ2UuY3BwCmluZGV4IDEw YWY5NGMuLmI4ZDRhYmEgMTAwNjQ0Ci0tLSBhL1dlYktpdC9xdC90ZXN0cy9xd2VicGFnZS90c3Rf cXdlYnBhZ2UuY3BwCisrKyBiL1dlYktpdC9xdC90ZXN0cy9xd2VicGFnZS90c3RfcXdlYnBhZ2Uu Y3BwCkBAIC0yMTg5LDIzICsyMTg5LDIzIEBAIHZvaWQgdHN0X1FXZWJQYWdlOjp0ZXN0U3RvcFNj aGVkdWxlZFBhZ2VSZWZyZXNoKCkKICAgICBRV2ViUGFnZSBwYWdlMTsKICAgICBwYWdlMS5zZXRO ZXR3b3JrQWNjZXNzTWFuYWdlcihuZXcgVGVzdE5ldHdvcmtNYW5hZ2VyKCZwYWdlMSkpOwogICAg IHBhZ2UxLm1haW5GcmFtZSgpLT5zZXRIdG1sKCI8aHRtbD48aGVhZD4iCi0gICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICI8bWV0YSBodHRwLWVxdWl2PVwicmVmcmVzaFwiY29udGVudD1c IjA7VVJMPWh0dHA6Ly9xdC5ub2tpYS5jb20vZmF2aWNvbi5pY29cIj4iCisgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICI8bWV0YSBodHRwLWVxdWl2PVwicmVmcmVzaFwiY29udGVudD1c IjA7VVJMPXFyYzovLy9yZXNvdXJjZXMvaW5kZXguaHRtbFwiPiIKICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgIjwvaGVhZD48Ym9keT48aDE+UGFnZSByZWRpcmVjdHMgaW1tZWRpYXRl bHkuLi48L2gxPiIKICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIjwvYm9keT48L2h0 bWw+Iik7CiAgICAgUVZFUklGWSg6OndhaXRGb3JTaWduYWwoJnBhZ2UxLCBTSUdOQUwobG9hZEZp bmlzaGVkKGJvb2wpKSkpOwogICAgIFFUZXN0OjpxV2FpdCg1MDApOwotICAgIFFDT01QQVJFKHBh Z2UxLm1haW5GcmFtZSgpLT51cmwoKS50b1N0cmluZygpLCBRU3RyaW5nKCJodHRwOi8vcXQubm9r aWEuY29tL2Zhdmljb24uaWNvIikpOworICAgIFFDT01QQVJFKHBhZ2UxLm1haW5GcmFtZSgpLT51 cmwoKSwgUVVybChRTGF0aW4xU3RyaW5nKCJxcmM6Ly8vcmVzb3VyY2VzL2luZGV4Lmh0bWwiKSkp OwogICAgIAogICAgIC8vIFdpdGggUVdlYlBhZ2U6OlN0b3BTY2hlZHVsZWRQYWdlUmVmcmVzaAog ICAgIFFXZWJQYWdlIHBhZ2UyOwogICAgIHBhZ2UyLnNldE5ldHdvcmtBY2Nlc3NNYW5hZ2VyKG5l dyBUZXN0TmV0d29ya01hbmFnZXIoJnBhZ2UyKSk7CiAgICAgcGFnZTIubWFpbkZyYW1lKCktPnNl dEh0bWwoIjxodG1sPjxoZWFkPiIKLSAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAiPG1l dGEgaHR0cC1lcXVpdj1cInJlZnJlc2hcImNvbnRlbnQ9XCIxO1VSTD1odHRwOi8vcXQubm9raWEu Y29tL2Zhdmljb24uaWNvXCI+IgorICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICI8bWV0 YSBodHRwLWVxdWl2PVwicmVmcmVzaFwiY29udGVudD1cIjE7VVJMPXFyYzovLy9yZXNvdXJjZXMv aW5kZXguaHRtbFwiPiIKICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAiPC9oZWFkPjxi b2R5PjxoMT5QYWdlIHJlZGlyZWN0IHRlc3Qgd2l0aCAxIHNlYyB0aW1lb3V0Li4uPC9oMT4iCiAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIjwvYm9keT48L2h0bWw+Iik7CiAgICAgcGFn ZTIudHJpZ2dlckFjdGlvbihRV2ViUGFnZTo6U3RvcFNjaGVkdWxlZFBhZ2VSZWZyZXNoKTsKICAg ICBRVGVzdDo6cVdhaXQoMTUwMCk7Ci0gICAgUUNPTVBBUkUocGFnZTIubWFpbkZyYW1lKCktPnVy bCgpLnRvU3RyaW5nKCksIFFTdHJpbmcoImFib3V0OmJsYW5rIikpOworICAgIFFDT01QQVJFKHBh Z2UyLm1haW5GcmFtZSgpLT51cmwoKS50b1N0cmluZygpLCBRTGF0aW4xU3RyaW5nKCJhYm91dDpi bGFuayIpKTsKIH0KIAogdm9pZCB0c3RfUVdlYlBhZ2U6OmZpbmRUZXh0KCkK