<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On May 21, 2009, at 9:10 PM, Peter Kasting wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote">On Thu, May 21, 2009 at 8:52 PM, Eric Seidel <span dir="ltr">&lt;<a href="mailto:eric@webkit.org">eric@webkit.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"> Our current defacto policy requires involvement on both sides.<br> Submitters need to be involved in finding people to review their<br> patches. &nbsp;Posting patches to the review queue does not automatically<br> get you a review, except occasionally by Darin Adler or myself.</blockquote><div><br></div><div>FWIW, I've always been told that WebKit&nbsp;etiquette&nbsp;is to not choose a reviewer (even when you know who would be good) and just mark a patch r?. &nbsp;(This is a stark contrast to the Mozilla way of _always_ selecting a reviewer.)</div> <div><br></div><div>It would probably be good to clarify this kind of thing.</div></div></blockquote><br></div><div>It's better not to explicitly select a reviewer, since almost always there will be several people qualified. However, it is totally OK to ask a specific person if they would consider reviewing out of band of the bug system, or to Cc possible qualified reviewers on the bug.</div><div><br></div><div>Regards,</div><div>Maciej</div><div><br></div></body></html>