[Webkit-unassigned] [Bug 63502] [EFL] Add EflWidgetBackingStoreCairo

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=63502





--- Comment #20 from Lucas De Marchi <demarchi at webkit.org>  2011-09-21 06:25:46 PST ---
(In reply to comment #18)
> (In reply to comment #17)
> > (From update of attachment 107977 [details] [details])
> > Yeah, I also think this is the way to go, so I'm r-'ing this patch.
> > 
> 
> As far as I know, we should not set r+ nor r- in unofficial review.
> 
> " Note that you should not put r+ nor r- on patches in such unofficial reviews."
> http://www.webkit.org/coding/commit-review-policy.html

This is not how things are working. Since official reviewers want our informal r+, letting it as "r?" will only make a lot of bugs appear as unreviewed on  "webkit-patch patches-to-review". And this is what Eric Seidel, for e.g., uses to review EFL bug requests. In fact he said me it's fine for informal-reviewers to set r- after we had some problems with him rubber-stamping bugs that were already informally r- but without removing the "r?".

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.