• 
      

    Exclude ancestor FileDiffs from the diffviewer

    Review Request #10124 — Created Aug. 22, 2018 and submitted — Latest diff uploaded

    Information

    Review Board
    release-4.0.x
    423f1c4...

    Reviewers

    When a review request is created with commit history, it is likely that
    there will be FileDiffs in multiple commits that modify the same file.
    For any FileDiff in a commit, all FileDiffs in previous commits are
    known as "ancestor" FileDiffs.

    These ancestor FileDiffs 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.