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

Issue 200843002: Convert smoothness to the new timeline based metric API. (Closed)

Created:
6 years, 9 months ago by nednguyen
Modified:
6 years, 9 months ago
Reviewers:
nduca, ernstm, Sami
CC:
chromium-reviews, telemetry+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Patch Set 4 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+212 lines, -172 lines) Patch
M tools/perf/measurements/repaint.py View 1 3 1 chunk +7 lines, -7 lines 0 comments Download
M tools/perf/measurements/smoothness.py View 1 3 2 chunks +7 lines, -7 lines 0 comments Download
A tools/perf/measurements/smoothness_controller.py View 1 chunk +65 lines, -0 lines 0 comments Download
M tools/perf/measurements/smoothness_unittest.py View 2 chunks +45 lines, -0 lines 0 comments Download
M tools/perf/measurements/timeline_based_measurement.py View 1 3 5 chunks +21 lines, -4 lines 0 comments Download
M tools/perf/measurements/timeline_based_measurement_unittest.py View 1 2 3 3 chunks +9 lines, -6 lines 0 comments Download
M tools/perf/metrics/smoothness.py View 2 chunks +21 lines, -81 lines 0 comments Download
D tools/perf/metrics/smoothness_unittest.py View 1 chunk +0 lines, -57 lines 0 comments Download
M tools/perf/metrics/timeline.py View 1 chunk +6 lines, -1 line 0 comments Download
M tools/perf/metrics/timeline_based_metric.py View 1 chunk +1 line, -0 lines 0 comments Download
M tools/perf/metrics/timeline_interaction_record.py View 2 chunks +16 lines, -8 lines 0 comments Download
M tools/perf/metrics/timeline_interaction_record_unittest.py View 1 chunk +1 line, -1 line 0 comments Download
M tools/telemetry/telemetry/page/page.py View 2 chunks +13 lines, -0 lines 0 comments Download

Messages

Total messages: 27 (0 generated)
nduca
lgtm when you're ready
6 years, 9 months ago (2014-03-14 20:23:45 UTC) #1
nednguyen
The CQ bit was checked by nednguyen@google.com
6 years, 9 months ago (2014-03-18 16:30:11 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/nednguyen@google.com/200843002/1
6 years, 9 months ago (2014-03-18 16:35:02 UTC) #3
nednguyen
The CQ bit was unchecked by nednguyen@google.com
6 years, 9 months ago (2014-03-18 19:35:52 UTC) #4
nednguyen
The CQ bit was checked by nednguyen@google.com
6 years, 9 months ago (2014-03-19 02:17:17 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/nednguyen@google.com/200843002/1
6 years, 9 months ago (2014-03-19 02:18:53 UTC) #6
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 9 months ago (2014-03-19 02:19:03 UTC) #7
commit-bot: I haz the power
Failed to apply patch for tools/perf/measurements/repaint.py: While running patch -p1 --forward --force --no-backup-if-mismatch; patching file ...
6 years, 9 months ago (2014-03-19 02:19:04 UTC) #8
nednguyen
The CQ bit was checked by nednguyen@google.com
6 years, 9 months ago (2014-03-19 02:23:10 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/nednguyen@google.com/200843002/20001
6 years, 9 months ago (2014-03-19 02:24:49 UTC) #10
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 9 months ago (2014-03-19 03:54:26 UTC) #11
commit-bot: I haz the power
Try jobs failed on following builders: tryserver.chromium on linux_chromium_chromeos_rel
6 years, 9 months ago (2014-03-19 03:54:27 UTC) #12
nednguyen
The CQ bit was checked by nednguyen@google.com
6 years, 9 months ago (2014-03-19 04:05:32 UTC) #13
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/nednguyen@google.com/200843002/20001
6 years, 9 months ago (2014-03-19 04:06:40 UTC) #14
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 9 months ago (2014-03-19 04:54:16 UTC) #15
commit-bot: I haz the power
Try jobs failed on following builders: tryserver.chromium on linux_chromium_chromeos_rel
6 years, 9 months ago (2014-03-19 04:54:17 UTC) #16
nednguyen
The CQ bit was checked by nednguyen@google.com
6 years, 9 months ago (2014-03-19 05:12:57 UTC) #17
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/nednguyen@google.com/200843002/20001
6 years, 9 months ago (2014-03-19 05:13:40 UTC) #18
nduca
if this is all flake, then uncheck commit bit, put NOTRY=True, check commit again and ...
6 years, 9 months ago (2014-03-19 10:28:35 UTC) #19
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 9 months ago (2014-03-19 12:07:20 UTC) #20
commit-bot: I haz the power
Failed to apply patch for tools/perf/measurements/repaint.py: While running patch -p1 --forward --force --no-backup-if-mismatch; patching file ...
6 years, 9 months ago (2014-03-19 12:07:20 UTC) #21
nednguyen
The CQ bit was checked by nednguyen@google.com
6 years, 9 months ago (2014-03-19 13:15:49 UTC) #22
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/nednguyen@google.com/200843002/40001
6 years, 9 months ago (2014-03-19 13:16:02 UTC) #23
commit-bot: I haz the power
Change committed as 257980
6 years, 9 months ago (2014-03-19 15:36:11 UTC) #24
nednguyen
The CQ bit was checked by nednguyen@google.com
6 years, 9 months ago (2014-03-19 16:35:17 UTC) #25
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/nednguyen@google.com/200843002/60001
6 years, 9 months ago (2014-03-19 16:39:07 UTC) #26
commit-bot: I haz the power
6 years, 9 months ago (2014-03-19 18:42:04 UTC) #27
Message was sent while issue was closed.
Change committed as 258045

Powered by Google App Engine
This is Rietveld 408576698