Message257263
The fix looks ok, however the test seem to contain unnecessary code and it duplicates most of the previous test.
I left a more detailed review on rietveld (see the "review" link next to the patch). |
|
Date |
User |
Action |
Args |
2016-01-01 02:13:21 | ezio.melotti | set | recipients:
+ ezio.melotti, rbcollins, michael.foord, Victor van den Elzen |
2016-01-01 02:13:21 | ezio.melotti | set | messageid: <1451614401.11.0.230058329994.issue25320@psf.upfronthosting.co.za> |
2016-01-01 02:13:21 | ezio.melotti | link | issue25320 messages |
2016-01-01 02:13:20 | ezio.melotti | create | |
|