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

Unified Diff: chrome/browser/resources/pdf/elements/viewer-page-indicator/viewer-page-indicator.js

Issue 2915773003: PDF Plugin: Add compile targets for a few more files. (Closed)
Patch Set: Addressing comments. Created 3 years, 7 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/pdf/elements/viewer-page-indicator/viewer-page-indicator.js
diff --git a/chrome/browser/resources/pdf/elements/viewer-page-indicator/viewer-page-indicator.js b/chrome/browser/resources/pdf/elements/viewer-page-indicator/viewer-page-indicator.js
index 65b1be2ffc3ae948e3d305794ba4803fc3151d2a..09f1b8012388bdfca4d9cd9fdf78f80d8b44ad9f 100644
--- a/chrome/browser/resources/pdf/elements/viewer-page-indicator/viewer-page-indicator.js
+++ b/chrome/browser/resources/pdf/elements/viewer-page-indicator/viewer-page-indicator.js
@@ -23,8 +23,10 @@ Polymer({
}
},
+ /** @type {number|undefined} */
timerId: undefined,
+ /** @override */
ready: function() {
var callback = this.fadeIn.bind(this, 2000);
window.addEventListener('scroll', function() {
@@ -36,6 +38,7 @@ Polymer({
this.fadeIn(6000);
},
+ /** @param {number} displayTime */
fadeIn: function(displayTime) {
var percent = window.scrollY /
(document.body.scrollHeight -

Powered by Google App Engine
This is Rietveld 408576698