remove unused fn and variable (#20111) (#20328)

elastic · Jun 29, 2018 · 97cf9c4 · 97cf9c4
1 parent f585621
commit 97cf9c4
Showing 1 changed file with 3 additions and 10 deletions.
diff --git a/x-pack/plugins/reporting/server/browsers/chromium/driver/index.js b/x-pack/plugins/reporting/server/browsers/chromium/driver/index.js
@@ -21,11 +21,6 @@ export class HeadlessChromiumDriver {
     this._logger = logger;
   }
 
-  async destroy() {
-    this.killed = true;
-    await this._client.close();
-  }
-
   async evaluate({ fn, args = [], awaitPromise = false, returnByValue = false }) {
     const { Runtime } = this._client;
 
@@ -63,9 +58,7 @@ export class HeadlessChromiumDriver {
 
     Page.screencastFrame(async ({ data, sessionId }) => {
       await this._writeData(path.join(recordPath, `${moment().utc().format('HH_mm_ss_SSS')}.png`), data);
-      if (!this.killed) {
-        await Page.screencastFrameAck({ sessionId });
-      }
+      await Page.screencastFrameAck({ sessionId });
     });
   }
 
@@ -123,13 +116,13 @@ export class HeadlessChromiumDriver {
   }
 
   async waitFor({ fn, args, toEqual }) {
-    while (!this.killed && (await this.evaluate({ fn, args })) !== toEqual) {
+    while ((await this.evaluate({ fn, args })) !== toEqual) {
       await delay(this._waitForDelayMs);
     }
   }
 
   async waitForSelector(selector) {
-    while (!this.killed) {
+    while (true) {
       const { nodeId } = await this._client.DOM.querySelector({ nodeId: this.documentNode.root.nodeId, selector });
       if (nodeId) {
         break;