DescriptionDo not treat auto_generated messages as published messages.
This is to ensure that no matter what logging is done before (reviewers added, CQ checked, etc), the first published
message contains the diff and does not contain 'Re:'.
As part of this CL I also reverted the workarounds done in:
* https://codereview.appspot.com/126680043/ (Delay the creation of the reviewers change log message to avoid "Re:" subject on initial comment)
* https://codereview.appspot.com/133880043/ (Fix initial diff email feature by calling put() on auto-generated Messages later)
This CL replaces <a href='https://code.google.com/p/rietveld/source/detail?r=af712bf0708b4ec1043c2783e197a75959a40006&name=chromium'>https://code.google.com/p/rietveld/source/detail?r=af712bf0708b4ec1043c2783e197a75959a40006&name=chromium</a> with a better approach.
Tested with the following CLs:
* https://skia-codereview-staging.appspot.com/6841001/ (testing add-reviewers-in-edit-then-publish)
* https://skia-codereview-staging.appspot.com/1871001/ (testing add-reviewers-in-publish)
* https://skia-codereview-staging.appspot.com/8831001/ (testing check-cq-then-publish)
BUG=<a href='https://code.google.com/p/chromium/issues/detail?id=407382'>chromium:407382</a>
Patch Set 1 : Initial upload #Patch Set 2 : Cleanup #Patch Set 3 : Making sure nothing changed locally #
MessagesTotal messages: 5
|