Exclude ancestor FileDiffs from the diffviewer
Review Request #10124 — Created Aug. 22, 2018 and submitted — Latest diff uploaded
When a review request is created with commit history, it is likely that
there will beFileDiff
s in multiple commits that modify the same file.
For anyFileDiff
in a commit, allFileDiff
s in previous commits are
known as "ancestor"FileDiff
s.These ancestor
FileDiff
s are now excluded from the diffviewer. This
removes the duplication of seeing the same file listed multiple times
with incremental changes. However, currently only the latest change for
that file is shown. True cumulative diffs are supported in /r/10130/.
- Ran unit tests.
- Viewed a review request with commit history and did not see repeated
files.