page.close() and browser.close() should be in try-catch-finally block

[ropel]

Everytime something goes wrong capturing a screenshot (i.e. no response, timeouts, ...) the function exits without closing page and browser, so that chrome instances keep growing until server dies. Putting page.close() and browser.close() in a "finally" block should avoid resources leaking

[sindresorhus]

I agree. Pull request welcome.

[ropel]

Ok, pull request done

[jopemachine]

Ok, pull request done

Is this issue resolved?

[AlejandroAkbal]

Was this ever merged? Because I'm having zombie processes and I imagine it's because of this