Merge "Reload actions when new change is loaded, not on new changeNum"

polygerrit-ui/app/elements/change/gr-change-actions/gr-change-actions.js

diff --git a/polygerrit-ui/app/elements/change/gr-change-actions/gr-change-actions.js b/polygerrit-ui/app/elements/change/gr-change-actions/gr-change-actions.js
index 299612e..9b0e458 100644
--- a/polygerrit-ui/app/elements/change/gr-change-actions/gr-change-actions.js
+++ b/polygerrit-ui/app/elements/change/gr-change-actions/gr-change-actions.js
@@ -323,7 +323,7 @@
     observers: [
       '_actionsChanged(actions.*, revisionActions.*, _additionalActions.*, ' +
           'editLoaded, editBasedOnCurrentPatchSet, change)',
-      '_changeOrPatchNumChanged(changeNum, patchNum)',
+      '_changeChanged(change)',
     ],
 
     listeners: {
@@ -354,7 +354,7 @@
       });
     },
 
-    _changeOrPatchNumChanged() {
+    _changeChanged() {
       this.reload();
     },
 

polygerrit-ui/app/elements/change/gr-change-actions/gr-change-actions_test.html

diff --git a/polygerrit-ui/app/elements/change/gr-change-actions/gr-change-actions_test.html b/polygerrit-ui/app/elements/change/gr-change-actions/gr-change-actions_test.html
index a8587c0..31ec02d 100644
--- a/polygerrit-ui/app/elements/change/gr-change-actions/gr-change-actions_test.html
+++ b/polygerrit-ui/app/elements/change/gr-change-actions/gr-change-actions_test.html
@@ -1150,6 +1150,14 @@
       assert.isTrue(handler.called);
     });
 
+    test('changing changeNum or patchNum does not reload', () => {
+      const reloadStub = sandbox.stub(element, 'reload');
+      element.changeNum = 123;
+      assert.isFalse(reloadStub.called);
+      element.patchNum = 456;
+      assert.isFalse(reloadStub.called);
+    });
+
     suite('setActionOverflow', () => {
       test('move action from overflow', () => {
         assert.isNotOk(element.$$('[data-action-key="cherrypick"]'));