2011-04-06 Benjamin Poulain <benjamin.poulain@nokia.com>

        Reviewed by Kenneth Rohde Christiansen.

        [Qt] LayoutTestController needs to implement numberOfPendingGeolocationPermissionRequests
        https://bugs.webkit.org/show_bug.cgi?id=56086

        * platform/qt/Skipped: Remove the skipped test which required numberOfPendingGeolocationPermissionRequests.
2011-04-06  Benjamin Poulain  <benjamin.poulain@nokia.com>

        Reviewed by Kenneth Rohde Christiansen.

        [Qt] LayoutTestController needs to implement numberOfPendingGeolocationPermissionRequests
        https://bugs.webkit.org/show_bug.cgi?id=56086

        Add the accessor numberOfPendingGeolocationPermissionRequests to call GeolocationClientMock::numberOfPendingPermissionRequests()
        from the LayoutTestController.

        * WebCoreSupport/DumpRenderTreeSupportQt.cpp:
        (DumpRenderTreeSupportQt::numberOfPendingGeolocationPermissionRequests):
        * WebCoreSupport/DumpRenderTreeSupportQt.h:
2011-04-06  Benjamin Poulain  <benjamin.poulain@nokia.com>

        Reviewed by Kenneth Rohde Christiansen.

        [Qt] LayoutTestController needs to implement numberOfPendingGeolocationPermissionRequests
        https://bugs.webkit.org/show_bug.cgi?id=56086

        Add the missing method.

        * DumpRenderTree/qt/DumpRenderTreeQt.cpp:
        (WebCore::DumpRenderTree::getAllPages): Add a method to return all the pages allocated for the current test.
        * DumpRenderTree/qt/DumpRenderTreeQt.h:
        * DumpRenderTree/qt/LayoutTestControllerQt.cpp:
        (LayoutTestController::numberOfPendingGeolocationPermissionRequests):

git-svn-id: svn://svn.chromium.org/blink/trunk@83137 bbb929c8-8fbe-4397-9dbb-9b2b20218538

third_party/WebKit/LayoutTests/ChangeLog

+2011-04-06  Benjamin Poulain  <benjamin.poulain@nokia.com>
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        [Qt] LayoutTestController needs to implement numberOfPendingGeolocationPermissionRequests
+        https://bugs.webkit.org/show_bug.cgi?id=56086
+
+        * platform/qt/Skipped: Remove the skipped test which required numberOfPendingGeolocationPermissionRequests.
+
 2011-04-06  Kent Tamura  <tkent@chromium.org>
 
         [Chromium] Update expectations for Geeza Pro and so on.

third_party/WebKit/LayoutTests/platform/qt/Skipped

@@ -3197,10 +3197,6 @@ http/tests/xmlviewer/dumpAsText/xul.xml
 # [Qt] geolocation case failing as needs implementation in DumpRenderTree : TODO bug
 fast/dom/Geolocation/window-close-crash.html
 
-# [Qt] LayoutTestController needs to implement numberOfPendingGeolocationPermissionRequests
-# https://bugs.webkit.org/show_bug.cgi?id=56086
-fast/dom/Geolocation/page-reload-cancel-permission-requests.html
-
 # [Qt] Two http auth tests fail with Qt 4.7.2
 # https://bugs.webkit.org/show_bug.cgi?id=55767
 http/tests/appcache/auth.html

third_party/WebKit/Source/WebKit/qt/ChangeLog

+2011-04-06  Benjamin Poulain  <benjamin.poulain@nokia.com>
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        [Qt] LayoutTestController needs to implement numberOfPendingGeolocationPermissionRequests
+        https://bugs.webkit.org/show_bug.cgi?id=56086
+
+        Add the accessor numberOfPendingGeolocationPermissionRequests to call GeolocationClientMock::numberOfPendingPermissionRequests()
+        from the LayoutTestController.
+
+        * WebCoreSupport/DumpRenderTreeSupportQt.cpp:
+        (DumpRenderTreeSupportQt::numberOfPendingGeolocationPermissionRequests):
+        * WebCoreSupport/DumpRenderTreeSupportQt.h:
+
 2011-04-06  Anders Bakken  <agbakken@gmail.com>
 
         Reviewed by Kenneth Rohde Christiansen.

third_party/WebKit/Source/WebKit/qt/WebCoreSupport/DumpRenderTreeSupportQt.cpp

@@ -822,6 +822,17 @@ void DumpRenderTreeSupportQt::setMockGeolocationError(QWebPage* page, int errorC
 #endif
 }
 
+int DumpRenderTreeSupportQt::numberOfPendingGeolocationPermissionRequests(QWebPage* page)
+{
+#if ENABLE(CLIENT_BASED_GEOLOCATION)
+    Page* corePage = QWebPagePrivate::core(page);
+    GeolocationClientMock* mockClient = toGeolocationClientMock(corePage->geolocationController()->client());
+    return mockClient->numberOfPendingPermissionRequests();
+#else
+    return -1;
+#endif
+}
+
 bool DumpRenderTreeSupportQt::isTargetItem(const QWebHistoryItem& historyItem)
 {
     QWebHistoryItem it = historyItem;

third_party/WebKit/Source/WebKit/qt/WebCoreSupport/DumpRenderTreeSupportQt.h

@@ -147,6 +147,7 @@ public:
     static void setMockGeolocationPermission(QWebPage*, bool allowed);
     static void setMockGeolocationPosition(QWebPage*, double latitude, double longitude, double accuracy);
     static void setMockGeolocationError(QWebPage*, int errorCode, const QString& message);
+    static int numberOfPendingGeolocationPermissionRequests(QWebPage*);
 
     static int workerThreadCount();
 

third_party/WebKit/Tools/ChangeLog

+2011-04-06  Benjamin Poulain  <benjamin.poulain@nokia.com>
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        [Qt] LayoutTestController needs to implement numberOfPendingGeolocationPermissionRequests
+        https://bugs.webkit.org/show_bug.cgi?id=56086
+
+        Add the missing method.
+
+        * DumpRenderTree/qt/DumpRenderTreeQt.cpp:
+        (WebCore::DumpRenderTree::getAllPages): Add a method to return all the pages allocated for the current test.
+        * DumpRenderTree/qt/DumpRenderTreeQt.h:
+        * DumpRenderTree/qt/LayoutTestControllerQt.cpp:
+        (LayoutTestController::numberOfPendingGeolocationPermissionRequests):
+
 2011-04-06  Dai Mikurube  <dmikurube@chromium.org>
 
         Reviewed by David Levin.

third_party/WebKit/Tools/DumpRenderTree/qt/DumpRenderTreeQt.cpp

@@ -1136,6 +1136,17 @@ void DumpRenderTree::switchFocus(bool focused)
 
 }
 
+QList<WebPage*> DumpRenderTree::getAllPages() const
+{
+    QList<WebPage*> pages;
+    pages.append(m_page);
+    foreach (QObject* widget, windows) {
+        if (WebPage* page = widget->findChild<WebPage*>())
+            pages.append(page);
+    }
+    return pages;
+}
+
 #if defined(Q_WS_X11)
 void DumpRenderTree::initializeFonts()
 {

third_party/WebKit/Tools/DumpRenderTree/qt/DumpRenderTreeQt.h

@@ -100,6 +100,7 @@ public:
     void switchFocus(bool focused);
 
     WebPage *webPage() const { return m_page; }
+    QList<WebPage*> getAllPages() const;
 
 #if defined(Q_WS_X11)
     static void initializeFonts();

third_party/WebKit/Tools/DumpRenderTree/qt/LayoutTestControllerQt.cpp

@@ -782,8 +782,12 @@ void LayoutTestController::setGeolocationPermission(bool allow)
 
 int LayoutTestController::numberOfPendingGeolocationPermissionRequests()
 {
-    // FIXME: Implement for Geolocation layout tests.
-    return -1;
+    int pendingPermissionCount = 0;
+    QList<WebCore::WebPage*> pages = m_drt->getAllPages();
+    foreach (WebCore::WebPage* page, pages)
+        pendingPermissionCount += DumpRenderTreeSupportQt::numberOfPendingGeolocationPermissionRequests(page);
+
+    return pendingPermissionCount;
 }
 
 void LayoutTestController::setGeolocationPermissionCommon(bool allow)