Merge 153342 - Print preview: Use an ID instead of memory pointer string in WebUI.

chrome/browser/resources/print_preview/preview_generator.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';
 
   /**
    * Interface to the Chromium print preview generator.
    * @param {!print_preview.DestinationStore} destinationStore Used to get the
@@ skipping 189 matching lines @@
     },
 
     /**
      * Dispatches a PAGE_READY event to signal that a page preview is ready.
      * @param {number} previewIndex Index of the page with respect to the pages
      *     shown in the preview. E.g an index of 0 is the first displayed page,
      *     but not necessarily the first original document page.
      * @param {number} pageNumber Number of the page with respect to the
      *     document. A value of 3 means it's the third page of the original
      *     document.
-     * @param {string} previewUid Unique identifier of the preview.
+     * @param {number} previewUid Unique identifier of the preview.
      * @private
      */
     dispatchPageReadyEvent_: function(previewIndex, pageNumber, previewUid) {
       var pageGenEvent = new cr.Event(PreviewGenerator.EventType.PAGE_READY);
       pageGenEvent.previewIndex = previewIndex;
-      pageGenEvent.previewUrl =
-          'chrome://print/' + previewUid + '/' + (pageNumber - 1) +
-          '/print.pdf';
+      pageGenEvent.previewUrl = 'chrome://print/' + previewUid.toString() +
+          '/' + (pageNumber - 1) + '/print.pdf';
       this.dispatchEvent(pageGenEvent);
     },
 
     /**
      * Dispatches a PREVIEW_START event. Signals that the preview should be
      * reloaded.
-     * @param {string} previewUid Unique identifier of the preview.
+     * @param {number} previewUid Unique identifier of the preview.
      * @param {number} index Index of the first page of the preview.
      * @private
      */
     dispatchPreviewStartEvent_: function(previewUid, index) {
       var previewStartEvent = new cr.Event(
           PreviewGenerator.EventType.PREVIEW_START);
       if (!this.printTicketStore_.isDocumentModifiable) {
         index = -1;
       }
-      previewStartEvent.previewUrl =
-          'chrome://print/' + previewUid + '/' + index + '/print.pdf';
+      previewStartEvent.previewUrl = 'chrome://print/' +
+          previewUid.toString() + '/' + index + '/print.pdf';
       this.dispatchEvent(previewStartEvent);
     },
 
     /**
      * @return {boolean} Whether the print ticket has changed sufficiently to
      *     determine whether a new preview request should be issued.
      * @private
      */
     hasPreviewChanged_: function() {
       var ticketStore = this.printTicketStore_;
@@ skipping 133 matching lines @@
       // current one.
       cr.dispatchSimpleEvent(this, PreviewGenerator.EventType.FAIL);
     }
   };
 
   // Export
   return {
     PreviewGenerator: PreviewGenerator
   };
 });