Print Preview: Add option to rasterize PDFs and add JPEG compression.

chrome/browser/resources/print_preview/settings/other_options_settings.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';
 
   /**
    * Represents a single option in the Other Options settings section.
    * @param {!print_preview.ticket_items.TicketItem} ticketItem The ticket item
@@ skipping 109 matching lines @@
    * UI component that renders checkboxes for various print options.
    * @param {!print_preview.ticket_items.Duplex} duplex Duplex ticket item.
    * @param {!print_preview.ticket_items.FitToPage} fitToPage Fit-to-page ticket
    *     item.
    * @param {!print_preview.ticket_items.CssBackground} cssBackground CSS
    *     background ticket item.
    * @param {!print_preview.ticket_items.SelectionOnly} selectionOnly Selection
    *     only ticket item.
    * @param {!print_preview.ticket_items.HeaderFooter} headerFooter Header
    *     footer ticket item.
+   * @param {!print_preview.ticket_items.Rasterize} rasterize Rasterize ticket
+   *     item.
    * @constructor
    * @extends {print_preview.SettingsSection}
    */
   function OtherOptionsSettings(
-      duplex, fitToPage, cssBackground, selectionOnly, headerFooter) {
+      duplex, fitToPage, cssBackground, selectionOnly, headerFooter,
+      rasterize) {
     print_preview.SettingsSection.call(this);
+    /**
+     * @private {boolean} rasterizeEnabled Whether the print as image feature is
+     *     enabled.
+     */
+    this.rasterizeEnabled_ = loadTimeData.getBoolean('printPdfAsImageEnabled');
 
-    /*
+    /**
      * @private {!Array<!CheckboxTicketItemElement>} checkbox ticket item
      *      elements representing the different options in the section.
      *      Selection only must always be the last element in the array.
      */
     this.elements_ = [
         new CheckboxTicketItemElement(headerFooter, true,

[dpapad, 2016/12/22 19:27:00]

Nit: Indentation should be 2 less.

[rbpotter, 2016/12/22 22:58:46]

Done.

                                       'header-footer-container'),
         new CheckboxTicketItemElement(fitToPage, false,
                                       'fit-to-page-container'),
         new CheckboxTicketItemElement(duplex, false, 'duplex-container'),
         new CheckboxTicketItemElement(cssBackground, true,
                                       'css-background-container'),
         new CheckboxTicketItemElement(selectionOnly, true,
                                       'selection-only-container')
     ];
-
+    if (this.rasterizeEnabled_) {
+      this.elements_.splice(4, 0, new CheckboxTicketItemElement(rasterize,

[dpapad, 2016/12/22 19:27:00]

Nit: This type of line breaking violates the "rectangle rule", see
https://google.github.io/styleguide/jsguide.html#formatting-rectangle-rule.

[rbpotter, 2016/12/22 22:58:46]

Done.

+            true, 'rasterize-container'));
+    }
   };
 
   OtherOptionsSettings.prototype = {
     __proto__: print_preview.SettingsSection.prototype,
 
     /** @override */
     isAvailable: function() {
       return this.elements_.some(function(element) {
         return element.ticketItem.isCapabilityAvailable();
       });
@@ skipping 35 matching lines @@
     exitDocument: function() {
       print_preview.SettingsSection.prototype.exitDocument.call(this);
       for (var i = 0; i < this.elements_.length; i++)
         this.elements_[i].exitDocument();
     },
 
     /** @override */
     decorateInternal: function() {
       for (var i = 0; i < this.elements_.length; i++)
         this.elements_[i].decorate();
+      if (!this.rasterizeEnabled_)
+        $('rasterize-container').style.display = 'none';

[dpapad, 2016/12/22 19:27:00]

Nit (optional): I think the following would also work (did not try it though).

$('rasterize-container').hidden = !this.rasterizeEnabled_;

[rbpotter, 2016/12/22 22:58:46]

Done.

     },
 
     /** @override */
     updateUiStateInternal: function() {
       if (this.isAvailable()) {
         for (var i = 0; i < this.elements_.length; i++)
           this.elements_[i].setVisibility(this.collapseContent);
       }
       print_preview.SettingsSection.prototype.updateUiStateInternal.call(this);
     },
 
     /** @override */
     isSectionVisibleInternal: function() {
       return this.elements_.some(function(element) {
         return element.isVisible(this.collapseContent);
       }, this);
     },
 
   };
 
   // Export
   return {
     OtherOptionsSettings: OtherOptionsSettings
   };
 });