Merge pull request #8138 from Snuffleupagus/bug-1345253

Get rid of `element.removeChild(element.firstChild)` usage (bug 1345253)

web/pdf_attachment_viewer.js

@@ -61,10 +61,8 @@ var PDFAttachmentViewer = (function PDFAttachmentViewerClosure() {
     reset: function PDFAttachmentViewer_reset(keepRenderedCapability) {
       this.attachments = null;
 
-      var container = this.container;
+      // Remove the attachments from the DOM.
-      while (container.firstChild) {
+      this.container.textContent = '';
-        container.removeChild(container.firstChild);
-      }
 
       if (!keepRenderedCapability) {
         // NOTE: The *only* situation in which the `_renderedCapability` should

web/pdf_outline_viewer.js

@@ -63,10 +63,11 @@ var PDFOutlineViewer = (function PDFOutlineViewerClosure() {
       this.outline = null;
       this.lastToggleIsShow = true;
 
-      var container = this.container;
+      // Remove the outline from the DOM.
-      while (container.firstChild) {
+      this.container.textContent = '';
-        container.removeChild(container.firstChild);
+      // Ensure that the left (right in RTL locales) margin is always reset,
-      }
+      // to prevent incorrect outline alignment if a new document is opened.
+      this.container.classList.remove('outlineWithDeepNesting');
     },
 
     /**