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 | | |