[Webkit-unassigned] [Bug 131969] [WebKit2] Cleanup the build from unused parameter in UIProcess Module

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon May 5 22:34:41 PDT 2014


https://bugs.webkit.org/show_bug.cgi?id=131969





--- Comment #3 from Shivakumar J M <shiva.jm at samsung.com>  2014-05-05 22:35:02 PST ---
(From update of attachment 230888)
>Index: Source/WebKit2/ChangeLog
>===================================================================
>--- Source/WebKit2/ChangeLog	(revision 168348)
>+++ Source/WebKit2/ChangeLog	(working copy)
>@@ -1,3 +1,17 @@
>+2014-05-05  Shivakumar JM  <shiva.jm at samsung.com>
>+
>+        Cleanup the build from unused parameter in UIProcess Module
>+        https://bugs.webkit.org/show_bug.cgi?id=131969
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Fix unused parameter by using UNUSED_PARAM macro
>+
>+        * UIProcess/API/C/WKPage.cpp:
>+        (WKPageSetPageFindClient):
>+        * UIProcess/WebFindClient.cpp:
>+        (WebKit::WebFindClient::didFindString):
>+
> 2014-05-05  Jeremy Jones  <jeremyj at apple.com>
> 
>         Implement scan backward and forward in video fullscreen.
>Index: Source/WebKit2/UIProcess/WebFindClient.cpp
>===================================================================
>--- Source/WebKit2/UIProcess/WebFindClient.cpp	(revision 168348)
>+++ Source/WebKit2/UIProcess/WebFindClient.cpp	(working copy)
>@@ -32,6 +32,8 @@ namespace WebKit {
> 
> void WebFindClient::didFindString(WebPageProxy* page, const String& string, uint32_t matchCount, int32_t matchIndex)
> {
>+    UNUSED_PARAM(matchIndex);
>+
>     if (!m_client.didFindString)
>         return;
> 
>Index: Source/WebKit2/UIProcess/API/C/WKPage.cpp
>===================================================================
>--- Source/WebKit2/UIProcess/API/C/WKPage.cpp	(revision 168348)
>+++ Source/WebKit2/UIProcess/API/C/WKPage.cpp	(working copy)
>@@ -704,6 +704,8 @@ void WKPageSetPageFindClient(WKPageRef p
>     private:
>         virtual void didFindString(WebPageProxy* page, const String& string, uint32_t matchCount, int32_t matchIndex) override
>         {
>+            UNUSED_PARAM(matchIndex);
>+
>             if (!m_client.didFindString)
>                 return;
>             
>Index: LayoutTests/ChangeLog
>===================================================================

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list