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

Unified Diff: chrome/browser/resources/print_preview/print_preview_animations.js

Issue 8605001: Print Preview: Fixing code style issues reported by gsjlint. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebasing Created 9 years, 1 month 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/print_preview/print_preview_animations.js
diff --git a/chrome/browser/resources/print_preview/print_preview_animations.js b/chrome/browser/resources/print_preview/print_preview_animations.js
index 87752c637085e32c40e5957331650684d7f35ce8..46aac350cbccb59dcc6f0eb4636f4e59cd78be12 100644
--- a/chrome/browser/resources/print_preview/print_preview_animations.js
+++ b/chrome/browser/resources/print_preview/print_preview_animations.js
@@ -26,7 +26,7 @@ function addAnimation(code) {
* @return {string} The css code for the fade in animation.
*/
function getFadeInAnimationCode(targetHeight) {
- return '0% { opacity: 0; height: 0; } ' +
+ return '0% { opacity: 0; height: 0; } ' +
'80% { height: ' + (targetHeight + 4) + 'px; }' +
'100% { opacity: 1; height: ' + targetHeight + 'px; }';
}
@@ -68,7 +68,7 @@ function fadeOutElement(el) {
var eventTracker = new EventTracker();
eventTracker.add(el, 'webkitTransitionEnd',
onFadeOutTransitionEnd.bind(el, animName, eventTracker),
- false );
+ false);
el.classList.add('closing');
el.classList.remove('visible');
}
@@ -76,8 +76,8 @@ function fadeOutElement(el) {
/**
* Executes when a fade out animation ends.
* @param {string} animationName The name of the animation to be removed.
- * @param {EventTracker} The |EventTracker| object that was used for adding this
- * listener.
+ * @param {EventTracker} eventTracker The |EventTracker| object that was used
+ * for adding this listener.
* @param {WebkitTransitionEvent} event The event that triggered this listener.
*/
function onFadeOutTransitionEnd(animationName, eventTracker, event) {
@@ -90,8 +90,8 @@ function onFadeOutTransitionEnd(animationName, eventTracker, event) {
/**
* Executes when a fade in animation ends.
- * @param{EventTracker} The |EventTracker| object that was used for adding this
- * listener.
+ * @param {EventTracker} eventTracker The |EventTracker| object that was used
+ * for adding this listener.
* @param {WebkitAnimationEvent} event The event that triggered this listener.
*/
function onFadeInAnimationEnd(eventTracker, event) {
« no previous file with comments | « chrome/browser/resources/print_preview/print_preview.js ('k') | chrome/browser/resources/print_preview/print_preview_cloud.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698