Initial support for cloudprint in print preview

chrome/browser/resources/print_preview.js

 // Copyright (c) 2011 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.
 
+// require: cr/ui/print_preview_cloud.js
+
 var localStrings = new LocalStrings();
 
+var useCloudPrint = false;
+var maxCloudPrinters = 10;
+
 // The total page count of the previewed document regardless of which pages the
 // user has selected.
 var totalPageCount = -1;
 
 // The previously selected pages by the user. It is used in
 // onPageSelectionMayHaveChanged() to make sure that a new preview is not
 // requested more often than necessary.
 var previouslySelectedPages = [];
 
 // Timer id of the page range textfield. It is used to reset the timer whenever
 // needed.
 var timerId;
 
 // Store the last selected printer index.
 var lastSelectedPrinterIndex = 0;
 
 // Used to disable some printing options when the preview is not modifiable.
 var previewModifiable = false;
 
 // Destination list special value constants.
+const ADD_PRINTER = 'addPrinter';
+const MANAGE_CLOUD_PRINTERS = 'manageCloudPrinters';
+const MANAGE_LOCAL_PRINTERS = 'manageLocalPrinters';
+const MORE_PRINTERS = 'morePrinters';
+const SIGN_IN = 'signIn';
 const PRINT_TO_PDF = 'Print To PDF';
-const MANAGE_PRINTERS = 'Manage Printers';
 
 // State of the print preview settings.
 var printSettings = new PrintSettings();
 
 // The name of the default or last used printer.
 var defaultOrLastUsedPrinterName = '';
 
 /**
  * Window onload handler, sets up the page and starts print preview by getting
  * the printer list.
@@ skipping 111 matching lines @@
 /**
  * Gets the selected printer capabilities and updates the controls accordingly.
  */
 function updateControlsWithSelectedPrinterCapabilities() {
   var printerList = $('printer-list');
   var selectedIndex = printerList.selectedIndex;
   if (selectedIndex < 0)
     return;
 
   var selectedValue = printerList.options[selectedIndex].value;
-  if (selectedValue == PRINT_TO_PDF) {
+  if (printerList.options[selectedIndex].isCloudPrint) {
+    updateWithCloudPrinterCapabilities();
+    return;

[sanjeevr, 2011/06/10 18:13:48]

Nit: Is there a way to restructure this so we don't have returns inside the if
blocks? Makes for more readable code, methinks.

[Albert Bodenhamer, 2011/06/10 23:13:45]

Done.

+  } else if (selectedValue == SIGN_IN ||
+             selectedValue == MANAGE_CLOUD_PRINTERS ||
+             selectedValue == MANAGE_LOCAL_PRINTERS) {
+    printerList.selectedIndex = lastSelectedPrinterIndex;
+    chrome.send(selectedValue);
+    return;
+  } else if (selectedValue == PRINT_TO_PDF) {
     updateWithPrinterCapabilities({'disableColorOption': true,
                                    'setColorAsDefault': true,
-                                   'disableCopiesOption': true});
-  } else if (selectedValue == MANAGE_PRINTERS) {
-    printerList.selectedIndex = lastSelectedPrinterIndex;
-    chrome.send('managePrinters');
-    return;
+                                   'disableCopiesOption': true,
+                                   'disableLandscapeOption': false});
   } else {
     // This message will call back to 'updateWithPrinterCapabilities'
     // function.
     chrome.send('getPrinterCapabilities', [selectedValue]);
   }
 
   lastSelectedPrinterIndex = selectedIndex;
 
   // Regenerate the preview data based on selected printer settings.
   setDefaultValuesAndRegeneratePreview();
 }
 
+function updateWithCloudPrinterCapabilities() {
+  var printerList = $('printer-list');
+  var selectedIndex = printerList.selectedIndex;
+  if (printerList.options[selectedIndex].capabilities) {
+    doUpdateCloudPrinterCapabilities(
+        printerList.options[selectedIndex],
+        printerList.options[selectedIndex].capabilities);
+  } else {
+    cloudprint.updatePrinterCaps(printerList.options[selectedIndex],
+                                 doUpdateCloudPrinterCapabilities);
+  }
+}
+
+function doUpdateCloudPrinterCapabilities(printer) {
+  var settings = {'disableColorOption': !cloudprint.supportsColor(printer),
+                  'setColorAsDefault': cloudprint.colorIsDefault(printer),
+                  'disableCopiesOption': true,
+                  'disableLandscapeOption': true};
+  updateWithPrinterCapabilities(settings);
+  var printerList = $('printer-list');
+  var selectedIndex = printerList.selectedIndex;
+  lastSelectedPrinterIndex = selectedIndex;
+
+  // Regenerate the preview data based on selected printer settings.
+  setDefaultValuesAndRegeneratePreview();
+}
+
 /**
  * Updates the controls with printer capabilities information.
  * @param {Object} settingInfo printer setting information.
  */
 function updateWithPrinterCapabilities(settingInfo) {
   var disableColorOption = settingInfo.disableColorOption;
   var disableCopiesOption = settingInfo.disableCopiesOption;
   var setColorAsDefault = settingInfo.setColorAsDefault;
+  var disableLandscapeOption = settingInfo.disableLandscapeOption;
   var colorOption = $('color');
   var bwOption = $('bw');
 
   if (disableCopiesOption) {
     fadeOutElement($('copies-option'));
     $('hr-before-copies').classList.remove('invisible');
   } else {
     fadeInElement($('copies-option'));
     $('hr-before-copies').classList.add('invisible');
   }
 
+  if (disableLandscapeOption) {
+    fadeOutElement($('landscape-option'));
+  } else {
+    fadeInElement($('landscape-option'));

[dpapad, 2011/06/10 20:29:07]

nit: Could you use the more compressed form?

disableLandscapeOption ? fadeOutElement($('landscape-option')) :
    fadeInElement($('landscape-option'));

[Albert Bodenhamer, 2011/06/10 23:13:45]

I have a strong dislike of the ternary operator for that sort of thing.  It
makes the code much less readable.

On 2011/06/10 20:29:07, dpapad wrote:

+  }
+
   disableColorOption ? fadeOutElement($('color-options')) :
       fadeInElement($('color-options'));
 
   if (colorOption.checked != setColorAsDefault) {
     colorOption.checked = setColorAsDefault;
     bwOption.checked = !setColorAsDefault;
     setColor(colorOption.checked);
   }
 }
 
 /**
+ * Turn on the integration of Cloud Print.
+ * @param {enable} true if cloud print should be used.
+ */
+function setUseCloudPrint(enable, cloudPrintURL) {
+  useCloudPrint = enable;
+  cloudprint.setBaseURL(cloudPrintURL);
+}
+
+/**
+ * Take the PDF data handed to us and submit it to the cloud, closing the print
+ * preview tab once the upload is successful.
+ * @param {pdfData} data to send as the print job.
+ */
+function printToCloud(data) {
+  cloudprint.printToCloud(data, finishedCloudPrinting);
+}
+
+/**
+ * Cloud print upload of the PDF file is finished, time to close the dialog.
+ */
+function finishedCloudPrinting() {
+  window.location = cloudprint.getBaseURL();
+}
+
+/**
+ * Disables or enables all controls in the options pane except for the cancel
+ * button.
+ */
+function setControlsDisabled(disabled) {

[dpapad, 2011/06/10 20:29:07]

This function was removed in earlier revisions, do you need it, or should you
rebase? I do not see it being called and we no longer disable any of the
controls, we just remove event listeners when needed.

[Albert Bodenhamer, 2011/06/10 23:13:45]

Done.

+  var elementList = $('controls').elements;
+  for (var i = 0; i < elementList.length; ++i) {
+    if (elementList[i] == $('cancel-button'))
+      continue;
+    elementList[i].disabled = disabled;
+  }
+}
+
+/**
  * Validates the copies text field value.
  * NOTE: An empty copies field text is considered valid because the blur event
  * listener of this field will set it back to a default value.
  * @return {boolean} true if the number of copies is valid else returns false.
  */
 function isNumberOfCopiesValid() {
   var copiesFieldText = $('copies').value.replace(/\s/g, '');
   if (copiesFieldText == '')
     return true;
 
@@ skipping 70 matching lines @@
   const LONG_EDGE = 1;
   return !isTwoSided() ? SIMPLEX : LONG_EDGE;
 }
 
 /**
  * Creates a JSON string based on the values in the printer settings.
  *
  * @return {string} JSON string with print job settings.
  */
 function getSettingsJSON() {
+  var printerList = $('printer-list');
+  var selectedPrinter = printerList.selectedIndex;
   var printAll = $('all-pages').checked;
   var deviceName = getSelectedPrinterName();
   var printToPDF = (deviceName == PRINT_TO_PDF);
 
-  return JSON.stringify({'deviceName': deviceName,
-                         'pageRange': getSelectedPageRanges(),
-                         'printAll': printAll,
-                         'duplex': getDuplexMode(),
-                         'copies': getCopies(),
-                         'collate': isCollated(),
-                         'landscape': isLandscape(),
-                         'color': isColor(),
-                         'printToPDF': printToPDF});
+  var settings = {'deviceName': deviceName,
+                  'pageRange': getSelectedPageRanges(),
+                  'printAll': printAll,
+                  'duplex': getDuplexMode(),
+                  'copies': getCopies(),
+                  'collate': isCollated(),
+                  'landscape': isLandscape(),
+                  'color': isColor(),
+                  'printToPDF': printToPDF};
+  if (printerList.options[selectedPrinter].isCloudPrint) {
+    settings['cloudPrintID'] =
+        printerList.options[selectedPrinter].value;
+  }
+
+  return JSON.stringify(settings);
 }
 
 /**
  * Returns the name of the selected printer or the empty string if no
  * printer is selected.
  */
 function getSelectedPrinterName() {
   var printerList = $('printer-list')
   var selectedPrinter = printerList.selectedIndex;
   var deviceName = '';
   if (selectedPrinter >= 0)
     deviceName = printerList.options[selectedPrinter].value;
   return deviceName;
 }
 
 /**
  * Asks the browser to print the preview PDF based on current print settings.
  */
 function printFile() {
   if (getSelectedPrinterName() != PRINT_TO_PDF) {
     $('print-button').classList.add('loading');
     $('cancel-button').classList.add('loading');
     $('print-summary').innerHTML = localStrings.getString('printing');
     removeEventListeners();
-    window.setTimeout(function() { chrome.send('print', [getSettingsJSON()]); },
-                      1000);
+    var printerList = $('printer-list')
+    var printer = printerList[printerList.selectedIndex];
+    window.setTimeout(function() { chrome.send('print',
+        [getSettingsJSON(), cloudprint.getPrintTicketJSON(printer)]); }, 1000);

[sanjeevr, 2011/06/10 18:13:48]

You probably need to check whether this is a cloud printer before calling
getPrintTicketJSON?

[Albert Bodenhamer, 2011/06/10 23:13:45]

getPrintTicketJSON now returns null for non-cloud printers.

On 2011/06/10 18:13:48, sanjeevr wrote:

   } else
-    chrome.send('print', [getSettingsJSON()]);
+    chrome.send('print', [getSettingsJSON(), null]);
 }
 
 /**
  * Asks the browser to generate a preview PDF based on current print settings.
  */
 function requestPrintPreview() {
   removeEventListeners();
   printSettings.save();
   showLoadingAnimation();
   chrome.send('getPreview', [getSettingsJSON()]);
@@ skipping 14 matching lines @@
   chrome.send('getPrinters');
 }
 
 /**
  * Fill the printer list drop down.
  * Called from PrintPreviewHandler::SendPrinterList().
  * @param {Array} printers Array of printer info objects.
  */
 function setPrinters(printers) {
   var printerList = $('printer-list');
+  if (useCloudPrint) {
+    cloudprint.fetchPrinters(addCloudPrinters);
+    if (printers.length > 0) {
+      option = addDestinationListOption('', '', false, true);
+      option = addDestinationListOption(localStrings.getString('localPrinters'),
+                                        '',
+                                        false,
+                                        true);
+    }
+  }
   // If there exists a dummy printer value, then setDefaultPrinter() already
   // requested a preview, so no need to do it again.
   var needPreview = (printerList[0].value == '');
   for (var i = 0; i < printers.length; ++i) {
     var isDefault = (printers[i].deviceName == defaultOrLastUsedPrinterName);
     addDestinationListOption(printers[i].printerName, printers[i].deviceName,
                              isDefault, false);
   }
 
   // Remove the dummy printer added in setDefaultPrinter().
   printerList.remove(0);
 
   if (printers.length != 0)
     addDestinationListOption('', '', false, true);
 
   // Adding option for saving PDF to disk.
   addDestinationListOption(localStrings.getString('printToPDF'),
                            PRINT_TO_PDF,
                            defaultOrLastUsedPrinterName == PRINT_TO_PDF,
                            false);
   addDestinationListOption('', '', false, true);
 
-  // Add an option to manage printers.
-  addDestinationListOption(localStrings.getString('managePrinters'),
-                           MANAGE_PRINTERS, false, false);
+  // Add options to manage printers.
+  if (!cr.isChromeOS) {
+    addDestinationListOption(localStrings.getString('manageLocalPrinters'),
+        MANAGE_LOCAL_PRINTERS, false, false);
+  }
+  if (useCloudPrint) {
+    addDestinationListOption(localStrings.getString('manageCloudPrinters'),
+        MANAGE_CLOUD_PRINTERS, false, false);
+  }
 
   printerList.disabled = false;
 
   if (needPreview)
     updateControlsWithSelectedPrinterCapabilities();
 }
 
 /**
  * Adds an option to the printer destination list.
  * @param {String} optionText specifies the option text content.
  * @param {String} optionValue specifies the option value.
  * @param {boolean} isDefault is true if the option needs to be selected.
  * @param {boolean} isDisabled is true if the option needs to be disabled.

[dpapad, 2011/06/10 20:29:07]

Update comments since it is now returning a value.

[Albert Bodenhamer, 2011/06/10 23:13:45]

Done.

  */
 function addDestinationListOption(optionText, optionValue, isDefault,
     isDisabled) {
   var option = document.createElement('option');
   option.textContent = optionText;
   option.value = optionValue;
   $('printer-list').add(option);
   option.selected = isDefault;
   option.disabled = isDisabled;
+  return option;
 }
 
 /**
+ * Add cloud printers to the list drop down.
+ * Called from the cloudprint object on receipt of printer information from the
+ * cloud print server.
+ */
+function addCloudPrinters(printers) {
+  addDestinationListOption(localStrings.getString('cloudPrinters'),
+                           '',
+                           false,
+                           true);
+  if (printers != null) {
+    var l = printers.length;
+    if (l > maxCloudPrinters) {
+      l = maxCloudPrinters;
+    }
+    for (var i = 0; i < l; i++) {
+      var option = addDestinationListOption(printers[i]['name'],
+                                            printers[i]['id'],
+                                            i == 0,

[dpapad, 2011/06/10 20:29:07]

I believe this should be 
printers[i]['name'] == defaultOrLastUsedPrinterName
otherwise the default or last used printer will not be pre-selected.

[Albert Bodenhamer, 2011/06/10 23:13:45]

Done.

+                                            false);
+      option.isCloudPrint = true;
+    }
+    if (printers.length == 0) {
+      addDestinationListOption(localStrings.getString('addCloudPrinter'),
+          ADD_PRINTER, false, false);
+    }
+    if (l < printers.length) {
+      var option = addDestinationListOption('', '', false, true);
+      addDestinationListOption(localStrings.getString('morePrinters'),
+          MORE_PRINTERS, false, false);
+    }
+  } else {
+    addDestinationListOption(localStrings.getString('signIn'),
+        SIGN_IN, false, false);
+  }
+}
+
+/**
  * Sets the color mode for the PDF plugin.
  * Called from PrintPreviewHandler::ProcessColorSetting().
  * @param {boolean} color is true if the PDF plugin should display in color.
  */
 function setColor(color) {
   var pdfViewer = $('pdf-viewer');
   if (!pdfViewer) {
     return;
   }
   pdfViewer.grayscale(!color);
+  var printerList = $('printer-list')
+  cloudprint.setColor(printerList[printerList.selectedIndex], color);
 }
 
 /**
  * Display an error message in the center of the preview area.
  * @param {string} errorMessage The error message to be displayed.
  * @param {boolean} showButton Indivates whether the "Reopen the page" button
  * should be displayed.
  */
 function displayErrorMessage(errorMessage, showButton) {
   $('overlay-layer').classList.remove('invisible');
@@ skipping 510 matching lines @@
   this.isLandscape = '';
 }
 
 /**
  * Takes a snapshot of the print settings.
  */
 PrintSettings.prototype.save = function() {
   this.deviceName = getSelectedPrinterName();
   this.isLandscape = isLandscape();
 }
+