SHA1 size is used at various places but usually defined as a local variable. A common constant in SHA1.h used by all parties would be a better solution.
Created attachment 218593 [details] Patch
Comment on attachment 218593 [details] Patch I like this change, but could you also make computeHash take an std::array<uint8_t, hashSize>. There's no need for it to take a Vector.
Comment on attachment 218593 [details] Patch Clearing flags on attachment: 218593 Committed r160228: <http://trac.webkit.org/changeset/160228>
All reviewed patches have been landed. Closing bug.