diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/ui/NavigationTable.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/ui/NavigationTable.java index fb394ebf7a..ce417bb6db 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/ui/NavigationTable.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/ui/NavigationTable.java @@ -189,17 +189,14 @@ public abstract class NavigationTable extends FancyFlexTable { } protected void movePointerTo(final int newRow, final boolean scroll) { - if (getRowItem(newRow) == null) { - return; - } - final CellFormatter fmt = table.getCellFormatter(); final boolean clear = 0 <= currentRow && currentRow < table.getRowCount(); if (clear) { final Element tr = DOM.getParent(fmt.getElement(currentRow, C_ARROW)); UIObject.setStyleName(tr, Gerrit.RESOURCES.css().activeRow(), false); } - if (newRow >= 0) { + if (0 <= newRow && newRow < table.getRowCount() + && getRowItem(newRow) != null) { table.setWidget(newRow, C_ARROW, pointer); final Element tr = DOM.getParent(fmt.getElement(newRow, C_ARROW)); UIObject.setStyleName(tr, Gerrit.RESOURCES.css().activeRow(), true);