<!--  As an open source project with a dedicated but small maintainer team, it can sometimes take a long time for issues to be addressed so please be patient and we will get back to you as soon as we can.
-->

### Preflight Checklist
<!-- Please ensure you've completed the following steps by replacing [ ] with [x]-->

* [x] I have read the [Contributing Guidelines](https://github.com/electron/electron/blob/master/CONTRIBUTING.md) for this project.
* [x] I agree to follow the [Code of Conduct](https://github.com/electron/electron/blob/master/CODE_OF_CONDUCT.md) that this project adheres to.
* [x] I have searched the issue tracker for an issue that matches the one I want to file, without success.

### Issue Details

* **Electron Version:**
5.0.0 to 8.0.0 beta7
* **Operating System:**
Windows 10

### Expected Behavior
The did-fail-load event should be fired.

### Actual Behavior
The did-fail-load event is not fired.

### To Reproduce
Add an event handler to the did-fail-load event.
Add an event handler to the certificate-error event and cancel the default handling. Then call the callback with false.
The did-fail-load event will not be fired.

```
mainWindow.webContents.on("certificate-error", (event, url, error, certificate, callback) => {
      event.preventDefault();
      callback(false)
});

mainWindow.webContents.on("did-fail-load", (event, url, error, certificate, callback) => {
      console.log("did-fail-load");
});
```

Electron fiddle gist:
https://gist.github.com/c1427e38bf0af9a5e78088fcea6e3c66

### Additional Information
loadURL returns a Promise, in the certificate error case this Promise will be rejected. The gist will show this too.

```
mainWindow.loadURL('https://expired.badssl.com')
  .then(() => console.log("success"))
  .catch(e => console.log("fail"))
```

shell/browser/api/electron_api_app.cc
WebContents* api_web_contents = WebContents::From(web_contents);
    
    if (api_web_contents) {
      api_web_contents->OnCreateWindow(target_url, referrer, frame_name,
                                       disposition, raw_features, body);
    }
  }
  return false;
}
void App::AllowCertificateError(
    content::WebContents* web_contents,
    int cert_error,
    const net::SSLInfo& ssl_info,
    const GURL& request_url,
    bool is_main_frame_request,
    bool strict_enforcement,
    base::OnceCallback<void(content::CertificateRequestResultType)> callback) {
  auto adapted_callback = base::AdaptCallbackForRepeating(std::move(callback));
  v8::Isolate* isolate = JavascriptEnvironment::GetIsolate();
  v8::Locker locker(isolate);
  v8::HandleScope handle_scope(isolate);
  bool prevent_default =
      Emit("certificate-error",
           WebContents::FromOrCreate(isolate, web_contents), request_url,
           net::ErrorToString(cert_error), ssl_info.cert, adapted_callback);
  
  if (!prevent_default)
    adapted_callback.Run(content::CERTIFICATE_REQUEST_RESULT_TYPE_DENY);
}
base::OnceClosure App::SelectClientCertificate(