Use std::array instead of custom Vector implementation when calling MD5 checksum(); and use a constant for hash size instead of integer values defined at call sites.
Created attachment 218875 [details] Patch
Comment on attachment 218875 [details] Patch Attachment 218875 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/45958329
Comment on attachment 218875 [details] Patch Attachment 218875 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/44518192
Created attachment 218886 [details] Patch
Comment on attachment 218886 [details] Patch Clearing flags on attachment: 218886 Committed r160386: <http://trac.webkit.org/changeset/160386>
All reviewed patches have been landed. Closing bug.
Comment on attachment 218886 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=218886&action=review > Source/WebCore/platform/network/curl/CurlCacheEntry.cpp:213 > + for (size_t i = 0; i < MD5::hasSize; i++) This won}t compile because it says “hasSize” instead of “hashSize”.
Thanks for the feedback, the typo fix is already landed in r160430 (https://bugs.webkit.org/show_bug.cgi?id=125571)