RESOLVED FIXED 102013
Replace unnecessary ArgumentDecoder member functions with decode overloads
https://bugs.webkit.org/show_bug.cgi?id=102013
Summary Replace unnecessary ArgumentDecoder member functions with decode overloads
Anders Carlsson
Reported 2012-11-12 17:01:47 PST
Replace unnecessary ArgumentDecoder member functions with decode overloads
Attachments
Patch (32.97 KB, patch)
2012-11-12 17:19 PST, Anders Carlsson
no flags
Patch (28.20 KB, patch)
2013-01-29 18:52 PST, Sam Weinig
andersca: review+
Anders Carlsson
Comment 1 2012-11-12 17:19:31 PST
Early Warning System Bot
Comment 2 2012-11-12 17:32:40 PST
EFL EWS Bot
Comment 3 2012-11-12 17:43:53 PST
Build Bot
Comment 4 2012-11-12 18:11:45 PST
Csaba Osztrogonác
Comment 5 2013-01-15 23:18:26 PST
Comment on attachment 173767 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=173767&action=review > Source/WebKit2/Platform/win/SharedMemoryWin.cpp:101 > + if (!decoder->decode(32(sourcePID)) I think you meant decode(size) > Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp:205 > + if (!decoder->decode(32(size)) ditto > Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp:283 > + if (!decoder->decode(32(parametersSize)) ditto > Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp:299 > + if (!decoder->decode(32(arrayParameterSize)) ditto > Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp:313 > + if (!decoder->decode(32(numberParameterSize)) ditto > Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp:425 > + if (!decoder->decode(32(operationsSize)) ditto > Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp:582 > + if (!decoder->decode(32(numberOfSteps)) ditto > Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp:666 > + if (!decoder->decode(32(fillMode)) ditto > Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp:688 > + if (!decoder->decode(32(keyframesSize)) ditto
Csaba Osztrogonác
Comment 6 2013-01-15 23:19:20 PST
(In reply to comment #5) > (From update of attachment 173767 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=173767&action=review > > > Source/WebKit2/Platform/win/SharedMemoryWin.cpp:101 > > + if (!decoder->decode(32(sourcePID)) > > I think you meant decode(size) decode(size) -> decode(*)
Sam Weinig
Comment 7 2013-01-29 18:52:19 PST
Sam Weinig
Comment 8 2013-01-29 19:30:50 PST
Note You need to log in before you can comment on or make changes to this bug.