Bug 134294 – Clean up the WebKit build from unused parameter warning in Webkit2/UIProcess module

• Component: WebKit2
• Status: RESOLVED FIXED

[Rohit]

Clean up WebKit build from usused parameters in newly added functionality in Webkit2/UIProcess module.

Build logs:

Linking CXX executable ../../bin/TestWebKitAPI/WebCore/LayoutUnit
[ 82%] Built target web-inspector-resources
[ 82%] Built target WebCoreTestSupport
[ 82%] Built target LayoutUnit
[ 82%] Built target URL
Scanning dependencies of target WebKit2
[ 82%] [ 82%] Building CXX object Source/WebKit2/CMakeFiles/WebKit2.dir/UIProcess/API/C/WKPage.cpp.o
Building CXX object Source/WebKit2/CMakeFiles/WebKit2.dir/UIProcess/WebPageProxy.cpp.o
In file included from /home/kumar.rohit/WebKit/Source/WebKit2/UIProcess/API/C/WKPage.cpp:35:0:
/home/kumar.rohit/WebKit/Source/WebKit2/UIProcess/API/APIUIClient.h:118:18: warning: unused parameter ‘totalBytesNeeded’ [-Wunused-parameter]
     virtual void reachedApplicationCacheOriginQuota(WebKit::WebPageProxy*, const WebCore::SecurityOrigin&, uint64_t currentQuota, uint64_t totalBytesNeeded, std::function<void (unsigned long long)> completionHandler)
                  ^
In file included from /home/kumar.rohit/WebKit/Source/WebKit2/UIProcess/WebPageProxy.cpp:35:0:
/home/kumar.rohit/WebKit/Source/WebKit2/UIProcess/API/APIUIClient.h:118:18: warning: unused parameter ‘totalBytesNeeded’ [-Wunused-parameter]
     virtual void reachedApplicationCacheOriginQuota(WebKit::WebPageProxy*, const WebCore::SecurityOrigin&, uint64_t currentQuota, uint64_t totalBytesNeeded, std::function<void (unsigned long long)> completionHandler)
                  ^
Linking CXX shared library ../../lib/libewebkit2.so

[Rohit]

Created attachment 233799 [details]
Patch

[Build Bot]

Comment on attachment 233799 [details]
Patch

Attachment 233799 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/4958240152485888

New failing tests:
media/W3C/video/networkState/networkState_during_loadstart.html

[Build Bot]

Created attachment 233800 [details]
Archive of layout-test-results from webkit-ews-11 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-11  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5

[Shivakumar J M]

As per the rule https://lists.webkit.org/pipermail/webkit-dev/2012-October/022369.html, we can omit parameter names for unused parameters entirely.

[Rohit]

(In reply to comment #4)
> As per the rule https://lists.webkit.org/pipermail/webkit-dev/2012-October/022369.html, we can omit parameter names for unused parameters entirely.


Yes, I tried that. But if we remove parameter name in function defintion then it shows error in check-webkit-style script.

[Shivakumar J M]

(In reply to comment #5)
> (In reply to comment #4)
> > As per the rule https://lists.webkit.org/pipermail/webkit-dev/2012-October/022369.html, we can omit parameter names for unused parameters entirely.
> 
> 
> Yes, I tried that. But if we remove parameter name in function defintion then it shows error in check-webkit-style script.

I feel that error is reported, without these fix as well, if we just run check-webkit-style for the file APIUIClient.h, it lists similar errors in some other lines also, for these kind of parameter( std::function<void (unsigned long long)> completionHandler). so we can fix that space error as per coding idiom or check can it be ignored as per coding idiom.

[Rohit]

Created attachment 233888 [details]
Patch

[WebKit Commit Bot]

Attachment 233888 [details] did not pass style-queue:


ERROR: Source/WebKit2/UIProcess/API/APIUIClient.h:118:  Extra space before ( in function call  [whitespace/parens] [4]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Rohit]

(In reply to comment #6)
> (In reply to comment #5)
> > (In reply to comment #4)
> > > As per the rule https://lists.webkit.org/pipermail/webkit-dev/2012-October/022369.html, we can omit parameter names for unused parameters entirely.
> > 
> > 
> > Yes, I tried that. But if we remove parameter name in function defintion then it shows error in check-webkit-style script.
> 
> I feel that error is reported, without these fix as well, if we just run check-webkit-style for the file APIUIClient.h, it lists similar errors in some other lines also, for these kind of parameter( std::function<void (unsigned long long)> completionHandler). so we can fix that space error as per coding idiom or check can it be ignored as per coding idiom.

Although I am not facing this space error if I don't omit parameter name, but I have cleaned up the warning using parameter name omission. I guess it happens because check-webkit-style script checks for only that particular statement where changes are made and hence it shows error only in that line and no other line in that file. For current patch I have ignored the warning.

[Gyuyoung Kim]

LGTM based on Darin's comment.
https://bugs.webkit.org/show_bug.cgi?id=131260#c8

However, I would like to let Darin review this patch.

[WebKit Commit Bot]

Comment on attachment 233888 [details]
Patch

Clearing flags on attachment: 233888

Committed r170643: <http://trac.webkit.org/changeset/170643>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.