Use more StringPieces in the PDF plugin.

pdf/out_of_process_instance.h

 // 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.
 
 #ifndef PDF_OUT_OF_PROCESS_INSTANCE_H_
 #define PDF_OUT_OF_PROCESS_INSTANCE_H_
 
 #include <stdint.h>
 #include <string.h>
 
@@ skipping 186 matching lines @@
   // Set new zoom scale.
   void SetZoom(double scale);
 
   // Reduces the document to 1 page and appends |print_preview_page_count_|
   // blank pages to the document for print preview.
   void AppendBlankPrintPreviewPages();
 
   // Process the preview page data information. |src_url| specifies the preview
   // page data location. The |src_url| is in the format:
   // chrome://print/id/page_number/print.pdf
-  // |dst_page_index| specifies the blank page index that needs to be replaced

[Lei Zhang, 2017/05/04 19:19:38]

We might have chosen "dst" because it's the same length as "src" but now I keep
reading it as Daylight Savings Time.

+  // |dest_page_index| specifies the blank page index that needs to be replaced
   // with the new page data.
-  void ProcessPreviewPageInfo(const std::string& src_url, int dst_page_index);
+  void ProcessPreviewPageInfo(const std::string& src_url, int dest_page_index);
   // Load the next available preview page into the blank page.
   void LoadAvailablePreviewPage();
 
   // Bound the given scroll offset to the document.
   pp::FloatPoint BoundScrollOffsetToDocument(
       const pp::FloatPoint& scroll_offset);
 
   pp::ImageData image_data_;
   // Used when the plugin is embedded in a page and we have to create the loader
   // ourself.
@@ skipping 167 matching lines @@
 
   // True if the plugin is loaded in print preview, otherwise false.
   bool is_print_preview_;
 
   DISALLOW_COPY_AND_ASSIGN(OutOfProcessInstance);
 };
 
 }  // namespace chrome_pdf
 
 #endif  // PDF_OUT_OF_PROCESS_INSTANCE_H_