[webkit-reviews] review granted: [Bug 116445] Move MD5, SHA1 unit tests from WTF to TestWebKitAPI : [Attachment 203369] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu May 30 16:29:57 PDT 2013


Benjamin Poulain <benjamin at webkit.org> has granted Zan Dobersek
<zandobersek at gmail.com>'s request for review:
Bug 116445: Move MD5, SHA1 unit tests from WTF to TestWebKitAPI
https://bugs.webkit.org/show_bug.cgi?id=116445

Attachment 203369: Patch
https://bugs.webkit.org/attachment.cgi?id=203369&action=review

------- Additional Comments from Benjamin Poulain <benjamin at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=203369&action=review


Great!

> Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:761
> +				CD5393C91757BAC400C07123 /* SHA1.cpp */,
> +				CD5393C71757BA9700C07123 /* MD5.cpp */,

The build section should be sorted alphabetically.

> Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:1182
>				2E7765CF16C4D81100BA2BB1 /* mainMac.mm in
Sources */,
> +				CD5393C81757BA9700C07123 /* MD5.cpp in Sources
*/,
> +				CD5393CA1757BAC400C07123 /* SHA1.cpp in Sources
*/,

Ditto.

> Tools/TestWebKitAPI/Tests/WTF/MD5.cpp:29
> +/*
> + * Copyright (C) 2010 Google Inc. All rights reserved.
> + *
> + * Redistribution and use in source and binary forms, with or without
> + * modification, are permitted provided that the following conditions are
> + * met:
> + *
> + *	  * Redistributions of source code must retain the above copyright
> + * notice, this list of conditions and the following disclaimer.
> + *	  * Redistributions in binary form must reproduce the above
> + * copyright notice, this list of conditions and the following disclaimer
> + * in the documentation and/or other materials provided with the
> + * distribution.
> + *	  * Neither the name of Google Inc. nor the names of its
> + * contributors may be used to endorse or promote products derived from
> + * this software without specific prior written permission.
> + *
> + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
> + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
> + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
> + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> + * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
> + * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
> + * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
> + * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
> + * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
> + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
> + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> + */

I am not sure this is the right copyright.

If the test comes from the original sqlite code, it should be the one starting
with "This code implements the MD5 message-digest algorithm."

> Tools/TestWebKitAPI/Tests/WTF/MD5.cpp:54
> +    // MD5 Test suite from http://www.ietf.org/rfc/rfc1321.txt

Missing period.

> Tools/TestWebKitAPI/Tests/WTF/SHA1.cpp:1
> +/*

ditto for the copyright.


More information about the webkit-reviews mailing list