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

Side by Side Diff: chrome/browser/resources/pdf/pdf.js

Issue 1312373002: Material PDF: Style audit and cleanup (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix tests for page selector Created 5 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 'use strict'; 5 'use strict';
6 6
7 /** 7 /**
8 * @return {number} Width of a scrollbar in pixels 8 * @return {number} Width of a scrollbar in pixels
9 */ 9 */
10 function getScrollbarWidth() { 10 function getScrollbarWidth() {
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
78 /** 78 /**
79 * The minimum number of pixels to offset the toolbar by from the bottom and 79 * The minimum number of pixels to offset the toolbar by from the bottom and
80 * right side of the screen. 80 * right side of the screen.
81 */ 81 */
82 PDFViewer.MIN_TOOLBAR_OFFSET = 15; 82 PDFViewer.MIN_TOOLBAR_OFFSET = 15;
83 83
84 /** 84 /**
85 * The height of the toolbar along the top of the page. The document will be 85 * The height of the toolbar along the top of the page. The document will be
86 * shifted down by this much in the viewport. 86 * shifted down by this much in the viewport.
87 */ 87 */
88 PDFViewer.MATERIAL_TOOLBAR_HEIGHT = 64; 88 PDFViewer.MATERIAL_TOOLBAR_HEIGHT = 56;
89 89
90 /** 90 /**
91 * Creates a new PDFViewer. There should only be one of these objects per 91 * Creates a new PDFViewer. There should only be one of these objects per
92 * document. 92 * document.
93 * @constructor 93 * @constructor
94 * @param {!BrowserApi} browserApi An object providing an API to the browser. 94 * @param {!BrowserApi} browserApi An object providing an API to the browser.
95 */ 95 */
96 function PDFViewer(browserApi) { 96 function PDFViewer(browserApi) {
97 this.browserApi_ = browserApi; 97 this.browserApi_ = browserApi;
98 this.loadState_ = LoadState.LOADING; 98 this.loadState_ = LoadState.LOADING;
(...skipping 740 matching lines...) Expand 10 before | Expand all | Expand 10 after
839 * Each bookmark is an Object containing a: 839 * Each bookmark is an Object containing a:
840 * - title 840 * - title
841 * - page (optional) 841 * - page (optional)
842 * - array of children (themselves bookmarks) 842 * - array of children (themselves bookmarks)
843 * @type {Array} the top-level bookmarks of the PDF. 843 * @type {Array} the top-level bookmarks of the PDF.
844 */ 844 */
845 get bookmarks() { 845 get bookmarks() {
846 return this.bookmarks_; 846 return this.bookmarks_;
847 } 847 }
848 }; 848 };
OLDNEW
« no previous file with comments | « chrome/browser/resources/pdf/index-material.css ('k') | chrome/test/data/pdf/material_elements_test.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698