Print Preview: Assign to the header summary label only once.

chrome/browser/resources/print_preview/print_header.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 cr.define('print_preview', function() {
   'use strict';
 
   /**
    * Creates a PrintHeader object. This object encapsulates all the elements
    * and logic related to the top part of the left pane in print_preview.html.
@@ skipping 141 matching lines @@
     /**
      * Updates the summary element based on the currently selected user options.
      * @private
      */
     updateSummary_: function() {
       if (!this.printTicketStore_.isTicketValid()) {
         this.getChildElement('.summary').innerHTML = '';
         return;
       }
 
-      var summaryLabel =
-          loadTimeData.getString('printPreviewSheetsLabelSingular');
-      var pagesLabel = loadTimeData.getString('printPreviewPageLabelPlural');
-
       var saveToPdfOrDrive = this.destinationStore_.selectedDestination &&
           (this.destinationStore_.selectedDestination.id ==
               print_preview.Destination.GooglePromotedId.SAVE_AS_PDF ||
            this.destinationStore_.selectedDestination.id ==
               print_preview.Destination.GooglePromotedId.DOCS);
-      if (saveToPdfOrDrive) {
-        summaryLabel = loadTimeData.getString('printPreviewPageLabelSingular');
-      }
 
       var numPages = this.printTicketStore_.pageRange.getPageNumberSet().size;
       var numSheets = numPages;
       if (!saveToPdfOrDrive && this.printTicketStore_.duplex.getValue()) {
         numSheets = Math.ceil(numPages / 2);
       }
 
       var copies = this.printTicketStore_.copies.getValueAsNumber();
       numSheets *= copies;
       numPages *= copies;
 
+      var pagesLabel = loadTimeData.getString('printPreviewPageLabelPlural');
+      var summaryLabel;
       if (numSheets > 1) {
         summaryLabel = saveToPdfOrDrive ? pagesLabel :
             loadTimeData.getString('printPreviewSheetsLabelPlural');
+      } else {
+        summaryLabel = saveToPdfOrDrive ?

[dpapad, 2017/04/24 17:23:40]

Nit: Can be compacted even further as follows:

summaryLabel = loadTimeData.getString(
    saveToPdfOrDrive ?
    printPreviewPageLabelSingular : printPreviewSheetsLabelSingular);

[Lei Zhang, 2017/04/24 23:12:17]

Done.

+            loadTimeData.getString('printPreviewPageLabelSingular') :
+            loadTimeData.getString('printPreviewSheetsLabelSingular');
       }
 
       var html;
       var label;
       if (numPages != numSheets) {
         html = loadTimeData.getStringF(
             'printPreviewSummaryFormatLong',
             '<b>' + numSheets.toLocaleString() + '</b>',
             '<b>' + summaryLabel + '</b>',
             numPages.toLocaleString(),
@@ skipping 79 matching lines @@
         this.getChildElement('button.print').focus();
       }
     }
   };
 
   // Export
   return {
     PrintHeader: PrintHeader
   };
 });