commit | facf9d55ec82b8cbec63203d2e244fe65f858a72 | [log] [tgz] |
---|---|---|
author | Dave Borowitz <dborowitz@google.com> | Fri May 30 12:31:26 2014 -0700 |
committer | Dave Borowitz <dborowitz@google.com> | Sat May 31 14:37:38 2014 -0700 |
tree | da6d0e9226f1e99b7e0ad3d76d2aa520c5f4f7ae | |
parent | b8dd435dec4f73342ee6c5709e5c785bbdaa7cc6 [diff] |
Populate CodeReviewCommit before passing to MergeValidators The onPreMerge method of MergeValidationListener may reasonably assume that the fields of CodeReviewCommit are populated, for example that commit.notes() will not NPE. The previous ordering of code in MergeOp.validateChangeList() made this assumption false. Change-Id: I9bd9e76bb09a25faf57df7080cdb0c8f878a7471