Bug 40345

Summary: Don't send mail for cancel/deny/granted for in-rietveld
Product: WebKit Reporter: Julie Parent <jparent>
Component: Tools / TestsAssignee: Julie Parent <jparent>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, ddkilzer, ojan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
patch ddkilzer: review+

Julie Parent
Reported 2010-06-08 18:07:23 PDT
We already don't send the regular flag change mail, but we also shouldn't send the specific mail for cancel/deny/granted that comes from state change from ? -> {+,-, ).
Attachments
patch (1.14 KB, patch)
2010-06-09 10:56 PDT, Julie Parent 		ddkilzer: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Julie Parent
Comment 1 2010-06-09 10:56:19 PDT
Created attachment 58261 [details] patch
Adam Barth
Comment 2 2010-06-09 11:13:17 PDT
Comment on attachment 58261 [details] patch LGTM, but BugsSite and I aren't friends. :)
David Kilzer (:ddkilzer)
Comment 3 2010-06-09 11:17:37 PDT
Comment on attachment 58261 [details] patch r=me
Julie Parent
Comment 4 2010-06-09 17:35:40 PDT
Committed http://trac.webkit.org/changeset/60904
Note You need to log in before you can comment on or make changes to this bug.