commit c5b212f21991e2013d987ff7bea581cbda0e294b
author Dhruv Srivastava <dhruvsri@google.com> Mon Jun 29 16:03:39 2020 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Mon Jun 29 16:03:39 2020 +0000
tree 89e7e10fb513250b435de02f01e3770699ca5d85
parent 7d4f2f3183ea9f5c6b125707ed123314cd61ee0d
parent 7941ab09c10643fbaff2c0128ee553c9c020169b

Merge "Send latest ps for right-patchset-changed event"

polygerrit-ui/app/elements/diff/gr-patch-range-select/gr-patch-range-select.js

diff --git a/polygerrit-ui/app/elements/diff/gr-patch-range-select/gr-patch-range-select.js b/polygerrit-ui/app/elements/diff/gr-patch-range-select/gr-patch-range-select.js
index 804fa38..806e147 100644
--- a/polygerrit-ui/app/elements/diff/gr-patch-range-select/gr-patch-range-select.js
+++ b/polygerrit-ui/app/elements/diff/gr-patch-range-select/gr-patch-range-select.js
@@ -289,14 +289,18 @@
   _handlePatchChange(e) {
     const detail = {patchNum: this.patchNum, basePatchNum: this.basePatchNum};
     const target = dom(e).localTarget;
-
+    const latestPatchNum = this.computeLatestPatchNum(this.availablePatches);
     if (target === this.$.patchNumDropdown) {
       if (detail.patchNum === e.detail.value) return;
       this.reporting.reportInteraction('right-patchset-changed',
-          {previous: detail.patchNum, current: e.detail.value});
+          {
+            previous: detail.patchNum,
+            current: e.detail.value,
+            latest: latestPatchNum,
+          });
       detail.patchNum = e.detail.value;
     } else {
-      if (detail.basePatchNum === e.detail.value) return;
+      if (this.patchNumEquals(detail.basePatchNum, e.detail.value)) return;
       this.reporting.reportInteraction('left-patchset-changed',
           {previous: detail.basePatchNum, current: e.detail.value});
       detail.basePatchNum = e.detail.value;