Print Preview: Fix data/ errors

chrome/browser/resources/print_preview/data/ticket_items/custom_margins.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.exportPath('print_preview.ticket_items');
+
+/**
+ * Enumeration of the orientations of margins.
+ * @enum {string}
+ */
+print_preview.ticket_items.CustomMarginsOrientation = {
+  TOP: 'top',
+  RIGHT: 'right',
+  BOTTOM: 'bottom',
+  LEFT: 'left'
+};
+
 cr.define('print_preview.ticket_items', function() {
   'use strict';
 

[dpapad, 2017/05/05 17:32:50]

Nit (optional): You might be able to create a shorter alias here as follows

var CustomMarginsOrientation =
print_preview.ticket_items.CustomMarginsOrientation;

and use CustomMarginsOrientation in actual code below this point to keep it as
short as it was.

[rbpotter, 2017/05/05 18:25:46]

Done.

   /**
    * Custom page margins ticket item whose value is a
    * {@code print_preview.Margins}.
    * @param {!print_preview.AppState} appState App state used to persist custom
    *     margins.
    * @param {!print_preview.DocumentInfo} documentInfo Information about the
    *     document to print.
    * @constructor
    * @extends {print_preview.ticket_items.TicketItem}
    */
   function CustomMargins(appState, documentInfo) {
     print_preview.ticket_items.TicketItem.call(
         this,
         appState,
-        print_preview.AppState.Field.CUSTOM_MARGINS,
+        print_preview.AppStateField.CUSTOM_MARGINS,
         null /*destinationStore*/,
         documentInfo);
-  };
-
-  /**
-   * Enumeration of the orientations of margins.
-   * @enum {string}
-   */
-  CustomMargins.Orientation = {
-    TOP: 'top',
-    RIGHT: 'right',
-    BOTTOM: 'bottom',
-    LEFT: 'left'
-  };
+  }
 
   /**
    * Mapping of a margin orientation to its opposite.
-   * @type {!Object<!print_preview.ticket_items.CustomMargins.Orientation,
-   *                 !print_preview.ticket_items.CustomMargins.Orientation>}
+   * @type {!Object<!print_preview.ticket_items.CustomMarginsOrientation,
+   *                 !print_preview.ticket_items.CustomMarginsOrientation>}
    * @private
    */
   CustomMargins.OppositeOrientation_ = {};
-  CustomMargins.OppositeOrientation_[CustomMargins.Orientation.TOP] =
-      CustomMargins.Orientation.BOTTOM;
-  CustomMargins.OppositeOrientation_[CustomMargins.Orientation.RIGHT] =
-      CustomMargins.Orientation.LEFT;
-  CustomMargins.OppositeOrientation_[CustomMargins.Orientation.BOTTOM] =
-      CustomMargins.Orientation.TOP;
-  CustomMargins.OppositeOrientation_[CustomMargins.Orientation.LEFT] =
-      CustomMargins.Orientation.RIGHT;
+  CustomMargins.OppositeOrientation_[
+      print_preview.ticket_items.CustomMarginsOrientation.TOP] =
+      print_preview.ticket_items.CustomMarginsOrientation.BOTTOM;
+  CustomMargins.OppositeOrientation_[
+      print_preview.ticket_items.CustomMarginsOrientation.RIGHT] =
+      print_preview.ticket_items.CustomMarginsOrientation.LEFT;
+  CustomMargins.OppositeOrientation_[
+      print_preview.ticket_items.CustomMarginsOrientation.BOTTOM] =
+      print_preview.ticket_items.CustomMarginsOrientation.TOP;
+  CustomMargins.OppositeOrientation_[
+      print_preview.ticket_items.CustomMarginsOrientation.LEFT] =
+      print_preview.ticket_items.CustomMarginsOrientation.RIGHT;
 
   /**
    * Minimum distance in points that two margins can be separated by.
    * @type {number}
    * @const
    * @private
    */
   CustomMargins.MINIMUM_MARGINS_DISTANCE_ = 72; // 1 inch.
 
   CustomMargins.prototype = {
     __proto__: print_preview.ticket_items.TicketItem.prototype,
 
     /** @override */
     wouldValueBeValid: function(value) {
       var margins = /** @type {!print_preview.Margins} */ (value);
-      for (var key in CustomMargins.Orientation) {
-        var o = CustomMargins.Orientation[key];
+      for (var key in print_preview.ticket_items.CustomMarginsOrientation) {
+        var o = print_preview.ticket_items.CustomMarginsOrientation[key];
         var max = this.getMarginMax_(
             o, margins.get(CustomMargins.OppositeOrientation_[o]));
         if (margins.get(o) > max || margins.get(o) < 0) {
           return false;
         }
       }
       return true;
     },
 
     /** @override */
     isCapabilityAvailable: function() {
       return this.getDocumentInfoInternal().isModifiable;
     },
 
     /** @override */
     isValueEqual: function(value) {
       return this.getValue().equals(value);
     },
 
     /**
-     * @param {!print_preview.ticket_items.CustomMargins.Orientation}
+     * @param {!print_preview.ticket_items.CustomMarginsOrientation}
      *     orientation Specifies the margin to get the maximum value for.
      * @return {number} Maximum value in points of the specified margin.
      */
     getMarginMax: function(orientation) {
       var oppositeOrient = CustomMargins.OppositeOrientation_[orientation];
       var margins = /** @type {!print_preview.Margins} */ (this.getValue());
       return this.getMarginMax_(orientation, margins.get(oppositeOrient));
     },
 
     /** @override */
     updateValue: function(value) {
       var margins = /** @type {!print_preview.Margins} */ (value);
       if (margins != null) {
         margins = new print_preview.Margins(
-            Math.round(margins.get(CustomMargins.Orientation.TOP)),
-            Math.round(margins.get(CustomMargins.Orientation.RIGHT)),
-            Math.round(margins.get(CustomMargins.Orientation.BOTTOM)),
-            Math.round(margins.get(CustomMargins.Orientation.LEFT)));
+            Math.round(margins.get(print_preview.ticket_items.
+                                   CustomMarginsOrientation.TOP)),
+            Math.round(margins.get(print_preview.ticket_items.
+                                   CustomMarginsOrientation.RIGHT)),
+            Math.round(margins.get(print_preview.ticket_items.
+                                   CustomMarginsOrientation.BOTTOM)),
+            Math.round(margins.get(print_preview.ticket_items.
+                                   CustomMarginsOrientation.LEFT)));
       }
       print_preview.ticket_items.TicketItem.prototype.updateValue.call(
           this, margins);
     },
 
     /**
      * Updates the specified margin in points while keeping the value within
      * a maximum and minimum.
-     * @param {!print_preview.ticket_items.CustomMargins.Orientation}
+     * @param {!print_preview.ticket_items.CustomMarginsOrientation}
      *     orientation Specifies the margin to update.
      * @param {number} value Updated margin value in points.
      */
     updateMargin: function(orientation, value) {
       var margins = /** @type {!print_preview.Margins} */ (this.getValue());
       var oppositeOrientation = CustomMargins.OppositeOrientation_[orientation];
       var max =
           this.getMarginMax_(orientation, margins.get(oppositeOrientation));
       value = Math.max(0, Math.min(max, value));
       this.updateValue(margins.set(orientation, value));
     },
 
     /** @override */
     getDefaultValueInternal: function() {
       return this.getDocumentInfoInternal().margins ||
              new print_preview.Margins(72, 72, 72, 72);
     },
 
     /** @override */
     getCapabilityNotAvailableValueInternal: function() {
       return this.getDocumentInfoInternal().margins ||
              new print_preview.Margins(72, 72, 72, 72);
     },
 
     /**
-     * @param {!print_preview.ticket_items.CustomMargins.Orientation}
+     * @param {!print_preview.ticket_items.CustomMarginsOrientation}
      *     orientation Specifies which margin to get the maximum value of.
      * @param {number} oppositeMargin Value of the margin in points
      *     opposite the specified margin.
      * @return {number} Maximum value in points of the specified margin.
      * @private
      */
     getMarginMax_: function(orientation, oppositeMargin) {
-      var dimensionLength = (orientation == CustomMargins.Orientation.TOP ||
-                             orientation == CustomMargins.Orientation.BOTTOM) ?
-                            this.getDocumentInfoInternal().pageSize.height :
-                            this.getDocumentInfoInternal().pageSize.width;
+      var dimensionLength =
+          (orientation ==
+           print_preview.ticket_items.CustomMarginsOrientation.TOP ||
+           orientation ==
+           print_preview.ticket_items.CustomMarginsOrientation.BOTTOM) ?
+          this.getDocumentInfoInternal().pageSize.height :
+          this.getDocumentInfoInternal().pageSize.width;
 
       var totalMargin = dimensionLength -
                         CustomMargins.MINIMUM_MARGINS_DISTANCE_;
       return Math.round(totalMargin > 0 ? totalMargin - oppositeMargin : 0);
     }
   };
 
   // Export
   return {
     CustomMargins: CustomMargins
   };
 });