[webkit-dev] Extra review requirement for web-facing API? (Was: Re: Spellcheck API for WebKit)

[Antonio Gomes]

> On Wed, Jun 22, 2011 at 11:41 AM, Dimitri Glazkov <dglazkov at chromium.org>
>  wrote:
>
>> To prevent this from happening again, we should remind everyone to:
>>
>> 1) Only review things they are comfortable with;
>> 2) Seek WebKit elder's review for public-facing APIs
>>
>> I don't think we need an explicit two-level review policy. If
>> anything, we need super-reviewers instead.
>
>
> I'm supportive of super-reviewers idea.
>
>
I suport the super review process too for Web facing API addition, including
new css properties and friends. BUT conditionally to the extra level of
reviewing being exclusive for such cases. For other cases, looking for a
reviewer that domains this code is still the best thing to do.


-- 
--Antonio Gomes
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20110622/05508146/attachment.html>