beforeunload dialog is not triggered

[khongten001]

As the title, would you please add a option to add {runBeforeUnload: true} when page.close? See: https://stackoverflow.com/questions/50832325/onbeforeunload-not-being-triggered-from-a-headless-browser

Thank you very much

[Ayc0]

Related to this, can we make so that page.close runs with page.close({ runBeforeUnload: true })?

This could be configured via an env variable.

If this is something that you're thing could be a valuable addition to jest-puppeteer, I'd be willing to add that to jest-environment-puppeteer

[UziTech]

@Ayc0 ya if you could make a PR with that change I could get it merges. Make sure to add tests 😁👍

[Ayc0]

@UziTech, I opened #504

Does it sound good to you? (It's my first PR on this repo so I'm not sure of all the best practices)

[UziTech]

Thanks for the PR. I will try to get a new release out in the next week.