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

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

Issue 966293002: Make sure the PDF material toolbar gets hidden after loading. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 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
« no previous file with comments | « no previous file | chrome/browser/resources/pdf/ui_manager.js » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 418 matching lines...) Expand 10 before | Expand all | Expand 10 after
429 // Document load failed. 429 // Document load failed.
430 this.errorScreen_.style.visibility = 'visible'; 430 this.errorScreen_.style.visibility = 'visible';
431 this.sizer_.style.display = 'none'; 431 this.sizer_.style.display = 'none';
432 if (!this.isMaterial_) 432 if (!this.isMaterial_)
433 this.toolbar_.style.visibility = 'hidden'; 433 this.toolbar_.style.visibility = 'hidden';
434 if (this.passwordScreen_.active) { 434 if (this.passwordScreen_.active) {
435 this.passwordScreen_.deny(); 435 this.passwordScreen_.deny();
436 this.passwordScreen_.active = false; 436 this.passwordScreen_.active = false;
437 } 437 }
438 } else if (progress == 100) { 438 } else if (progress == 100) {
439 if (this.isMaterial_)
440 this.uiManager_.showUi();
Sam McNally 2015/03/02 02:01:01 How about setting the initial timeout in the const
raymes 2015/03/02 02:23:15 Done.
439 // Document load complete. 441 // Document load complete.
440 if (this.lastViewportPosition_) 442 if (this.lastViewportPosition_)
441 this.viewport_.position = this.lastViewportPosition_; 443 this.viewport_.position = this.lastViewportPosition_;
442 this.paramsParser_.getViewportFromUrlParams( 444 this.paramsParser_.getViewportFromUrlParams(
443 this.streamDetails_.originalUrl, this.handleURLParams_.bind(this)); 445 this.streamDetails_.originalUrl, this.handleURLParams_.bind(this));
444 this.loaded_ = true; 446 this.loaded_ = true;
445 this.sendScriptingMessage_({ 447 this.sendScriptingMessage_({
446 type: 'documentLoaded' 448 type: 'documentLoaded'
447 }); 449 });
448 while (this.delayedScriptingMessages_.length > 0) 450 while (this.delayedScriptingMessages_.length > 0)
(...skipping 350 matching lines...) Expand 10 before | Expand all | Expand 10 after
799 * Each bookmark is an Object containing a: 801 * Each bookmark is an Object containing a:
800 * - title 802 * - title
801 * - page (optional) 803 * - page (optional)
802 * - array of children (themselves bookmarks) 804 * - array of children (themselves bookmarks)
803 * @type {Array} the top-level bookmarks of the PDF. 805 * @type {Array} the top-level bookmarks of the PDF.
804 */ 806 */
805 get bookmarks() { 807 get bookmarks() {
806 return this.bookmarks_; 808 return this.bookmarks_;
807 } 809 }
808 }; 810 };
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/resources/pdf/ui_manager.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698