Fix bug in getFileInChangeEdit

Due to a recent refactor, getFileInChangeEdit's return value object
structure changed to not include res.ok. This change adds that back in.

Bug: Issue 4437
Change-Id: I90d20b166dcd0da22da7fe3653342e43a459f81a
diff --git a/polygerrit-ui/app/elements/shared/gr-rest-api-interface/gr-rest-api-interface.js b/polygerrit-ui/app/elements/shared/gr-rest-api-interface/gr-rest-api-interface.js
index 6c052ae..5118d70 100644
--- a/polygerrit-ui/app/elements/shared/gr-rest-api-interface/gr-rest-api-interface.js
+++ b/polygerrit-ui/app/elements/shared/gr-rest-api-interface/gr-rest-api-interface.js
@@ -1344,7 +1344,7 @@
             // X-FYI-Content-Type header of the response.
             const type = res.headers.get('X-FYI-Content-Type');
             return this.getResponseObject(res).then(content => {
-              return {content, type};
+              return {content, type, ok: true};
             });
           });
     },
diff --git a/polygerrit-ui/app/elements/shared/gr-rest-api-interface/gr-rest-api-interface_test.html b/polygerrit-ui/app/elements/shared/gr-rest-api-interface/gr-rest-api-interface_test.html
index bf238af..6542bb6 100644
--- a/polygerrit-ui/app/elements/shared/gr-rest-api-interface/gr-rest-api-interface_test.html
+++ b/polygerrit-ui/app/elements/shared/gr-rest-api-interface/gr-rest-api-interface_test.html
@@ -1199,7 +1199,8 @@
           .returns(Promise.resolve('new content'));
 
       return element.getFileInChangeEdit('1', 'test/path').then(res => {
-        assert.deepEqual(res, {content: 'new content', type: 'text/java'});
+        assert.deepEqual(res,
+            {content: 'new content', type: 'text/java', ok: true});
       });
     });
   });