Summary: | REGRESSION: webkit-patch land can't land "rubber-stamped" patches | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eric Seidel (no email) <eric> | ||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abarth, commit-queue | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | OS X 10.5 | ||||||
Attachments: |
|
Description
Eric Seidel (no email)
2010-03-25 00:07:29 PDT
Sigh. I don't think that we should be prescribing "rubber stamp". It's common for folks to use "rubber-stamped". ok Created attachment 51690 [details]
Patch
Comment on attachment 51690 [details]
Patch
I still think this is error prone. I'm also saddened by lack of testing on this step.
Comment on attachment 51690 [details]
Patch
Yeah. It's a bit of an experiment. We might need to try a different approach at some point.
Comment on attachment 51690 [details] Patch Clearing flags on attachment: 51690 Committed r56589: <http://trac.webkit.org/changeset/56589> All reviewed patches have been landed. Closing bug. |