Merge pull request #16875 from Snuffleupagus/more-optional-chaining-4

Introduce more optional chaining in the code-base
mozilla · Aug 27, 2023 · e142bae · e142bae
2 parents 598421b + ec3d2be
commit e142bae
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 11 deletions.
diff --git a/src/core/worker_stream.js b/src/core/worker_stream.js
@@ -40,9 +40,8 @@ class PDFWorkerStream {
   }
 
   cancelAllRequests(reason) {
-    if (this._fullRequestReader) {
-      this._fullRequestReader.cancel(reason);
-    }
+    this._fullRequestReader?.cancel(reason);
+
     for (const reader of this._rangeRequestReaders.slice(0)) {
       reader.cancel(reason);
     }

diff --git a/src/display/api.js b/src/display/api.js
@@ -3153,11 +3153,7 @@ class PDFObjects {
    * @returns {Object}
    */
   #ensureObj(objId) {
-    const obj = this.#objs[objId];
-    if (obj) {
-      return obj;
-    }
-    return (this.#objs[objId] = {
+    return (this.#objs[objId] ||= {
       capability: new PromiseCapability(),
       data: null,
     });

diff --git a/src/pdf.sandbox.js b/src/pdf.sandbox.js
@@ -107,9 +107,7 @@ class Sandbox {
   }
 
   dumpMemoryUse() {
-    if (this._module) {
-      this._module.ccall("dumpMemoryUse", null, []);
-    }
+    this._module?.ccall("dumpMemoryUse", null, []);
   }
 
   nukeSandbox() {