EWS does not need to process obsolete patches
Created attachment 71499 [details] wip
Created attachment 71585 [details] Patch
Comment on attachment 71585 [details] Patch Hum... Feels like a big testing tax.
I'm not sure what you mean. Are you suggesting that our shared fixtures are a burden? I would agree they are, but the problem is that if we don't use them then we get in to the problems that this patch fixes: of having one part of the code look at one fixture while another looks at different one. Using shared fixtures throughout makes this problem go away, but does cause our tests to be tightly coupled and make changing those central fixures more painful.
I just meant that the ratio of code change to test change was really out of whack in this patch. I think you're improving things so that will be less the case going forward, but we need to watch this stuff to make sure new code and new tests won't have the same problem.
Comment on attachment 71585 [details] Patch Clearing flags on attachment: 71585 Committed r70328: <http://trac.webkit.org/changeset/70328>
All reviewed patches have been landed. Closing bug.