Replace unnecessary ArgumentDecoder member functions with decode overloads
Created attachment 173767 [details] Patch
Comment on attachment 173767 [details] Patch Attachment 173767 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/14821342
Comment on attachment 173767 [details] Patch Attachment 173767 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/14803950
Comment on attachment 173767 [details] Patch Attachment 173767 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/14822231
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
(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(*)
Created attachment 185374 [details] Patch
Committed r141209: <http://trac.webkit.org/changeset/141209>