Merge changes I7ed36dd1,Idf9b07b7 * changes: Rewrite router tests for GROUP_LIST in a more robust way Let gr-router use a local instance of page.js instead of a global one
diff --git a/java/com/google/gerrit/server/comment/CommentContextLoader.java b/java/com/google/gerrit/server/comment/CommentContextLoader.java index 8fbb259..0df7729 100644 --- a/java/com/google/gerrit/server/comment/CommentContextLoader.java +++ b/java/com/google/gerrit/server/comment/CommentContextLoader.java
@@ -225,6 +225,11 @@ private static Optional<Range> getStartAndEndLines(ContextInput comment) { if (comment.range() != null) { + if (comment.range().endLine < comment.range().startLine) { + // Seems like comments, created in reply to robot comments sometimes have invalid ranges + // Fix here, otherwise the range is invalid and we throw an error later on. + return Optional.of(Range.create(comment.range().startLine, comment.range().startLine + 1)); + } return Optional.of(Range.create(comment.range().startLine, comment.range().endLine + 1)); } else if (comment.lineNumber() > 0) { return Optional.of(Range.create(comment.lineNumber(), comment.lineNumber() + 1));