We should allow non-reviewers to set r- on patches. For e.g.: In case a reviewer who has no expertise in a particular tricky area of code (and so probably shouldn’t r+), notices an obvious error (e.g. a specific link that accesses memory out-of-bounds) it would be appropriate (but not required), for him/her to set r- on the patch. This would help other reviewers with the review.
Created attachment 362262 [details] Proposed patch
Comment on attachment 362262 [details] Proposed patch I disagree with this policy change. I think it makes it makes things more confusing for no real gain. Ideally, we should codify this rule by not allowing non-reviewers to change the review flag to r+ or r-.
Seems like this is WontFix given the latest webkit-dev discussions.