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

Side by Side Diff: chrome/browser/resources/print_preview/data/page_number_set.js

Issue 574193002: Compile print_preview, part 1: remove all warnings (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@true_master
Patch Set: rebase Created 6 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 cr.define('print_preview', function() { 5 cr.define('print_preview', function() {
6 'use strict'; 6 'use strict';
7 7
8 /** 8 /**
9 * An immutable ordered set of page numbers. 9 * An immutable ordered set of page numbers.
10 * @param {!Array.<number>} pageNumberList A list of page numbers to include 10 * @param {!Array.<number>} pageNumberList A list of page numbers to include
(...skipping 17 matching lines...) Expand all
28 28
29 /** 29 /**
30 * @param {number} index 0-based index of the page number to get. 30 * @param {number} index 0-based index of the page number to get.
31 * @return {number} Page number at the given index. 31 * @return {number} Page number at the given index.
32 */ 32 */
33 getPageNumberAt: function(index) { 33 getPageNumberAt: function(index) {
34 return this.pageNumberSet_[index]; 34 return this.pageNumberSet_[index];
35 }, 35 },
36 36
37 /** 37 /**
38 * @param {number} 1-based page number to check for. 38 * @param {number} pageNumber 1-based page number to check for.
39 * @return {boolean} Whether the given page number is in the page range. 39 * @return {boolean} Whether the given page number is in the page range.
40 */ 40 */
41 hasPageNumber: function(pageNumber) { 41 hasPageNumber: function(pageNumber) {
42 return arrayContains(this.pageNumberSet_, pageNumber); 42 return arrayContains(this.pageNumberSet_, pageNumber);
43 }, 43 },
44 44
45 /** 45 /**
46 * @param {number} 1-based number of the page to get index of. 46 * @param {number} pageNumber 1-based number of the page to get index of.
47 * @return {number} 0-based index of the given page number with respect to 47 * @return {number} 0-based index of the given page number with respect to
48 * all of the pages in the page range. 48 * all of the pages in the page range.
49 */ 49 */
50 getPageNumberIndex: function(pageNumber) { 50 getPageNumberIndex: function(pageNumber) {
51 return this.pageNumberSet_.indexOf(pageNumber); 51 return this.pageNumberSet_.indexOf(pageNumber);
52 }, 52 },
53 53
54 /** @return {!Array.<number>} Array representation of the set. */ 54 /** @return {!Array.<number>} Array representation of the set. */
55 asArray: function() { 55 asArray: function() {
56 return this.pageNumberSet_.slice(0); 56 return this.pageNumberSet_.slice(0);
57 }, 57 },
58 }; 58 };
59 59
60 // Export 60 // Export
61 return { 61 return {
62 PageNumberSet: PageNumberSet 62 PageNumberSet: PageNumberSet
63 }; 63 };
64 }); 64 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698