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

Unified Diff: tools/telemetry/telemetry/page/actions/scroll.js

Issue 688423002: tools: Add tough_fast_scrolling_cases page set. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add fast scrolling cases to tough scrolling page set with a "fastscrolling" label instead Created 6 years, 2 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 | « tools/perf/page_sets/tough_scrolling_cases/lorem_ipsum.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/telemetry/telemetry/page/actions/scroll.js
diff --git a/tools/telemetry/telemetry/page/actions/scroll.js b/tools/telemetry/telemetry/page/actions/scroll.js
index 1e3337b1025f1159f2eacb0bc5c55709f36d542f..77beb678c6c51ff1ef563424a2f9504bdc619cf7 100644
--- a/tools/telemetry/telemetry/page/actions/scroll.js
+++ b/tools/telemetry/telemetry/page/actions/scroll.js
@@ -9,7 +9,7 @@
'use strict';
(function() {
- var MAX_SCROLL_LENGTH_PIXELS = 5000;
+ var MAX_SCROLL_LENGTH_TIME_MS = 6250;
function ScrollGestureOptions(opt_options) {
if (opt_options) {
@@ -90,7 +90,9 @@
ScrollAction.prototype.startGesture_ = function() {
this.beginMeasuringHook();
- var distance = Math.min(MAX_SCROLL_LENGTH_PIXELS,
+ var max_scroll_length_pixels = (MAX_SCROLL_LENGTH_TIME_MS / 1000) *
+ this.options_.speed_;
+ var distance = Math.min(max_scroll_length_pixels,
this.getScrollDistance_());
var rect = __GestureCommon_GetBoundingVisibleRect(this.options_.element_);
« no previous file with comments | « tools/perf/page_sets/tough_scrolling_cases/lorem_ipsum.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698