Merge branch 'jdb/stop-mr-options-polling-onblur' into 'master'

Add check for window focus before polling

See merge request gitlab-org/gitlab!26626

app/assets/javascripts/vue_merge_request_widget/mr_widget_options.vue

@@ -212,6 +212,8 @@ export default {
       return new MRWidgetService(this.getServiceEndpoints(store));
     },
     checkStatus(cb, isRebased) {
+      if (document.visibilityState !== 'visible') return Promise.resolve();
+
       return this.service
         .checkStatus()
         .then(({ data }) => {

spec/frontend/vue_mr_widget/mr_widget_options_spec.js

@@ -259,16 +259,40 @@ describe('mrWidgetOptions', () => {
 
     describe('methods', () => {
       describe('checkStatus', () => {
-        it('should tell service to check status', () => {
+        let cb;
+        let isCbExecuted;
+
+        beforeEach(() => {
           jest.spyOn(vm.service, 'checkStatus').mockReturnValue(returnPromise(mockData));
           jest.spyOn(vm.mr, 'setData').mockImplementation(() => {});
           jest.spyOn(vm, 'handleNotification').mockImplementation(() => {});
 
-          let isCbExecuted = false;
-          const cb = () => {
+          isCbExecuted = false;
+          cb = () => {
             isCbExecuted = true;
           };
+        });
+
+        it('should not tell service to check status if document is not visible', () => {
+          Object.defineProperty(document, 'visibilityState', {
+            value: 'hidden',
+            configurable: true,
+          });
+          vm.checkStatus(cb);
+
+          return vm.$nextTick().then(() => {
+            expect(vm.service.checkStatus).not.toHaveBeenCalled();
+            expect(vm.mr.setData).not.toHaveBeenCalled();
+            expect(vm.handleNotification).not.toHaveBeenCalled();
+            expect(isCbExecuted).toBeFalsy();
+            Object.defineProperty(document, 'visibilityState', {
+              value: 'visible',
+              configurable: true,
+            });
+          });
+        });
 
+        it('should tell service to check status if document is visible', () => {
           vm.checkStatus(cb);
 
           return vm.$nextTick().then(() => {