commit 89fe0501e14496971460c59868c3a271bc971efe
author Becky Siegel <beckysiegel@google.com> Tue Sep 12 16:54:19 2017 -0700
committer Becky Siegel <beckysiegel@google.com> Thu Sep 14 13:47:38 2017 -0700
tree 2e99e24deffaf55f009a7bab5434089b2254ada1
parent 9797e3850b556669183d0710a3dea28d9be98b1a

Utilize gr-patch-range-select in the change view

Slight refactoring required in order to satisfy both use cases:
- Fire an event when patch range changes and let parent element
  handle it.
- Support comment strings
- availablePatches becomes an array of objects instead of an
  array of integers.

Change-Id: Ia8da9296f41eb2d45c9358d03fbec3940273ad9d

polygerrit-ui/app/elements/diff/gr-diff-view/gr-diff-view.js

diff --git a/polygerrit-ui/app/elements/diff/gr-diff-view/gr-diff-view.js b/polygerrit-ui/app/elements/diff/gr-diff-view/gr-diff-view.js
index 56be27d..30f1d58 100644
--- a/polygerrit-ui/app/elements/diff/gr-diff-view/gr-diff-view.js
+++ b/polygerrit-ui/app/elements/diff/gr-diff-view/gr-diff-view.js
@@ -580,7 +580,9 @@
     },
 
     _computeAvailablePatches(revs) {
-      return this.sortRevisions(Object.values(revs)).map(e => e._number);
+      return this.sortRevisions(Object.values(revs)).map(e => {
+        return {num: e._number};
+      });
     },
 
     /**
@@ -672,6 +674,13 @@
       this.$.dropdown.open();
     },
 
+    _handlePatchChange(e) {
+      const rightPatch = e.detail.rightPatch;
+      const leftPatch = e.detail.leftPatch;
+      Gerrit.Nav.navigateToDiff(
+          this._change, this._path, rightPatch, leftPatch);
+    },
+
     _handlePrefsTap(e) {
       e.preventDefault();
       this.$.diffPreferences.open();