Bug 140005

Summary:

Fix build warnings after r177792

Product:

WebKit

Reporter:

Bartlomiej Gajda <b.gajda>

Component:

New Bugs

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

andersca, commit-queue, ossy, sam

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

139978

Attachments:

Description	Flags
patch v1	none

Description Bartlomiej Gajda 2014-12-30 04:19:26 PST

After that revision following errors appeared:
DerivedSources/WebKit2/NetworkResourceLoaderMessageReceiver.cpp:39:6: warning: unused parameter ‘connection’ [-Wunused-parameter]
DerivedSources/WebKit2/PluginProcessMessageReceiver.cpp:41:6: warning: unused parameter ‘connection’ [-Wunused-parameter]
DerivedSources/WebKit2/NetworkProcessMessageReceiver.cpp:50:6: warning: unused parameter ‘connection’ [-Wunused-parameter]
DerivedSources/WebKit2/NetworkConnectionToWebProcessMessageReceiver.cpp:80:6: warning: unused parameter ‘connection’ [-Wunused-parameter]
DerivedSources/WebKit2/WebProcessConnectionMessageReceiver.cpp:87:6: warning: unused parameter ‘connection’ [-Wunused-parameter]
DerivedSources/WebKit2/PluginControllerProxyMessageReceiver.cpp:74:6: warning: unused parameter ‘connection’ [-Wunused-parameter]
DerivedSources/WebKit2/WebFullScreenManagerMessageReceiver.cpp:37:6: warning: unused parameter ‘connection’ [-Wunused-parameter]
DerivedSources/WebKit2/WebProcessProxyMessageReceiver.cpp:132:6: warning: unused parameter ‘connection’ [-Wunused-parameter]
DerivedSources/WebKit2/NetworkProcessConnectionMessageReceiver.cpp:49:6: warning: unused parameter ‘connection’ [-Wunused-parameter]
DerivedSources/WebKit2/PluginProcessConnectionMessageReceiver.cpp:39:6: warning: unused parameter ‘connection’ [-Wunused-parameter]
DerivedSources/WebKit2/WebProcessMessageReceiver.cpp:51:6: warning: unused parameter ‘connection’ [-Wunused-parameter]
DerivedSources/WebKit2/WebResourceLoaderMessageReceiver.cpp:48:6: warning: unused parameter ‘connection’ [-Wunused-parameter]
DerivedSources/WebKit2/PluginProxyMessageReceiver.cpp:45:6: warning: unused parameter ‘connection’ [-Wunused-parameter]
DerivedSources/WebKit2/CoordinatedLayerTreeHostMessageReceiver.cpp:41:6: warning: unused parameter ‘connection’ [-Wunused-parameter]
DerivedSources/WebKit2/WebPageMessageReceiver.cpp:128:6: warning: unused parameter ‘connection’ [-Wunused-parameter]

Comment 1 Bartlomiej Gajda 2014-12-30 04:23:25 PST

Created attachment 243821 [details]
patch v1

The most important part is to add UNUSED_PARAM even if it's LegacyReceiver, but I also refactored a bit code around, so there are no unneccessary ifs.

Comment 2 WebKit Commit Bot 2014-12-31 09:17:00 PST

Comment on attachment 243821 [details]
patch v1

Clearing flags on attachment: 243821

Committed r177833: <http://trac.webkit.org/changeset/177833>

Comment 3 WebKit Commit Bot 2014-12-31 09:17:04 PST

All reviewed patches have been landed.  Closing bug.