fix: deprecate setLayoutZoomLevelLimits

[nornagon]

Description of Change

This hasn't worked since Chromium removed it and I believe it's beyond our capacity to continue to support it. The function doesn't work, but instead of throwing an error it seems nicer to warn about it in 8.x and wait till 9.x to delete it completely.

Checklist

• PR description included and stakeholders cc'd
• npm test passes
• tests are changed or added
• relevant documentation is changed or added
• PR title follows semantic commit guidelines
• PR release notes describe the change in a way relevant to app developers, and are capitalized, punctuated, and past tense.

Release Notes

Notes: setLayoutZoomLevelLimits no longer works in Electron 8, so emit a warning message when using it. The function will be removed in Electron 9.

[deepak1556]

This was added in #8041 , the issue it mentioned was fixed in another way #8537, can you please verify if there is a regression before merging this.

[nornagon]

@deepak1556 do you mean this issue? #6958

[deepak1556]

Yup that one, users seem to have been using this api as a workaround to persist the zoom level across navigations but we later fixed it on electron core, just want to make sure its still fixed, so that we can remove this api safely.

[nornagon]

https://gist.github.com/8ace446c40111aca67391104c2a9cc76 I tested this in a Fiddle and it seems to work OK. However, I'm not totally sure what the original failing behavior was so I can't say for certain that I didn't accidentally dodge it somehow. Wish there had been a test written :(

[release-clerk]

Release Notes Persisted

setLayoutZoomLevelLimits no longer works in Electron 8, so emit a warning message when using it. The function will be removed in Electron 9.

[nornagon]

/trop run backport-to 8-x-y

[trop]

The backport process for this PR has been manually initiated -
sending your commits to "8-x-y"!

[trop]

I have automatically backported this PR to "8-x-y", please check out #21360