Removing general compiling warnings from WebKit2 and WebCore
Created attachment 214684 [details] Patch
Comment on attachment 214684 [details] Patch Looks like these are all unused parameter warnings? If so, could you update the title of the bug and ChangeLog?
(In reply to comment #2) > (From update of attachment 214684 [details]) > Looks like these are all unused parameter warnings? If so, could you update the title of the bug and ChangeLog? Sure :)
Created attachment 214698 [details] Patch
Comment on attachment 214698 [details] Patch Clearing flags on attachment: 214698 Committed r157700: <http://trac.webkit.org/changeset/157700>
All reviewed patches have been landed. Closing bug.
Comment on attachment 214698 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=214698&action=review > Source/WebCore/Modules/mediastream/RTCDTMFToneChangeEvent.cpp:47 > ASSERT(type == eventNames().tonechangeEvent); Remove the type parameter will break the debug build due to the assert using it.
(In reply to comment #7) > (From update of attachment 214698 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=214698&action=review > > > Source/WebCore/Modules/mediastream/RTCDTMFToneChangeEvent.cpp:47 > > ASSERT(type == eventNames().tonechangeEvent); > > Remove the type parameter will break the debug build due to the assert using it. Sorry, you are right. Didn't pay attention to the debug build because I thought that the build bots would alert me. Maybe they don't have MediaStream enabled. Fixing that in other patch now