[Qt] Remove empty function stubs from TestRunnerQt.
authorzeno.albisser@digia.com <zeno.albisser@digia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 20 Mar 2013 13:29:41 +0000 (13:29 +0000)
committerzeno.albisser@digia.com <zeno.albisser@digia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 20 Mar 2013 13:29:41 +0000 (13:29 +0000)
https://bugs.webkit.org/show_bug.cgi?id=112799

We are reducing the code in TestRunnerQt step by step,
in order to finally replace all of it with the generic
TestRunner implementation.
This patch removes several simple function stubs that
currently do not implement any actual functionality.

Reviewed by Kenneth Rohde Christiansen.

* DumpRenderTree/qt/TestRunnerQt.cpp:
* DumpRenderTree/qt/TestRunnerQt.h:
(TestRunnerQt):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@146337 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/DumpRenderTree/qt/TestRunnerQt.cpp
Tools/DumpRenderTree/qt/TestRunnerQt.h

index b3fcfe1..764cbc0 100644 (file)
@@ -1,3 +1,20 @@
+2013-03-20  Zeno Albisser  <zeno@webkit.org>
+
+        [Qt] Remove empty function stubs from TestRunnerQt.
+        https://bugs.webkit.org/show_bug.cgi?id=112799
+
+        We are reducing the code in TestRunnerQt step by step,
+        in order to finally replace all of it with the generic
+        TestRunner implementation.
+        This patch removes several simple function stubs that
+        currently do not implement any actual functionality.
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        * DumpRenderTree/qt/TestRunnerQt.cpp:
+        * DumpRenderTree/qt/TestRunnerQt.h:
+        (TestRunnerQt):
+
 2013-03-20  Sudarsana Nagineni  <sudarsana.nagineni@intel.com>
 
         [EFL][WK2] Don't show the SearchBar by default when MiniBrowser opened
index 7faf233..7771391 100644 (file)
@@ -227,11 +227,6 @@ void TestRunnerQt::removeAllWebNotificationPermissions()
     DumpRenderTreeSupportQt::clearNotificationPermissions();
 }
 
-void TestRunnerQt::simulateWebNotificationClick(const QWebElement& notification)
-{
-    // FIXME: implement.
-}
-
 void TestRunnerQt::simulateLegacyWebNotificationClick(const QString& title)
 {
     DumpRenderTreeSupportQt::simulateDesktopNotificationClick(title);
@@ -483,11 +478,6 @@ void TestRunnerQt::setPopupBlockingEnabled(bool enable)
     m_drt->webPage()->settings()->setAttribute(QWebSettings::JavascriptCanOpenWindows, !enable);
 }
 
-void TestRunnerQt::setPluginsEnabled(bool flag)
-{
-    // FIXME: Implement
-}
-
 void TestRunnerQt::setPOSIXLocale(const QString& locale)
 {
     QLocale qlocale(locale);
@@ -499,11 +489,6 @@ void TestRunnerQt::setWindowIsKey(bool isKey)
     m_drt->switchFocus(isKey);
 }
 
-void TestRunnerQt::setMainFrameIsFirstResponder(bool isFirst)
-{
-    //FIXME: only need this for the moment: https://bugs.webkit.org/show_bug.cgi?id=32990
-}
-
 void TestRunnerQt::setJavaScriptCanAccessClipboard(bool enable)
 {
     m_drt->webPage()->settings()->setAttribute(QWebSettings::JavascriptCanAccessClipboard, enable);
@@ -519,27 +504,11 @@ void TestRunnerQt::setXSSAuditorEnabled(bool enable)
     m_drt->webPage()->settings()->setAttribute(QWebSettings::XSSAuditingEnabled, enable);
 }
 
-void TestRunnerQt::dispatchPendingLoadRequests()
-{
-    // FIXME: Implement for testing fix for 6727495
-}
-
 void TestRunnerQt::clearAllApplicationCaches()
 {
     DumpRenderTreeSupportQt::clearAllApplicationCaches();
 }
 
-void TestRunnerQt::clearApplicationCacheForOrigin(const QString& url)
-{
-    // FIXME: Implement to support deleting all application caches for an origin.
-}
-
-long long TestRunnerQt::localStorageDiskUsageForOrigin(const QString& originIdentifier)
-{
-    // FIXME: Implement to support getting disk usage in bytes for an origin.
-    return 0;
-}
-
 void TestRunnerQt::setApplicationCacheOriginQuota(unsigned long long quota)
 {
     if (!m_topLoadingFrame)
@@ -547,28 +516,12 @@ void TestRunnerQt::setApplicationCacheOriginQuota(unsigned long long quota)
     m_topLoadingFrame->securityOrigin().setApplicationCacheQuota(quota);
 }
 
-long long TestRunnerQt::applicationCacheDiskUsageForOrigin(const QString& origin)
-{
-    // FIXME: Implement to support getting disk usage by all application caches for an origin.
-    return 0;
-}
-
 QStringList TestRunnerQt::originsWithApplicationCache()
 {
     // FIXME: Implement to get origins that have application caches.
     return QStringList();
 }
 
-void TestRunnerQt::setCacheModel(int model)
-{
-    // qwebsetting doesn't have matched setting yet :
-    // WEBKIT_CACHE_MODEL_DOCUMENT_VIEWER
-    // WEBKIT_CACHE_MODEL_DOCUMENT_BROWSER
-    // WEBKIT_CACHE_MODEL_WEB_BROWSER
-
-    // FIXME: Implement.
-}
-
 void TestRunnerQt::setDatabaseQuota(int size)
 {
     if (!m_topLoadingFrame)
@@ -747,11 +700,6 @@ bool TestRunner::findString(JSContextRef context, JSStringRef string, JSObjectRe
     return drt->webPage()->findText(JSStringCopyQString(string), findFlags);
 }
 
-void TestRunnerQt::authenticateSession(const QString&, const QString&, const QString&)
-{
-    // FIXME: If there is a concept per-session (per-process) credential storage, the credentials should be added to it for later use.
-}
-
 void TestRunnerQt::setIconDatabaseEnabled(bool enable)
 {
     if (enable && !m_drt->persistentStoragePath().isEmpty())
@@ -805,29 +753,12 @@ void TestRunnerQt::setMockGeolocationPosition(double latitude, double longitude,
         DumpRenderTreeSupportQt::setMockGeolocationPosition(page->handle(), latitude, longitude, accuracy);
 }
 
-void TestRunnerQt::addMockSpeechInputResult(const QString& result, double confidence, const QString& language)
-{
-    // FIXME: Implement for speech input layout tests.
-    // See https://bugs.webkit.org/show_bug.cgi?id=39485.
-}
-
-void TestRunnerQt::setMockSpeechInputDumpRect(bool flag)
-{
-    // FIXME: Implement for speech input layout tests.
-    // See https://bugs.webkit.org/show_bug.cgi?id=39485.
-}
-
 void TestRunnerQt::startSpeechInput(const QString& inputElement)
 {
     // FIXME: Implement for speech input layout tests.
     // See https://bugs.webkit.org/show_bug.cgi?id=39485.
 }
 
-void TestRunnerQt::evaluateScriptInIsolatedWorldAndReturnValue(int worldID, const QString& script)
-{
-    // FIXME: Implement.
-}
-
 void TestRunnerQt::evaluateScriptInIsolatedWorld(int worldID, const QString& script)
 {
     DumpRenderTreeSupportQt::evaluateScriptInIsolatedWorld(m_drt->mainFrameAdapter(), worldID, script);
@@ -850,46 +781,6 @@ void TestRunnerQt::addURLToRedirect(const QString& origin, const QString& destin
     DumpRenderTreeSupportQt::addURLToRedirect(origin, destination);
 }
 
-void TestRunnerQt::originsWithLocalStorage()
-{
-    // FIXME: Implement.
-}
-
-void TestRunnerQt::deleteAllLocalStorage()
-{
-    // FIXME: Implement.
-}
-
-void TestRunnerQt::deleteLocalStorageForOrigin(const QString& originIdentifier)
-{
-    // FIXME: Implement.
-}
-
-void TestRunnerQt::observeStorageTrackerNotifications(unsigned number)
-{
-    // FIXME: Implement.
-}
-
-void TestRunnerQt::syncLocalStorage()
-{
-    // FIXME: Implement.
-}
-
-void TestRunnerQt::resetPageVisibility()
-{
-    // FIXME: Implement this.
-}
-
-void TestRunnerQt::setPageVisibility(const char*)
-{
-    // FIXME: Implement this.
-}
-
-void TestRunnerQt::setAutomaticLinkDetectionEnabled(bool)
-{
-    // FIXME: Implement this.
-}
-
 void TestRunnerQt::setTextDirection(const QString& directionName)
 {
     if (directionName == "auto")
index 96a6111..4b16b8f 100644 (file)
@@ -119,7 +119,6 @@ public Q_SLOTS:
     void grantWebNotificationPermission(const QString& origin);
     void denyWebNotificationPermission(const QString& origin);
     void removeAllWebNotificationPermissions();
-    void simulateWebNotificationClick(const QWebElement&);
     void display();
     void displayInvalidatedRegion();
     void clearBackForwardList();
@@ -139,12 +138,10 @@ public Q_SLOTS:
     void setFixedContentsSize(int width, int height);
     void setPrivateBrowsingEnabled(bool);
     void setSpatialNavigationEnabled(bool);
-    void setPluginsEnabled(bool flag);
     void setPopupBlockingEnabled(bool);
     void setPOSIXLocale(const QString& locale);
     void resetLoadFinished() { m_loadFinished = false; }
     void setWindowIsKey(bool);
-    void setMainFrameIsFirstResponder(bool);
     void setDeferMainResourceDataLoad(bool);
     void setJavaScriptCanAccessClipboard(bool enable);
     void setXSSAuditorEnabled(bool);
@@ -157,14 +154,9 @@ public Q_SLOTS:
     void addOriginAccessWhitelistEntry(const QString& sourceOrigin, const QString& destinationProtocol, const QString& destinationHost, bool allowDestinationSubdomains);
     void removeOriginAccessWhitelistEntry(const QString& sourceOrigin, const QString& destinationProtocol, const QString& destinationHost, bool allowDestinationSubdomains);
 
-    void dispatchPendingLoadRequests();
-
     void clearAllApplicationCaches();
-    void clearApplicationCacheForOrigin(const QString& url);
     void setApplicationCacheOriginQuota(unsigned long long);
     QStringList originsWithApplicationCache();
-    long long applicationCacheDiskUsageForOrigin(const QString&); 
-    void setCacheModel(int);
 
     void setDatabaseQuota(int size);
     void clearAllDatabases();
@@ -188,15 +180,8 @@ public Q_SLOTS:
     bool isGeolocationPermissionSet() const { return m_isGeolocationPermissionSet; }
     bool geolocationPermission() const { return m_geolocationPermission; }
 
-    void addMockSpeechInputResult(const QString& result, double confidence, const QString& language);
-    void setMockSpeechInputDumpRect(bool flag);
     void startSpeechInput(const QString& inputElement);
 
-    void setPageVisibility(const char*);
-    void resetPageVisibility();
-
-    void setAutomaticLinkDetectionEnabled(bool);
-
     void addURLToRedirect(const QString& origin, const QString& destination);
 
     /*
@@ -205,19 +190,9 @@ public Q_SLOTS:
     */
     void setScrollbarPolicy(const QString& orientation, const QString& policy);
 
-    // Simulate a request an embedding application could make, populating per-session credential storage.
-    void authenticateSession(const QString& url, const QString& username, const QString& password);
-
-    void evaluateScriptInIsolatedWorldAndReturnValue(int worldID, const QString& script);
     void evaluateScriptInIsolatedWorld(int worldID, const QString& script);
     void addUserStyleSheet(const QString& sourceCode);
     
-    void originsWithLocalStorage();
-    void deleteAllLocalStorage();
-    void deleteLocalStorageForOrigin(const QString& originIdentifier);
-    long long localStorageDiskUsageForOrigin(const QString& originIdentifier);
-    void observeStorageTrackerNotifications(unsigned number);
-    void syncLocalStorage();
     void setTextDirection(const QString& directionName);
     void goBack();
     void setDefersLoading(bool);