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

Issue 160683003: Collect touch scroll latency for JS driven scroll (Closed)

Created:
6 years, 10 months ago by Yufeng Shen (Slow to review)
Modified:
6 years, 10 months ago
Reviewers:
nduca
CC:
chromium-reviews, chrome-speed-team+watch_google.com
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

Collect touch scroll latency for JS driven scroll tough_scheduling_cases have several cases touch_handler_scrolling.html?* that use touch move handler to scroll the page. This CL adds the touch scroll latency collection for those cases. BUG=246034 TEST=smoothness test on touch_scheduling_cases.touch_handler_scrolling.html?* now show mean_touch_scroll_latency. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=250861

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -4 lines) Patch
M tools/perf/metrics/rendering_stats.py View 2 chunks +6 lines, -4 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Yufeng Shen (Slow to review)
6 years, 10 months ago (2014-02-12 21:12:52 UTC) #1
nduca
lgtm
6 years, 10 months ago (2014-02-12 21:28:38 UTC) #2
Yufeng Shen (Slow to review)
The CQ bit was checked by miletus@chromium.org
6 years, 10 months ago (2014-02-12 21:45:07 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/miletus@chromium.org/160683003/1
6 years, 10 months ago (2014-02-12 21:49:00 UTC) #4
commit-bot: I haz the power
6 years, 10 months ago (2014-02-13 00:19:14 UTC) #5
Message was sent while issue was closed.
Change committed as 250861

Powered by Google App Engine
This is Rietveld 408576698