Pass full PatchSet into ApprovalCopier

This is called from contexts where the new PatchSet is created and
inserted into the database, but transaction semantics in non-SQL
implementations do not guarantee it is able to be read back. We still
need it to get the commit SHA for change kind computation, so pass it
in.

Change-Id: I048453a39c405e5b4c185d3f1629fee4c9acae14
Fix render logic for RelatedChangesTab.NavigationList

+ Re-render the NavigationList whenever the visible rows change.
  This fixes a bug where calling setMaxHeight() after the NavigationList
  has been rendered, does not actually add rows to fill the new visible
  range. Also, guard against executing the render logic until the
  rowHeight has been measured.

+ Correctly remember the selected row across iterations of
  DisplayCommand.

Change-Id: Ifb17f1c2199eac67e166b862833ce7221fea6732
Merge "Fix render logic for RelatedChangesTab.NavigationList"