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

Unified Diff: ui/file_manager/file_manager/foreground/js/ui/scrollbar.js

Issue 2777513002: Remove resolved TODO (Closed)
Patch Set: Created 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/file_manager/file_manager/foreground/js/ui/scrollbar.js
diff --git a/ui/file_manager/file_manager/foreground/js/ui/scrollbar.js b/ui/file_manager/file_manager/foreground/js/ui/scrollbar.js
index a5f389869f3d1d1a00d2e6a50891558a73325fd4..d187c9630025ddd03526c2a278de54a9ade265e2 100644
--- a/ui/file_manager/file_manager/foreground/js/ui/scrollbar.js
+++ b/ui/file_manager/file_manager/foreground/js/ui/scrollbar.js
@@ -181,7 +181,6 @@ ScrollBar.prototype.onMouseMove_ = function(event) {
}
var clientSize = this.clientHeight_;
var totalSize = this.scrollHeight_;
- // TODO(hirono): Fix the geometric calculation. crbug.com/253779
var buttonSize = Math.max(50, clientSize / totalSize * clientSize);
var buttonPosition = this.buttonPressedPosition_ +
(event.screenY - this.buttonPressedEvent_.screenY);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698