[webkit-reviews] review denied: [Bug 20534] DumpRenderTree needs a way to override settings on a per-test basis : [Attachment 27296] Alternate patch to issue 20534

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu May 21 23:16:30 PDT 2009


Maciej Stachowiak <mjs at apple.com> has denied Glenn Wilson
<gwilson at google.com>'s request for review:
Bug 20534: DumpRenderTree needs a way to override settings on a per-test basis
https://bugs.webkit.org/show_bug.cgi?id=20534

Attachment 27296: Alternate patch to issue 20534
https://bugs.webkit.org/attachment.cgi?id=27296&action=review

------- Additional Comments from Maciej Stachowiak <mjs at apple.com>
One minor comment:

+- (void)setPreference:(NSString *)key stringValue:(NSString *)value

Private ObjC methods on API classes should start with an underscore.

Otherwise looks good, assuming it doesn't break the tests now. Please resubmit
with that minor change.


More information about the webkit-reviews mailing list