Make setDocumentMetadata_() only take one parameter.

setDocumentMetadata_() has 3 parameters, which all comes from the same
structure MetadataMessageData. This CL makes setDocumentMetadata_() only
takes one MetadataMessageData type parameter instead.

Bug: 177188
Change-Id: I59c53252d91996ac13824c60ae1e7b276feb940e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2386297
Commit-Queue: Hui Yingst <nigi@chromium.org>
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#804059}

chrome/browser/resources/pdf/pdf_viewer.js

@@ -708,9 +708,7 @@ export class PDFViewerElement extends PDFViewerBaseElement {
             destinationData.zoom);
         return;
       case 'metadata':
-        const metadata = /** @type {!MetadataMessageData} */ (data);
-        this.setDocumentMetadata_(
-            metadata.title, metadata.bookmarks, metadata.canSerializeDocument);
+        this.setDocumentMetadata_(/** @type {!MetadataMessageData} */ (data));
         return;
       case 'setIsEditing':
         // Editing mode can only be entered once, and cannot be exited.
@@ -808,16 +806,14 @@ export class PDFViewerElement extends PDFViewerBaseElement {
 
   /**
    * Sets document metadata from the current controller.
-   * @param {string} title
-   * @param {!Array<!Bookmark>} bookmarks
-   * @param {boolean} canSerializeDocument
+   * @param {!MetadataMessageData} metadata
    * @private
    */
-  setDocumentMetadata_(title, bookmarks, canSerializeDocument) {
-    this.title_ = title ? title : getFilenameFromURL(this.originalUrl);
+  setDocumentMetadata_(metadata) {
+    this.title_ = metadata.title || getFilenameFromURL(this.originalUrl);
     document.title = this.title_;
-    this.bookmarks_ = bookmarks;
-    this.canSerializeDocument_ = canSerializeDocument;
+    this.bookmarks_ = metadata.bookmarks;
+    this.canSerializeDocument_ = metadata.canSerializeDocument;
   }
 
   /**