Fast is good!
Created attachment 45255 [details] Patch
style-queue ran check-webkit-style on attachment 45255 [details] without any errors.
Comment on attachment 45255 [details] Patch remove_api seems unrelated. I don't see why you broke queue_name and bug_id out into local variables in UpdateStatus Why change it to use a regexp if we compile every time? 74 regexp = re.compile("-") 75 return regexp.sub("_", dashed_name) replace is simpler, no? Why do we have 3 copies of the string "attachment-summary"? Seems error prone. Looks OK though.
Comment on attachment 45255 [details] Patch Clearing flags on attachment: 45255 Committed r52404: <http://trac.webkit.org/changeset/52404>
All reviewed patches have been landed. Closing bug.