[webkit-reviews] review denied: [Bug 67023] Add prefix to LayoutTest/http/tests/filesystem/resources/fs-worker-common.js : [Attachment 105467] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Dec 19 12:12:39 PST 2011


Eric Seidel <eric at webkit.org> has denied Taiju Tsuiki <tzik at chromium.org>'s
request for review:
Bug 67023: Add prefix to
LayoutTest/http/tests/filesystem/resources/fs-worker-common.js
https://bugs.webkit.org/show_bug.cgi?id=67023

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

------- Additional Comments from Eric Seidel <eric at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=105467&action=review


>> LayoutTests/ChangeLog:4
>> +	    https://bugs.webkit.org/show_bug.cgi?id=67023
> 
> Need to say why not just what we are doing.

I agree with Darin the ChangeLog should say "why".  Otherwise this change is
fine.


More information about the webkit-reviews mailing list