tree f4e7ba55408e67859e186eec53b4623f9af22118
parent b334bae2355b61e1c745914692868d14877a9aa7
author Youssef Elghareeb <ghareeb@google.com> 1613641846 +0100
committer Youssef Elghareeb <ghareeb@google.com> 1613642209 +0100

Return empty context if the comment range is outside file boundaries

This is replacing the current behaviour of throwing an exception. See
attached bug where a file in the UI contained an extra line at the end
of file. This is due to our custom handling of newlines in the diff
endpoint. This change mitigates this issue until I come up with a better
solution.

We don't need to increase the cache version since no entries should be
invalidated. Previously failing cache loads will now succeed and should
be cached with this fix.

Bug: Issue 14102
Change-Id: I076437df2aae1495ab2c6f9d44fbd36c60147b77