feat(core): add config to skip waiting for browser close (#120)

Summary:
Issue: #95

Pull Request resolved: #120

Test Plan:
```
memlab build-core && memlab test && arc lint
```

Reviewed By: twobassdrum

Differential Revision: D57819075

Pulled By: JacksonGL

fbshipit-source-id: e60d1aa9d8b59565534a660bb029ceb68499b3d6

packages/core/src/lib/Config.ts

@@ -240,6 +240,7 @@ export class MemLabConfig {
   noReCluster: boolean;
   maxSamplesForClustering: number;
   filterTraceByName: Nullable<string>;
+  skipBrowserCloseWait: boolean;
 
   constructor(options: ConfigOption = {}) {
     // init properties, they can be configured manually
@@ -370,6 +371,8 @@ export class MemLabConfig {
     // if specified via CLI options, this will filter leak traces by
     // node and edge names in the leak trace
     this.filterTraceByName = null;
+    // if true, memlab will not wait for the browser to close successfully
+    this.skipBrowserCloseWait = false;
   }
 
   // initialize configurable parameters

packages/core/src/lib/Utils.ts

@@ -1325,6 +1325,8 @@ async function closePuppeteer(
   if (config.isLocalPuppeteer && !options.warmup) {
     await Promise.all(pages.map(page => page.close()));
     await browser.disconnect();
+  } else if (config.skipBrowserCloseWait) {
+    browser.close();
   } else {
     await browser.close();
   }