Bug 84293

Summary: [Qt][Wk2] unskip page-reload-cancel-permission-requests from geolocation for wk2
Product: WebKit Reporter: Mahesh Kulkarni <maheshk>
Component: New BugsAssignee: Mahesh Kulkarni <maheshk>
Status: RESOLVED INVALID    
Severity: Normal CC: zarvai
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Description Mahesh Kulkarni 2012-04-18 15:49:38 PDT
Created attachment 137790 [details]
Patch

r105988 implemented layoutTestController.setGeolocationPermission so we can unskip the -fast/dom/Geolocation/page-reload-cancel-permission-requests.html
Comment 1 Mahesh Kulkarni 2012-04-18 16:01:17 PDT
Created attachment 137794 [details]
Patch
Comment 2 Zoltan Arvai 2012-04-20 05:16:24 PDT
It seems the unskipped test fails with Qt5 on our x86-64 Linux Qt Release WebKit2 (Amazon EC2) and x86-32 Linux Qt Release WebKit2 bots:

--- /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/fast/dom/Geolocation/page-reload-cancel-permission-requests-expected.txt
+++ /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/fast/dom/Geolocation/page-reload-cancel-permission-requests-actual.txt
@@ -1,10 +1,3 @@
-Tests that when a page is reloaded, the frame is properly detached from the Geolocation object to ensure that no permission requests are in progress.
+CONSOLE MESSAGE: line 13: TypeError: 'undefined' is not a function (evaluating 'layoutTestController.numberOfPendingGeolocationPermissionRequests()')
+FAIL: Timed out waiting for notifyDone to be called
 
-On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
-
-
-PASS numPendingRequests is 0
-PASS successfullyParsed is true
-
-TEST COMPLETE
-


http://build.webkit.sed.hu/results/x86-64%20Linux%20Qt%20Release%20WebKit2%20%28Amazon%20EC2%29/r114688%20%281470%29/results.html
Comment 3 Jocelyn Turcotte 2014-02-03 03:20:36 PST
=== Bulk closing of Qt bugs ===

If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary.

If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.