Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(223)

Unified Diff: chrome/browser/resources/print_preview.js

Issue 6533006: Print Preview: Hook up the print button to initiate printing without displaying a print dialog. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: '' Created 9 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/print_preview.js
diff --git a/chrome/browser/resources/print_preview.js b/chrome/browser/resources/print_preview.js
index 6c3aee2931c4f0e13ab02b4fbc390e98a67eb779..16ec3ac520745029931917c9cc417a476e61a619 100644
--- a/chrome/browser/resources/print_preview.js
+++ b/chrome/browser/resources/print_preview.js
@@ -1,4 +1,4 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// 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.
@@ -9,9 +9,8 @@ var hasPDFPlugin = true;
* Window onload handler, sets up the page.
*/
function load() {
- $('print-button').addEventListener('click', function(e) {
- chrome.send('print');
- });
+ $('print-button').addEventListener('click', printFile);
+
$('cancel-button').addEventListener('click', function(e) {
window.close();
});
@@ -20,6 +19,78 @@ function load() {
};
/**
+ * Page range text validation.
+ * Returns true if |printFromText| and |printToText| are valid page numbers.
+ * TODO (kmadhusu): Get the expected page count and validate the page range
+ * with total number of pages.
+ */
+function isValidPageRange(printFromText, printToText) {
+ var numericExp = /^[0-9]+$/;
+ if (printFromText.match(numericExp) && printToText.match(numericExp)) {
+ var printFrom = parseInt(printFromText, 10);
+ var printTo = parseInt(printToText, 10);
+ if ((printFrom > 0) && (printTo > 0) && (printFrom <= printTo))
+ return true;
+ }
+ return false;
+}
+
+/**
+ * Parse page range text.
+ * Eg: If page range is specified as "1-3,7-9,8". Create an array with three
+ * elements. Each array element contains the range information.
+ * [{from:1, to:3}, {from:7, to:9}, {from:8, to:8}]
+ * TODO (kmadhusu): Handle invalid characters.
+ */
+function getPageRanges() {
+ var pageRangesInfo = new Array();
+ var pageRangeText = $('pages').value;
+ var pageRangeList = pageRangeText.split(',');
+ for (var i = 0; i < pageRangeList.length; i++) {
+ var tempRange = pageRangeList[i].split('-');
+ var printFrom;
+ var printTo;
+ if (tempRange.length > 1) {
+ printFrom = tempRange[0];
+ printTo = tempRange[1];
+ } else {
+ printFrom = tempRange[0];
+ printTo = tempRange[0];
+ }
+ // Validate the page range information.
+ if (isValidPageRange(printFrom, printTo)) {
+ var pageRangeParams = new Object();
+ pageRangeParams.from = parseInt(printFrom, 10);
+ pageRangeParams.to = parseInt(printTo, 10);
+ pageRangesInfo.push(pageRangeParams);
+ }
+ }
+ return pageRangesInfo;
+}
+
+function printFile() {
+ var selectedPrinter = $('printer-list').selectedIndex;
+ var printerName = $('printer-list').options[selectedPrinter].textContent;
+ var pageRanges = getPageRanges();
+ var printAll = $('all-pages').checked;
+ var twoSided = $('two-sided').checked;
+ var copies = $('copies').value;
+ var collate = $('collate').checked;
+ var layout = $('layout').options[$('layout').selectedIndex].value;
+ var color = $('color').options[$('color').selectedIndex].value;
+
+ var jobSettings = JSON.stringify({"printerName": printerName,
+ "pageRange": pageRanges,
Lei Zhang 2011/02/18 05:21:42 nit: indentation.
kmadhusu 2011/02/21 01:30:26 Done.
+ "printAll": printAll,
+ "twoSided": twoSided,
+ "copies": copies,
+ "collate": collate,
+ "layout": layout,
+ "color": color});
+ chrome.send('print', [jobSettings]);
+}
+
+/**
* Fill the printer list drop down.
*/
function setPrinters(printers) {

Powered by Google App Engine
This is Rietveld 408576698