Crash in AllowCertificateError

[csetera]

• I have read the Contributing Guidelines for this project.
• I agree to follow the Code of Conduct that this project adheres to.
• I have searched the issue tracker for an issue that matches the one I want to file, without success.

Issue Details

• Electron Version:
  • v8.0.0
• Operating System:
  • macOS 10.15.3
  • Ubuntu Linux 18.04
• Last Known Working Electron version:
  • 5.x

Expected Behavior

Should not crash

Actual Behavior

Crashed when navigating. This happens in my app when a BrowserView is opened to iheartradio.com

To Reproduce

I don't have a simple public example at this time. Can try to put something together.

Following information was captured by Crash Reporter to Backtrace.

CrashReporterInfo.zip

[csetera]

A bit more information. If I create the following:

        app.on('certificate-error', (event, webContents, url, error, certificate, callback) => {
            this.logger.debug('Certificate Error for URL: %s error: %s', url, error);
            if (error === 'net::ERR_CERT_WEAK_SIGNATURE_ALGORITHM') {
                event.preventDefault()
                callback(true)
            } else {
                callback(false);
            }
          });

I will see that the error is:
Certificate Error for URL: wss://quickio.iheart.com/ error: net::ERR_CERT_WEAK_SIGNATURE_ALGORITHM

I can cause the crash with a registered handler without any handling logic:

        app.on('certificate-error', (event, webContents, url, error, certificate, callback) => {
            this.logger.debug('Certificate Error for URL: %s error: %s', url, error);
            // if (error === 'net::ERR_CERT_WEAK_SIGNATURE_ALGORITHM') {
            //     event.preventDefault()
            //     callback(true)
            // } else {
            //     callback(false);
            // }
          });

The same error will be logged, however the app will crash. It appears that this crash is caused by unhandled certificate errors.

[csetera]

This is resolved by #22124