Fix unused parameter compile warnings
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 14 Dec 2012 01:15:08 +0000 (01:15 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 14 Dec 2012 01:15:08 +0000 (01:15 +0000)
https://bugs.webkit.org/show_bug.cgi?id=104907

Patch by Seokju Kwon <seokju.kwon@gmail.com> on 2012-12-13
Reviewed by Kentaro Hara.

Use UNUSED_PARAM macro to fix build warning -Wunused-parameter
when ENABLE_INSPECTOR is disabled.

Source/WebKit/efl:

* WebCoreSupport/DumpRenderTreeSupportEfl.cpp:
(DumpRenderTreeSupportEfl::evaluateInWebInspector):

Source/WebKit2:

* UIProcess/API/C/WKInspector.cpp:
(WKInspectorGetPage):
(WKInspectorIsVisible):
(WKInspectorIsFront):
(WKInspectorShow):
(WKInspectorClose):
(WKInspectorShowConsole):
(WKInspectorShowResources):
(WKInspectorShowMainResourceForFrame):
(WKInspectorIsAttached):
(WKInspectorAttach):
(WKInspectorDetach):
(WKInspectorIsDebuggingJavaScript):
(WKInspectorToggleJavaScriptDebugging):
(WKInspectorIsProfilingJavaScript):
(WKInspectorToggleJavaScriptProfiling):
(WKInspectorIsProfilingPage):
(WKInspectorTogglePageProfiling):
* UIProcess/API/C/WKPage.cpp:
(WKPageGetInspector):

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

Source/WebKit/efl/ChangeLog
Source/WebKit/efl/WebCoreSupport/DumpRenderTreeSupportEfl.cpp
Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/API/C/WKInspector.cpp
Source/WebKit2/UIProcess/API/C/WKPage.cpp

index 9a52b50..e01e9e2 100644 (file)
@@ -1,3 +1,16 @@
+2012-12-13  Seokju Kwon  <seokju.kwon@gmail.com>
+
+        Fix unused parameter compile warnings
+        https://bugs.webkit.org/show_bug.cgi?id=104907
+
+        Reviewed by Kentaro Hara.
+
+        Use UNUSED_PARAM macro to fix build warning -Wunused-parameter
+        when ENABLE_INSPECTOR is disabled.
+
+        * WebCoreSupport/DumpRenderTreeSupportEfl.cpp:
+        (DumpRenderTreeSupportEfl::evaluateInWebInspector):
+
 2012-12-13  Jussi Kukkonen  <jussi.kukkonen@intel.com>
 
         [EFL][GTK] Don't call deprecated g_type_init when glib => 2.35
index 14ff753..6277c26 100644 (file)
@@ -62,6 +62,7 @@
 #include <history/HistoryItem.h>
 #include <workers/WorkerThread.h>
 #include <wtf/HashMap.h>
+#include <wtf/UnusedParam.h>
 
 #if ENABLE(GEOLOCATION)
 #include <GeolocationClientMock.h>
@@ -525,6 +526,10 @@ void DumpRenderTreeSupportEfl::evaluateInWebInspector(const Evas_Object* ewkView
 
     if (page->inspectorController())
         page->inspectorController()->evaluateForTestInFrontend(callId, script);
+#else
+    UNUSED_PARAM(ewkView);
+    UNUSED_PARAM(callId);
+    UNUSED_PARAM(script);
 #endif
 }
 
index 30dfac3..c8e7238 100644 (file)
@@ -1,5 +1,36 @@
 2012-12-13  Seokju Kwon  <seokju.kwon@gmail.com>
 
+        Fix unused parameter compile warnings
+        https://bugs.webkit.org/show_bug.cgi?id=104907
+
+        Reviewed by Kentaro Hara.
+
+        Use UNUSED_PARAM macro to fix build warning -Wunused-parameter
+        when ENABLE_INSPECTOR is disabled.
+
+        * UIProcess/API/C/WKInspector.cpp:
+        (WKInspectorGetPage):
+        (WKInspectorIsVisible):
+        (WKInspectorIsFront):
+        (WKInspectorShow):
+        (WKInspectorClose):
+        (WKInspectorShowConsole):
+        (WKInspectorShowResources):
+        (WKInspectorShowMainResourceForFrame):
+        (WKInspectorIsAttached):
+        (WKInspectorAttach):
+        (WKInspectorDetach):
+        (WKInspectorIsDebuggingJavaScript):
+        (WKInspectorToggleJavaScriptDebugging):
+        (WKInspectorIsProfilingJavaScript):
+        (WKInspectorToggleJavaScriptProfiling):
+        (WKInspectorIsProfilingPage):
+        (WKInspectorTogglePageProfiling):
+        * UIProcess/API/C/WKPage.cpp:
+        (WKPageGetInspector):
+
+2012-12-13  Seokju Kwon  <seokju.kwon@gmail.com>
+
         [QT][GTK][EFL] Add guard for WebInspectorServer
         https://bugs.webkit.org/show_bug.cgi?id=104889
 
index 891388c..5dcd8d5 100644 (file)
@@ -28,6 +28,7 @@
 
 #include "WKAPICast.h"
 #include "WebInspectorProxy.h"
+#include <wtf/UnusedParam.h>
 
 using namespace WebKit;
 
@@ -45,6 +46,7 @@ WKPageRef WKInspectorGetPage(WKInspectorRef inspectorRef)
 #if ENABLE(INSPECTOR)
     return toAPI(toImpl(inspectorRef)->page());
 #else
+    UNUSED_PARAM(inspectorRef);
     return 0;
 #endif
 }
@@ -54,6 +56,7 @@ bool WKInspectorIsVisible(WKInspectorRef inspectorRef)
 #if ENABLE(INSPECTOR)
     return toImpl(inspectorRef)->isVisible();
 #else
+    UNUSED_PARAM(inspectorRef);
     return false;
 #endif
 }
@@ -63,6 +66,7 @@ bool WKInspectorIsFront(WKInspectorRef inspectorRef)
 #if ENABLE(INSPECTOR)
     return toImpl(inspectorRef)->isFront();
 #else
+    UNUSED_PARAM(inspectorRef);
     return false;
 #endif
 }
@@ -71,6 +75,8 @@ void WKInspectorShow(WKInspectorRef inspectorRef)
 {
 #if ENABLE(INSPECTOR)
     toImpl(inspectorRef)->show();
+#else
+    UNUSED_PARAM(inspectorRef);
 #endif
 }
 
@@ -78,6 +84,8 @@ void WKInspectorClose(WKInspectorRef inspectorRef)
 {
 #if ENABLE(INSPECTOR)
     toImpl(inspectorRef)->close();
+#else
+    UNUSED_PARAM(inspectorRef);
 #endif
 }
 
@@ -85,6 +93,8 @@ void WKInspectorShowConsole(WKInspectorRef inspectorRef)
 {
 #if ENABLE(INSPECTOR)
     toImpl(inspectorRef)->showConsole();
+#else
+    UNUSED_PARAM(inspectorRef);
 #endif
 }
 
@@ -92,6 +102,8 @@ void WKInspectorShowResources(WKInspectorRef inspectorRef)
 {
 #if ENABLE(INSPECTOR)
     toImpl(inspectorRef)->showResources();
+#else
+    UNUSED_PARAM(inspectorRef);
 #endif
 }
 
@@ -99,6 +111,9 @@ void WKInspectorShowMainResourceForFrame(WKInspectorRef inspectorRef, WKFrameRef
 {
 #if ENABLE(INSPECTOR)
     toImpl(inspectorRef)->showMainResourceForFrame(toImpl(frameRef));
+#else
+    UNUSED_PARAM(inspectorRef);
+    UNUSED_PARAM(frameRef);
 #endif
 }
 
@@ -107,6 +122,7 @@ bool WKInspectorIsAttached(WKInspectorRef inspectorRef)
 #if ENABLE(INSPECTOR)
     return toImpl(inspectorRef)->isAttached();
 #else
+    UNUSED_PARAM(inspectorRef);
     return false;
 #endif
 }
@@ -115,6 +131,8 @@ void WKInspectorAttach(WKInspectorRef inspectorRef)
 {
 #if ENABLE(INSPECTOR)
     toImpl(inspectorRef)->attach();
+#else
+    UNUSED_PARAM(inspectorRef);
 #endif
 }
 
@@ -122,6 +140,8 @@ void WKInspectorDetach(WKInspectorRef inspectorRef)
 {
 #if ENABLE(INSPECTOR)
     toImpl(inspectorRef)->detach();
+#else
+    UNUSED_PARAM(inspectorRef);
 #endif
 }
 
@@ -130,6 +150,7 @@ bool WKInspectorIsDebuggingJavaScript(WKInspectorRef inspectorRef)
 #if ENABLE(INSPECTOR)
     return toImpl(inspectorRef)->isDebuggingJavaScript();
 #else
+    UNUSED_PARAM(inspectorRef);
     return false;
 #endif
 }
@@ -138,6 +159,8 @@ void WKInspectorToggleJavaScriptDebugging(WKInspectorRef inspectorRef)
 {
 #if ENABLE(INSPECTOR)
     toImpl(inspectorRef)->toggleJavaScriptDebugging();
+#else
+    UNUSED_PARAM(inspectorRef);
 #endif
 }
 
@@ -146,6 +169,7 @@ bool WKInspectorIsProfilingJavaScript(WKInspectorRef inspectorRef)
 #if ENABLE(INSPECTOR)
     return toImpl(inspectorRef)->isProfilingJavaScript();
 #else
+    UNUSED_PARAM(inspectorRef);
     return false;
 #endif
 }
@@ -154,6 +178,8 @@ void WKInspectorToggleJavaScriptProfiling(WKInspectorRef inspectorRef)
 {
 #if ENABLE(INSPECTOR)
     toImpl(inspectorRef)->toggleJavaScriptProfiling();
+#else
+    UNUSED_PARAM(inspectorRef);
 #endif
 }
 
@@ -162,6 +188,7 @@ bool WKInspectorIsProfilingPage(WKInspectorRef inspectorRef)
 #if ENABLE(INSPECTOR)
     return toImpl(inspectorRef)->isProfilingPage();
 #else
+    UNUSED_PARAM(inspectorRef);
     return false;
 #endif
 }
@@ -170,5 +197,7 @@ void WKInspectorTogglePageProfiling(WKInspectorRef inspectorRef)
 {
 #if ENABLE(INSPECTOR)
     toImpl(inspectorRef)->togglePageProfiling();
+#else
+    UNUSED_PARAM(inspectorRef);
 #endif
 }
index fbf98d5..e0fb1cb 100644 (file)
@@ -34,6 +34,7 @@
 #include "WebPageProxy.h"
 #include "WebProcessProxy.h"
 #include <WebCore/Page.h>
+#include <wtf/UnusedParam.h>
 
 #ifdef __BLOCKS__
 #include <Block.h>
@@ -187,6 +188,7 @@ WKInspectorRef WKPageGetInspector(WKPageRef pageRef)
 #if defined(ENABLE_INSPECTOR) && ENABLE_INSPECTOR
     return toAPI(toImpl(pageRef)->inspector());
 #else
+    UNUSED_PARAM(pageRef);
     return 0;
 #endif
 }