Merge pull request #12796 from Snuffleupagus/BaseTreeViewer._finishRendering

Extract common functionality into a new `BaseTreeViewer._finishRendering` method

web/base_tree_viewer.js

@@ -103,6 +103,20 @@ class BaseTreeViewer {
     this._toggleTreeItem(this.container, !this._lastToggleIsShow);
   }
 
+  /**
+   * @private
+   */
+  _finishRendering(fragment, count, hasAnyNesting = false) {
+    if (hasAnyNesting) {
+      this.container.classList.add("treeWithDeepNesting");
+
+      this._lastToggleIsShow = !fragment.querySelector(".treeItemsHidden");
+    }
+    this.container.appendChild(fragment);
+
+    this._dispatchEvent(count);
+  }
+
   render(params) {
     throw new Error("Not implemented: render");
   }

web/pdf_attachment_viewer.js

@@ -187,9 +187,7 @@ class PDFAttachmentViewer extends BaseTreeViewer {
       attachmentsCount++;
     }
 
-    this.container.appendChild(fragment);
+    this._finishRendering(fragment, attachmentsCount);
-
-    this._dispatchEvent(attachmentsCount);
   }
 
   /**

web/pdf_layer_viewer.js

@@ -174,16 +174,8 @@ class PDFLayerViewer extends BaseTreeViewer {
         levelData.parent.appendChild(div);
       }
     }
-    if (hasAnyNesting) {
-      this.container.classList.add("treeWithDeepNesting");
 
-      this._lastToggleIsShow =
+    this._finishRendering(fragment, layersCount, hasAnyNesting);
-        fragment.querySelectorAll(".treeItemsHidden").length === 0;
-    }
-
-    this.container.appendChild(fragment);
-
-    this._dispatchEvent(layersCount);
   }
 
   /**

web/pdf_outline_viewer.js

@@ -171,16 +171,8 @@ class PDFOutlineViewer extends BaseTreeViewer {
         outlineCount++;
       }
     }
-    if (hasAnyNesting) {
-      this.container.classList.add("treeWithDeepNesting");
 
-      this._lastToggleIsShow =
+    this._finishRendering(fragment, outlineCount, hasAnyNesting);
-        fragment.querySelectorAll(".treeItemsHidden").length === 0;
-    }
-
-    this.container.appendChild(fragment);
-
-    this._dispatchEvent(outlineCount);
   }
 }