Make the passwordCapability field, in WorkerTransport, actually private as intended

This commit is contained in:
Jonas Jenwald 2023-07-30 11:45:35 +02:00
parent 81dfa61777
commit 930cbc4d27

View File

@ -2367,6 +2367,8 @@ class WorkerTransport {
#pagePromises = new Map();
#passwordCapability = null;
constructor(messageHandler, loadingTask, networkStream, params, factory) {
this.messageHandler = messageHandler;
this.loadingTask = loadingTask;
@ -2384,7 +2386,6 @@ class WorkerTransport {
this.destroyed = false;
this.destroyCapability = null;
this._passwordCapability = null;
this._networkStream = networkStream;
this._fullReader = null;
@ -2495,11 +2496,9 @@ class WorkerTransport {
this.destroyed = true;
this.destroyCapability = new PromiseCapability();
if (this._passwordCapability) {
this._passwordCapability.reject(
new Error("Worker was destroyed during onPassword callback")
);
}
this.#passwordCapability?.reject(
new Error("Worker was destroyed during onPassword callback")
);
const waitOn = [];
// We need to wait for all renderings to be completed, e.g.
@ -2702,27 +2701,27 @@ class WorkerTransport {
});
messageHandler.on("PasswordRequest", exception => {
this._passwordCapability = new PromiseCapability();
this.#passwordCapability = new PromiseCapability();
if (loadingTask.onPassword) {
const updatePassword = password => {
if (password instanceof Error) {
this._passwordCapability.reject(password);
this.#passwordCapability.reject(password);
} else {
this._passwordCapability.resolve({ password });
this.#passwordCapability.resolve({ password });
}
};
try {
loadingTask.onPassword(updatePassword, exception.code);
} catch (ex) {
this._passwordCapability.reject(ex);
this.#passwordCapability.reject(ex);
}
} else {
this._passwordCapability.reject(
this.#passwordCapability.reject(
new PasswordException(exception.message, exception.code)
);
}
return this._passwordCapability.promise;
return this.#passwordCapability.promise;
});
messageHandler.on("DataLoaded", data => {